[FFmpeg-devel] [PATCH] Mention unneeded casts and parenthesis in Coding Rules (doc)

[Vitor]

M?ns Rullg?rd wrote:
>>> Since these two things show up very frequently when reviewing patches,
>>> I think they are worth a mention in the documentation.
>>>
>>> Warning: I'm a non native english speaker!

[...]

>>> +Ideally, casts should be used only when necessary. Unneeded parenthesis
>>> +should also be avoided if they don't make the code easier to understand.
>>> +
>>>   
>> i think plural is parentheses
> 
> It is.  I'd also drop the "ideally".  An unnecessary cast is a bug.
> Actually, a necessary cast is also often a bug.
> 

Changed both.

-Vitor
-------------- next part --------------
A non-text attachment was scrubbed...
Name: doc2.diff
Type: text/x-patch
Size: 476 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070627/c5e6be68/attachment.bin>