[boost] [property_tree] Re: Volunteering to fix tab and min/max issues

Subject: [boost] [property_tree] Re: Volunteering to fix tab and min/max issues
From: Jeremiah Willcock (jewillco_at_[hidden])
Date: 2009-05-20 15:47:05

On Wed, 20 May 2009, Eric Niebler wrote:

> Jeremiah Willcock wrote:
>> Since tabs in source files and min/max guideline violations are so easy to 
>> correct, is it OK if I just fix them in all libraries (except for the 
>> inspection tool's examples)?  I will not do any that appear to be 
>> intentional or that would change the meaning of the code.  Do I have 
>> permission to do that and just commit the changes?
>
> That's fine by me. And let me also say: thanks!

I've done most of these.  I also filed a bug about the inspect tool 
reporting min/max problems in comments; strings might also need to be 
skipped although I put extra quotes in the one example that had that 
issue.  There were two files that had tabs that might or might not have 
been intentional (they were in test data strings):

libs/property_tree/test/test_json_parser.cpp: *Tab*
libs/property_tree/test/xml_parser_test_data.hpp: *Tab*</pre>

Should I change those tabs to spaces as well, or replace them with \t, or 
something else?

-- Jeremiah Willcock