[FFmpeg-devel] [PATCH 03/11] ffserver_config: map ffserver options to AVOptions

Reynaldo H. Verdejo Pinochet reynaldo at osg.samsung.com
Fri Nov 21 23:17:17 CET 2014


Hi

On 11/20/2014 10:14 PM, Lukasz Marek wrote:
> On 18.11.2014 21:35, Reynaldo H. Verdejo Pinochet wrote:
> [...]
> Also, please make an effort
>> to break lines at 80 chars as long as it doesn't make the
>> code harder to read. This seems particularly possible on the
>> function declarations.
> 
> regarding this part, I set my editor to notice 90 chars and I try to
> respect that, but with some reasonable margin. TBH, 80 chars is
> prehistoric, probably from the era of crt's with text mode.
> I don't remember official ffmpeg's rules about that, but common, its
> 21th century...

https://www.ffmpeg.org/developer.html#toc-Code-formatting-conventions

Please try to when possible. It's not a requirement but a sane
suggestion many avid by.

Bests,

-- 
Reynaldo H. Verdejo Pinochet
Open Source Group
Samsung Research America / Silicon Valley


More information about the ffmpeg-devel mailing list