On Tue, 10 May 2011, Mikulas Patocka wrote: > > > On Mon, 9 May 2011, Jesper Juhl wrote: > > > On Sun, 8 May 2011, Mikulas Patocka wrote: > > > > > Move declaration up, so that there are no out-of-scope pointers > > > > > > Reported-by: Jesper Juhl <jj@xxxxxxxxxxxxx> > > > Signed-off-by: Mikulas Patocka <mikulas@xxxxxxxxxxxxxxxxxxxxxxxx> > > > > > So change it to: > > Signed-off-by: Jesper Juhl <jj@xxxxxxxxxxxxx> > Kinda hard when it's already merged in mainline ;-) > > Hmm. I'm a little currious how my original Signed-off-by got turned into a > > Reported-by for this patch? > > I don't know. I probably checked the code independently after your report > and ended up fixing exactly the same thing that you fixed. > Just seems a little odd that you would check the code based on my report (you must have been aware of it given the reported-by) and then ending up with the exact same fix would chose to submit a completely new patch with the exact same contents but with you as author... Ohh well, never mind - it's a trivial patch and the important thing is that it got merged. I just got a little annoyed at seeing a patch that I had submitted and even ping'ed people about later, getting merged with a different author and me just listed as 'reported-by'... I just care about these things, even for trivial stuff - it's a matter of "credit where credit is due" even when the change is minor... but let's just leave it where it is - the fix got merged, that's the really important bit :-) -- Jesper Juhl <jj@xxxxxxxxxxxxx> http://www.chaosbits.net/ Don't top-post http://www.catb.org/jargon/html/T/top-post.html Plain text mails only, please. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
