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;
|
