← Back to team overview

widelands-dev team mailing list archive

Re: [Merge] lp:~widelands-dev/widelands/compiler_warnings into lp:widelands

 

I am bit nervous about all that static_casts. All variable that can be reworked to uint32_t should be reworked. I know this might be a lot of work...

Alternativelly I would vote for assert(x>=0) before all (remaining) static_casts. To be on safe side...
-- 
https://code.launchpad.net/~widelands-dev/widelands/compiler_warnings/+merge/278240
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/compiler_warnings into lp:widelands.


References