Boost logo

Boost :

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

Boost list run by bdawes at, gregod at, cpdaniel at, john at