Travis Vitek wrote:
Martin Sebor wrote:

Great, that fixes locale.cpp. Thanks for doing that. I was also
(or mainly) pointing out the same problems in braceexp.cpp. I
hesitate to commit a fix myself in case you're working on the
file but here's a patch that addresses the remaining problems:


Martin,

Your patch looks fine to me. Feel free to apply it.

I ended up checking in a slightly more extensive change to also
replace all occurrences of _RWSTD_SIZE_T with plain size_t for
readability.

http://svn.apache.org/viewvc?rev=644098&view=rev

Martin

Reply via email to