[FFmpeg-cvslog] r14267 - trunk/libavcodec/ra288.c

[Vitor Sessak]

Reimar D?ffinger wrote:
> Hello,
> On Fri, Jul 18, 2008 at 12:35:55PM +0200, Vitor Sessak wrote:
>> Diego Biurrun wrote:
>>> - at least mention which function you simplified.
>> That I'm not against. Are you ok with something like "Minor/Major 
>> simplification in decode()"?
> 
> Well, it is better, but obviously that works best when the function
> names are not completely worthless. In this case, where the function
> has such a completely useless name as "pred" (and no, it is not just
> a 3-line function) it obviously does not help much.

If ra288.c wasn't an ugly mess, I wouldn't spend my time cleaning it up ;-)

-Vitor


PS: if you want to see an even worse mess, looks at ra144.c at revision 
13009...