Am 27.02.2012 18:22, schrieb Marcel Holtmann:
> while reading mmsd sources I stumbled over missing length
> checks in src/push.c:mms_push_notify(). I didn't re-read
> the entire source to prove overall ;)
please use git format-patch and git send-email to send us inline
patches. In case your email client doesn't support inline patches
Well, inline patches aren't requested at
. You probably should do.
Reviewing patches that are attached is super painful and in most
cases they get ignored then.
Well, my mail client (Thunderbird) and most modern clients I
know are able to show attached files of text/x-patch (and
similar) type. OTOH I wonder why you want to review patches
in your mail client. Wouldn't it much better to use a
specialized too which highlights the changes like
$ gvim "+vert diffpatch <patchfile>" <originalfile>
$ xfdiff ?
Don't you have to save the patch files in such cases anyway?