From: Olly B. <ol...@su...> - 2008-07-29 11:35:15
|
On 2008-07-26, William S Fulton <ws...@fu...> wrote: > - Remove STL usage and replace with DOH containers/string. Sorry! Don't forget that you offered to convert patches from STL to DOH: http://thread.gmane.org/gmane.comp.programming.swig.devel/18074/focus=18109 "With regard to patches, I'd be happy to take patches that use the standard library and I'd convert them to DOH in the mean time." Personally, I think we'll be stuck will DOH forever if we insist on an all-in-one conversion because it's just too big a change to make in one go without breaking things. I bet we'd be shaking new bugs out for ages. But sadly I don't have the time to work on even a piecemeal conversion at present though, and I suspect that's true for all of us. It's certainly true that having to mess around with DOH puts me off working with SWIG more than I do. DOH is a very cute hack, but it's trying to create the feel of a scripting language, yet requires explicit memory management, and using it wrongly can lead to undefined behaviour. If we want the feel of a scripting language, we should use a scripting language (and we could even mix scripting language and C/C++ code - we have a tool for that!) Random patchwork use of DOH and STL within the same area certainly isn't a good place to be, but STL use within particular modules seems less problematic to me. It certainly seems unhelpful to be forcing would be contributors to jump through the extra hoop of learning DOH. Cheers, Olly |