[FFmpeg-devel] [PATCH] avutil/parseutils: add av_warn_unused_result
michael at niedermayer.cc
Fri Oct 16 03:05:18 CEST 2015
On Thu, Oct 15, 2015 at 08:15:20PM -0400, Ganesh Ajjanagadde wrote:
> On Thu, Oct 15, 2015 at 8:06 PM, James Almer <jamrial at gmail.com> wrote:
> > On 10/15/2015 8:52 PM, Ganesh Ajjanagadde wrote:
> >> This triggers a few warnings that will need to be fixed - not that bad,
> >> the current code is mostly fine.
> > I think i asked you before, but in any case i'll do it again. Please,
> > send patchsets contained in a single thread. You're making things
> > hard to track and organize on people's inboxes.
> > See https://kernel.org/pub/software/scm/git/docs/git-send-email.html
> > --thread, --chain-reply-to and --in-reply-to options.
> I find the organization part hard to understand: the pattern
> av_warn_unused_result is easily grep-able and all mail clients I know
> of have some kind of label/folder option.
> Nevertheless, I will try to do this for the next commits. It is
> actually easier for me to simply attach all of them to a single email.
> Is that also fine?
I dont know about others but
I keep track of which patches i still want to look at by keeping the
mails marked as unread in my MUA.
If there are multiple patches in a single mail i cannot keep track of
them using that method
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
No human being will ever know the Truth, for even if they happen to say it
by chance, they would not even known they had done so. -- Xenophanes
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 181 bytes
Desc: Digital signature
More information about the ffmpeg-devel