[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: kdelibs/khtml
From:       Frans Englich <frans.englich () telia ! com>
Date:       2005-03-21 20:26:21
Message-ID: 200503212026.21107.frans.englich () telia ! com
[Download RAW message or body]


FYI, I think you also fixed #49263, I can't reproduce it with your changes.

http://bugs.kde.org/show_bug.cgi?id=49263


Cheers,

		Frans


On Monday 21 March 2005 19:10, Allan Sandfeld Jensen wrote:
> CVS commit by carewolf:
>
> Correct parsing of white-space immediately after pre tag.
> BUG: 81393
>
>
>   M +4 -0      ChangeLog   1.406
>   M +6 -8      html/htmltokenizer.cpp   1.299
>
>
> --- kdelibs/khtml/ChangeLog  #1.405:1.406
> @@ -1,2 +1,6 @@
> +2005-03-21  Allan Sandfeld Jensen <kde@carewolf.com>
> +
> +        * html/htmltokenizer.cpp: Only skip first LF if immediately after
> PRE tag +
>  2005-03-18  Allan Sandfeld Jensen <kde@carewolf.com>
>
>
> --- kdelibs/khtml/html/htmltokenizer.cpp  #1.298:1.299
> @@ -1136,6 +1136,6 @@ void HTMLTokenizer::parseTag(TokenizerSt
>              switch( tagID ) {
>              case ID_PRE:
> -                discard = LFDiscard;
>                  prePos = 0;
> +                discard = AllDiscard;
>                  break;
>              case ID_SCRIPT:
> @@ -1365,10 +1365,8 @@ void HTMLTokenizer::write( const Tokeniz
>              if ( pending ) {
>                  // pre context always gets its spaces/linefeeds
> -                if ( pre )
> -                    addPending();
>                  // only add in existing inline context or if
>                  // we just started one, i.e. we're about to insert real
> text -                else if ( !parser->selectMode() &&
> -                          ( !parser->noSpaces() || dest > buffer )) {
> +                if ( pre || script || (!parser->selectMode() &&
> +                          ( !parser->noSpaces() || dest > buffer ))) {
>                      addPending();
>                      discard = AllDiscard;
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic