* fixed build for PathContainer.cpp (stupid last minute change...)

* fixed crashing bug in Shape, it doesn't "own" the Style...
* fixed problem in Shape in for non Icon-O-Matic build, fNeedsUpdate
  still needs to be maintained.
  
Having a read-only version has definitely improved efficiency! The
icon import time must have dropped 50%. The overall penalty for
reading vector icons in Tracker seems to be 1.5 to 2 ms now, including
parsing and rendering at 32x32. 



git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@18398 a95241bf-73f2-0310-859d-f6bbb57e9c96

src/libs/icon/shape/PathContainer.cpp

@@ -24,9 +24,9 @@ PathContainerListener::~PathContainerListener() {}
 // constructor
 PathContainer::PathContainer(bool ownsPaths)
 	: fPaths(16),
-	  fOwnsPaths(ownsPaths),
+	  fOwnsPaths(ownsPaths)
 #ifdef ICON_O_MATIC
-	  fListeners(2)
+	, fListeners(2)
 #endif
 {
 }

src/libs/icon/shape/Shape.cpp

@@ -416,8 +416,6 @@ Shape::SetStyle(::Style* style)
 		fStyle->Release();
 	}
 	::Style* oldStyle = fStyle;
-#else
-	delete fStyle;
 #endif
 
 	fStyle = style;
@@ -505,6 +503,8 @@ Shape::AddTransformer(Transformer* transformer, int32 index)
 	transformer->AddObserver(this);
 
 	_NotifyTransformerAdded(transformer, index);
+#else
+	fNeedsUpdate = true;
 #endif
 	return true;
 }
@@ -518,6 +518,8 @@ Shape::RemoveTransformer(Transformer* transformer)
 		transformer->RemoveObserver(this);
 		
 		_NotifyTransformerRemoved(transformer);
+#else
+		fNeedsUpdate = true;
 #endif
 		return true;
 	}