[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-commits
Subject: Re: branches/KDE/3.5/kdelibs/khtml/html
From: Frans Englich <frans.englich () telia ! com>
Date: 2006-08-24 15:32:03
Message-ID: 200608241532.03958.frans.englich () telia ! com
[Download RAW message or body]
On Thursday 24 August 2006 15:10, Dirk Mueller wrote:
> SVN commit 576668 by mueller:
>
> compile++
>
>
> M +2 -0 html_inlineimpl.cpp
>
>
> --- branches/KDE/3.5/kdelibs/khtml/html/html_inlineimpl.cpp #576667:576668
> @@ -175,11 +175,13 @@
> switch(attr->id())
> {
> case ATTR_HREF:
> + {
> bool hadAnchor = m_hasAnchor;
> m_hasAnchor = attr->val() != 0;
> getDocument()->incDOMTreeVersion();
> if (hadAnchor != m_hasAnchor)
> setChanged();
> + }
> break;
> case ATTR_TARGET:
> m_hasTarget = attr->val() != 0;
Hm, how did this fix a breakage? Was the scope needed to avoid name clashing?
(Any particular reason why the break statement is separated from the other
code in the case branch?)
Cheers,
Frans
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic