On 6/12/2012 1:44 PM, Peter Kümmel wrote:

I found a much simpler patch:

https://github.com/syntheticpp/ninja/commit/0ea34053a4692f190b8c13ce0ff032a57cece047


Could you please have a look at it before I create a merge request?

I would change RemoveQuotes to only remove them if they are at the start and end of the string.

Something like this:

if(str.size() == 0)
  return str;
if (str[0] == '\"' && str[str.size()-1] == '\"')
  {
  cleaned = str.substr(1, str.size()-2);
  return cleaned;
  }
return str;


-Bill
--

Powered by www.kitware.com

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers

Reply via email to