Developer forums (C::B DEVELOPMENT STRICTLY!) > Development
Add more styling options to class wizard
oBFusCATed:
--- Quote from: sodev on March 12, 2019, 02:56:07 pm ---The key point is to write flexible, reusable code and to achieve that...
--- End quote ---
I've stopped reading after this point, sorry. You just repeat what all those modern c++ people say.
BlueHazzard:
Ok, i will look into this as soon as i find time and commit it with your comments fixed.
--- Code: ---(const auto& x : container)
--- End code ---
is my preferred since some performance problems lately :)
--- Quote ---4. Do not prefix enum types with 'e'.
--- End quote ---
Why?
oBFusCATed:
I don't see a reason to hide the type in the ranged-based-for loops. If you like it fine. But if you do it in a code where I'm interested in looking I'll just change it.
4. Because it is ugly and I think types must start at least with capital letter according the the limited style guide. Also any form of Hungarian notation is stupid.
sodev:
--- Quote from: oBFusCATed on March 18, 2019, 08:35:58 am ---I don't see a reason to hide the type in the ranged-based-for loops.
--- End quote ---
--- Quote from: oBFusCATed on March 18, 2019, 08:35:58 am ---Also any form of Hungarian notation is stupid.
--- End quote ---
I wouldnt have expected to see both sentences from one person in one post :D. Sadly it is only CodeBlocks that "hides" the type of auto variables, pretty much every other IDE shows you their type (hint, hint ;)).
oBFusCATed:
@sodev: You see the type in notepad++, github, simple patches shown in text editors? I review a lot of code and using auto makes it a lot harder for me to do a good job.
BTW, even clangd didn't show the type for auto variables back when I've tried it last year. Hopefully they'll implement this someday.
Navigation
[0] Message Index
[#] Next page
[*] Previous page
Go to full version