Commit 50560aa1 authored by darin's avatar darin

Reviewed by Dave.

	- fixed a bunch of places where KHTML did !x == y instead of !(x == y) or x != y.

        * khtml/css/cssstyleselector.cpp: (CSSStyleSelector::applyRule):
        * khtml/html/htmlparser.cpp: (KHTMLParser::insertNode):
        * khtml/misc/loader.cpp: (Cache::requestImage), (Cache::requestStyleSheet):
        * khtml/rendering/render_flow.cpp: (RenderFlow::setStyle):
	Changed !x == y to x != y. In each case, there's a reason that this does not create
	any problem in practice, which is why we didn't rush to get this in for beta 1.


git-svn-id: http://svn.webkit.org/repository/webkit/trunk@3304 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 895eae19
2003-01-12 Darin Adler <darin@apple.com>
Reviewed by Dave.
- fixed a bunch of places where KHTML did !x == y instead of !(x == y) or x != y.
* khtml/css/cssstyleselector.cpp: (CSSStyleSelector::applyRule):
* khtml/html/htmlparser.cpp: (KHTMLParser::insertNode):
* khtml/misc/loader.cpp: (Cache::requestImage), (Cache::requestStyleSheet):
* khtml/rendering/render_flow.cpp: (RenderFlow::setStyle):
Changed !x == y to x != y. In each case, there's a reason that this does not create
any problem in practice, which is why we didn't rush to get this in for beta 1.
2003-01-11 Darin Adler <darin@apple.com>
Reviewed by John.
......
2003-01-12 Darin Adler <darin@apple.com>
Reviewed by Dave.
- fixed a bunch of places where KHTML did !x == y instead of !(x == y) or x != y.
* khtml/css/cssstyleselector.cpp: (CSSStyleSelector::applyRule):
* khtml/html/htmlparser.cpp: (KHTMLParser::insertNode):
* khtml/misc/loader.cpp: (Cache::requestImage), (Cache::requestStyleSheet):
* khtml/rendering/render_flow.cpp: (RenderFlow::setStyle):
Changed !x == y to x != y. In each case, there's a reason that this does not create
any problem in practice, which is why we didn't rush to get this in for beta 1.
2003-01-11 Darin Adler <darin@apple.com>
Reviewed by John.
......
......@@ -2212,7 +2212,7 @@ void CSSStyleSelector::applyRule( DOM::CSSProperty *prop )
case CSS_PROP_HEIGHT:
case CSS_PROP_MIN_HEIGHT:
// +inherit +auto !can be calculted directly!
if(!prop->m_id == CSS_PROP_MAX_HEIGHT && primitiveValue &&
if(prop->m_id != CSS_PROP_MAX_HEIGHT && primitiveValue &&
primitiveValue->getIdent() == CSS_VAL_AUTO)
apply = true;
if(value->cssValueType() == CSSValue::CSS_INHERIT)
......
......@@ -389,7 +389,7 @@ bool KHTMLParser::insertNode(NodeImpl *n, bool flat)
break;
case ID_HEAD:
// ### alllow not having <HTML> in at all, as per HTML spec
if (!current->isDocumentNode() && !current->id() == ID_HTML )
if (!current->isDocumentNode() && current->id() != ID_HTML )
return false;
break;
// We can deal with a base, meta and link element in the body, by just adding the element to head.
......
......@@ -1499,7 +1499,7 @@ CachedImage *Cache::requestImage( DocLoader* dl, const DOMString & url, bool rel
o->setExpireDate(_expireDate, true);
if(!o->type() == CachedObject::Image)
if(o->type() != CachedObject::Image)
{
#ifdef CACHE_DEBUG
kdDebug( 6060 ) << "Cache::Internal Error in requestImage url=" << kurl.url() << "!" << endl;
......@@ -1573,7 +1573,7 @@ CachedCSSStyleSheet *Cache::requestStyleSheet( DocLoader* dl, const DOMString &
o->setExpireDate(_expireDate, true);
if(!o->type() == CachedObject::CSSStyleSheet)
if(o->type() != CachedObject::CSSStyleSheet)
{
#ifdef CACHE_DEBUG
kdDebug( 6060 ) << "Cache::Internal Error in requestStyleSheet url=" << kurl.url() << "!" << endl;
......
......@@ -76,7 +76,7 @@ void RenderFlow::setStyle(RenderStyle *_style)
if (!isTableCell() && (isPositioned() || isRelPositioned() || style()->overflow()==OHIDDEN) && !m_layer)
m_layer = new (renderArena()) RenderLayer(this);
if(isFloating() || !style()->display() == INLINE)
if(isFloating() || style()->display() != INLINE)
setInline(false);
if (isInline() && !m_childrenInline)
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment