From ade73e8c851b39f043d5211a44d9f3c6d849a242 Mon Sep 17 00:00:00 2001
From: Ryan Leavengood <leavengood@gmail.com>
Date: Mon, 10 Nov 2008 17:07:47 +0000
Subject: [PATCH] Make the BObjectList member for font families a non-pointer.

git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@28596 a95241bf-73f2-0310-859d-f6bbb57e9c96
---
 src/add-ons/screen_savers/message/Message.cpp | 17 ++++++-----------
 1 file changed, 6 insertions(+), 11 deletions(-)

diff --git a/src/add-ons/screen_savers/message/Message.cpp b/src/add-ons/screen_savers/message/Message.cpp
index a16e2b66e6..4f28e02ee8 100644
--- a/src/add-ons/screen_savers/message/Message.cpp
+++ b/src/add-ons/screen_savers/message/Message.cpp
@@ -97,7 +97,7 @@ class Message : public BScreenSaver
 		status_t	StartSaver(BView *view, bool preview);
 		
 	private:
-		BObjectList<font_family>	*fFontFamilies;
+		BObjectList<font_family>	fFontFamilies;
 		float						fScaleFactor;
 		bool						fPreview;
 };
@@ -111,19 +111,15 @@ BScreenSaver *instantiate_screen_saver(BMessage *msg, image_id image)
 
 Message::Message(BMessage *archive, image_id id)
  :	BScreenSaver(archive, id)
- ,  fFontFamilies(NULL)
 {
 }
 
 
 Message::~Message()
 {
-	if (fFontFamilies) {
-		for (int32 i = 0; i < fFontFamilies->CountItems(); i++) {
-			if (fFontFamilies->ItemAt(i))
-				delete fFontFamilies->ItemAt(i);
-		}
-		delete fFontFamilies;
+	for (int32 i = 0; i < fFontFamilies.CountItems(); i++) {
+		if (fFontFamilies.ItemAt(i))
+			delete fFontFamilies.ItemAt(i);
 	}
 }
 
@@ -146,14 +142,13 @@ Message::StartSaver(BView *view, bool preview)
 
 	// Get font families
 	int numFamilies = count_font_families();
-	fFontFamilies = new BObjectList<font_family>();
 	for (int32 i = 0; i < numFamilies; i++) {
 		font_family *family = new font_family[1];
 		uint32 flags;
 		if (get_font_family(i, family, &flags) == B_OK) {
 			// Do not add fixed fonts
 			if (!(flags & B_IS_FIXED))
-				fFontFamilies->AddItem(family);
+				fFontFamilies.AddItem(family);
 		}
 	}
 
@@ -199,7 +194,7 @@ Message::Draw(BView *view, int32 frame)
 	BFont font;
 	offscreen.GetFont(&font);
 	font.SetFace(B_BOLD_FACE);
-	font.SetFamilyAndStyle(*(fFontFamilies->ItemAt(rand() % fFontFamilies->CountItems())), NULL);
+	font.SetFamilyAndStyle(*(fFontFamilies.ItemAt(rand() % fFontFamilies.CountItems())), NULL);
 	offscreen.SetFont(&font);
 
 	// Get the message