Re: [cssparser-developers] TAB handling

[Alan K. <al...@tr...>]

Waldbaer wrote:
> I wondered why the columnNumbers for characters following a TAB 
> character are so big. The TAB character seems to be treated like 8 
> characters. I found the following:
>
> com.steadystate.css.parser.ASCII_CharStream.UpdateLineColumn(char c), 
> line 162:
>
>          case '\t' :
>              this.column--;
>              this.column += (8 - (this.column & 07));
>              break;
>
> Why is this special treatment for TAB?
>   
According to the SourceForge CVS browser, that looks like it was one of 
your changes.  :-)

http://cssparser.cvs.sourceforge.net/viewvc/cssparser/cssparser/src/com/steadystate/css/parser/ASCII_CharStream.java?hideattic=0&view=log#rev1.3 
<http://cssparser.cvs.sourceforge.net/viewvc/cssparser/cssparser/src/com/steadystate/css/parser/ASCII_CharStream.java?hideattic=0&view=log#rev1.3>

    Revision *1.3* - (view
    <http://cssparser.cvs.sourceforge.net/viewvc/cssparser/cssparser/src/com/steadystate/css/parser/ASCII_CharStream.java?hideattic=0&revision=1.3&view=markup>)
    (download
    <http://cssparser.cvs.sourceforge.net/viewvc/*checkout*/cssparser/cssparser/src/com/steadystate/css/parser/ASCII_CharStream.java?revision=1.3>)
    (annotate
    <http://cssparser.cvs.sourceforge.net/viewvc/cssparser/cssparser/src/com/steadystate/css/parser/ASCII_CharStream.java?annotate=1.3&hideattic=0>)
    - [select for diffs]
    <http://cssparser.cvs.sourceforge.net/viewvc/cssparser/cssparser/src/com/steadystate/css/parser/ASCII_CharStream.java?hideattic=0&r1=1.3&view=log>

    /Tue Apr 11 08:18:35 2006 UTC/ (2 years, 7 months ago) by /waldbaer/
    Branch: *MAIN*
    <http://cssparser.cvs.sourceforge.net/viewvc/cssparser/cssparser/src/com/steadystate/css/parser/ASCII_CharStream.java?view=log&hideattic=0&pathrev=MAIN>

    CVS Tags: *PRE_MAVEN*
    <http://cssparser.cvs.sourceforge.net/viewvc/cssparser/cssparser/src/com/steadystate/css/parser/ASCII_CharStream.java?view=log&hideattic=0&pathrev=PRE_MAVEN>

    Changes since *1.2: +125 -125 lines*
    Diff to previous 1.2
    <http://cssparser.cvs.sourceforge.net/viewvc/cssparser/cssparser/src/com/steadystate/css/parser/ASCII_CharStream.java?hideattic=0&r1=1.2&r2=1.3>cleanup

http://cssparser.cvs.sourceforge.net/viewvc/cssparser/cssparser/src/com/steadystate/css/parser/ASCII_CharStream.java?hideattic=0&view=diff&r1=1.2&r2=1.3 <http://cssparser.cvs.sourceforge.net/viewvc/cssparser/cssparser/src/com/steadystate/css/parser/ASCII_CharStream.java?hideattic=0&view=diff&r1=1.2&r2=1.3>

162 
<http://cssparser.cvs.sourceforge.net/viewvc/cssparser/cssparser/src/com/steadystate/css/parser/ASCII_CharStream.java?annotate=1.3&hideattic=0#l162> 
	         case '\t' : 	         case '\t' :
163 
<http://cssparser.cvs.sourceforge.net/viewvc/cssparser/cssparser/src/com/steadystate/css/parser/ASCII_CharStream.java?annotate=1.3&hideattic=0#l163> 
	            column--; 	             this.column--;
164 
<http://cssparser.cvs.sourceforge.net/viewvc/cssparser/cssparser/src/com/steadystate/css/parser/ASCII_CharStream.java?annotate=1.3&hideattic=0#l164> 
	            column += (8 - (column & 07)); 	             this.column += 
(8 - (this.column & 07));
165 
<http://cssparser.cvs.sourceforge.net/viewvc/cssparser/cssparser/src/com/steadystate/css/parser/ASCII_CharStream.java?annotate=1.3&hideattic=0#l165> 
	            break; 	            break;