From kde-commits  Thu Aug 24 15:22:10 2006
From: George Staikos <staikos () kde ! org>
Date: Thu, 24 Aug 2006 15:22:10 +0000
To: kde-commits
Subject: Re: branches/KDE/3.5/kdelibs/khtml/html
Message-Id: <200608241122.10610.staikos () kde ! org>
X-MARC-Message: https://marc.info/?l=kde-commits&m=115643296723776

On Thursday 24 August 2006 11:32, Frans Englich wrote:
> > --- 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?)

  The jump table isn't supposed to cross initialization boundaries.

-- 
George Staikos
KDE Developer				http://www.kde.org/
Staikos Computing Services Inc.		http://www.staikos.net/