Julien Gilli wrote:
Hello,
Jean-Philippe Barrette-LaPierre a écrit :
I attached a modified version of the patch including Vadim suggestion.
IMHO, the typedef brings some confusion : it would make StringList a
type and a type alias. I'd personnaly prefer using std::list<string>
explicitly than a typedef which has the same name than an exisiting
class.
This approach (using explicit std::list<string>) everywhere signficantly
lowers Signal/Noice ratio and makes code MUCH less lisible. If You
believe that StringList name could create confusion
(i don't think so, but anyway) we can use 'strlist' for example
Vadim
_______________________________________________
Wengophone-devel mailing list
[email protected]
http://dev.openwengo.com/mailman/listinfo/wengophone-devel
