https://defect.opensolaris.org/bz/show_bug.cgi?id=19108

Jens Elkner <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]
                   |                            |.de

--- Comment #2 from Jens Elkner <[email protected]> 2011-12-16 
19:54:13 UTC ---
(In reply to comment #1)
...
> In the default project configuration, this should not be an issue, because no
> custom tab size should be set. In 0.11rc this has changed though.
> Unintentionally, I guess. Now Configuration's constructor calls:
> 
>         setTabSize(4);

No, intentionally. Actually in 0.10 there was no way to specify the tab size
(and IIRC it wasn't even honored) and if one forced programatically to use
tabs, the result was unacceptable, because \t got always replaced with 8 spaces
... I also checked the CSS3 tab-size property (I actually what I would prefer
in the 1st place), but at this point no browser supported it. So I made tabsize
settings work and set it to 4 per default - not 8 because in the browser it
does not really matter because of nowrap, but with 4 spaces it is still
readable, compact wrt. line width and blows the content size less (bandwith).   

> The immediate fix for this problem is to remove that call from the constructor
> so that the default tab size is reverted to unspecified.

Or simply use -t 0 when calling the Indexer.

-- 
Configure bugmail: https://defect.opensolaris.org/bz/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
opengrok-dev mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/opengrok-dev

Reply via email to