Joel de Guzman wrote:
James Fowler wrote:
Joel de Guzman wrote:
We need a program options flag for this which defaults
to 4 spaces. Thoughts?
It's converting tabs to four spaces now, which seems to be the
"preferred" indent scheme for Boost. The recommended approach to tabs
in Boost code is to simply avoid them
<http://www.boost.org/more/lib_guide.htm#Tabs>, which IMHO is the
right thing to do for Boost. I'm a little wary of adding flags or
options which make it easier to support tabs which probably shouldn't
be there in the first place in any code included in Boost docs, but we
can also deal with that by just issuing nasty warnings whenever the
highlighting process encounters tabs... Anyhow, the "flaky" part is
that the current tab expansion is simply "tab" -> "space space space
space", which works pretty well for leading indentation, but thrashes
stuff like
[...]
because it's not smart enough to replace a tab with "enough spaces to
reach the next tab stop". We can handle this through doing tab
expansion just before we process the highlighting, but this isn't
really a high priority for me. If anyone is interested, I can point
them to where it would need to go, or graft it in given a tab
expansion algorithm which works on standard iterators or std::string.
Right. In light of this, I am now leaning towards simply issuing
a warning and not doing any tab to space conversion at all.
I think that's the right way to go.
I took a peek at the code, and, alas... it's innevitable, we need
to handle tabs. Take a peek at list_format_action, for example.
The quickbook syntax relies on block layout (ala python) in
certain places such as list handling. We can issue a warning,
sure, but we'll still have to correctly handle them when they
are present, otherwise, block formatting will go awry.
Cheers,
--
Joel de Guzman
http://www.boost-consulting.com
http://spirit.sf.net
-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
Boost-docs mailing list
[email protected]
Unsubscribe and other administrative requests:
https://lists.sourceforge.net/lists/listinfo/boost-docs
