emelie/tidy-html5
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Issue #133 - ever increasing indent!

Browse source 
This is a simple but profound change in pprint.c.
Since leading space is preserved on script code, after tidy indents
the code once, a second run on that tidied file would add more indent
to already indented code. This fix should be carefully checked,
and removed if there are other bad consequences.
Bump the version point to 4 for this change.
This commit is contained in:
Geoff McLane 2015-02-01 14:27:02 +01:00
parent e6930feb02
commit 0f80c08355
1 changed files with 12 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
src/pprint.c
View file

@ -652,19 +652,27 @@ static void PFlushLineImpl( TidyDocImpl* doc )
{
TidyPrintImpl* pprint = &doc->pprint;
uint i;
uint i, j;
CheckWrapLine( doc );
j = 0; /* Issue #133 - start text output */
if ( WantIndent(doc) )
{
uint spaces = GetSpaces( pprint );
for ( i = 0; i < spaces; ++i )
for ( i = 0; i < spaces; ++i ) {
TY_(WriteChar)( ' ', doc->docOut );
if (( j < pprint->linelen ) && ( pprint->linebuf[j] == ' ' )) {
/*\ Issue #133 - ever increasing indent on each tidy run
* Now removed any leading spaces by the amount of the indent
\*/
j++;
}
}
}
for ( i = 0; i < pprint->linelen; ++i )
TY_(WriteChar)( pprint->linebuf[i], doc->docOut );
for ( ; j < pprint->linelen; j++ )
TY_(WriteChar)( pprint->linebuf[j], doc->docOut );
if ( IsInString(pprint) )
TY_(WriteChar)( '\\', doc->docOut );