[FFmpeg-cvslog] r14363 - in trunk/libavcodec: ra288.c ra288.h

[Benoit Fouet]

M?ns Rullg?rd wrote:
> Vitor Sessak wrote:
>   
>> M?ns Rullg?rd wrote:
>>     
>>> Benoit Fouet wrote:
>>>       
>>>> M?ns Rullg?rd wrote:
>>>>         
>>>>> Diego Biurrun wrote:
>>>>>
>>>>>           
>>>>>> On Thu, Jul 24, 2008 at 06:15:50AM +0200, vitor wrote:
>>>>>>
>>>>>>             
>>>>>>> New Revision: 14363
>>>>>>>
>>>>>>> Log:
>>>>>>> More tables renaming/commenting
>>>>>>>
>>>>>>>               
>>>>>> Hmmm, now r14363 has
>>>>>>
>>>>>>   More tables renaming/commenting
>>>>>>
>>>>>> as the commit message, while r14360 has
>>>>>>
>>>>>>   Give tables a more descriptive name
>>>>>>
>>>>>> This is not good, since both commits are hard to tell apart.  Could you
>>>>>> please add the table names to the commit messages?
>>>>>>
>>>>>>             
>>>>> And there's still no clue left behind to indicate that this all applies
>>>>> to ra288.
>>>>>
>>>>>           
>>>> apart from the filename, of course...
>>>>         
>>> Which isn't displayed by any of the normal log viewing methods.
>>>       
>> Why do you post such messages? You already know what people will answer
>> to you ("Then fix the tools."). If you are nostalgic of this flamewar,
>> go read the logs. Or please come up with new (and hopefully better)
>> arguments.
>>     
>
> Why do you continue to deny that I would find better log messages useful?
>   

nobody does that...

> THE FILENAME ISN'T THE FULL STORY, DAMN IT!!!!!!!
>   

in this case, the log message + filename is enough, I don't see your point

-- 
Benoit Fouet
Purple Labs S.A.
www.purplelabs.com