From: Tom H. <tom...@pa...> - 2010-08-18 14:58:14
|
And now to list as well :P Those warnings are legitimate and helpful, but for some reason people find adding __UNUSED__ to parameters annoying so I decided to give this one up in order to get the general acceptance about the rest :P When it comes to warnings (and parties) I really believe in "the more the merrier", I compile my own projects with -Wall -Wextra -Werror, but I don't think -Werror is relevant for efl because things would just break on mac/windows/weird platforms, and that's just not acceptable. -- Tom. On Wed, 2010-08-18 at 10:35 -0400, Christopher Michael wrote: > Personally I like the idea. It would certainly help spot errors in code > and could help enforce cleaner code from others. WRT unused paramaters > tho, I would lean toward the side of not disabling those tho because > there are some situations where those warnings are legitimate. > > dh > > On 08/18/2010 10:27 AM, Tom Hacohen wrote: > > Hey all, > > > > Wall and Wextra are great compilation flags they help finding a lot of > > coding mistakes. Warnings about unused parameters is a bit annoying in a > > library that has a lot of callbacks, so disabling that makes sense, but > > other than that, all warnings are welcomed. > > > > What do you guys say about making "-Wall -Wextra -Wno-unused-parameter" > > our new compilation flags default? > > > > -- > > Tom. > > > > > ------------------------------------------------------------------------------ > This SF.net email is sponsored by > > Make an app they can't live without > Enter the BlackBerry Developer Challenge > http://p.sf.net/sfu/RIM-dev2dev > _______________________________________________ > enlightenment-devel mailing list > enl...@li... > https://lists.sourceforge.net/lists/listinfo/enlightenment-devel |