From 64de66f3dd7973eff9962046a0bbf93f6204d69f Mon Sep 17 00:00:00 2001
From: "sspitzer%netscape.com" <sspitzer%netscape.com>
Date: Wed, 14 Jul 1999 15:39:35 +0000
Subject: [PATCH] don't get the prefs on every call.  do it once, for
 performance.

---
 .../addrbook/resources/content/editcard.xul   | 27 +++++++++++--------
 1 file changed, 16 insertions(+), 11 deletions(-)

diff --git a/mailnews/addrbook/resources/content/editcard.xul b/mailnews/addrbook/resources/content/editcard.xul
index d21d8eafbc7..fc23a970e5a 100644
--- a/mailnews/addrbook/resources/content/editcard.xul
+++ b/mailnews/addrbook/resources/content/editcard.xul
@@ -58,6 +58,10 @@
 
 <html:script language="JavaScript">
 
+var pref;
+var prefInitialized = false;
+var lastFirst = false;
+
 function GenerateDisplayName()
 {
         /* see AB_GenerateDisplayName in http://lxr.mcom.com/nova/source/ns/lib/libaddr/abglue.cpp for the proper logic */ 
@@ -70,18 +74,19 @@ function GenerateDisplayName()
 	/* this used to be XP_GetString(MK_ADDR_FIRST_LAST_SEP) */
 	var firstLastSep = " ";
 
-	/* todo: move pref out to be a global var for performance? */
-	var pref = Components.classes['component://netscape/preferences'];
-	var lastFirst = false;
+	if (!prefInitialized) {
+		pref = Components.classes['component://netscape/preferences'];
 
-	if (pref) {
-        	pref = pref.getService();
-        } 
-	if (pref) {
-		pref = pref.QueryInterface(Components.interfaces.nsIPref);
-	}
-	if (pref) {
-		lastFirst = pref.GetBoolPref("mail.addr_book.lastnamefirst");
+		if (pref) {
+			pref = pref.getService();
+		} 
+		if (pref) {
+			pref = pref.QueryInterface(Components.interfaces.nsIPref);
+		}
+		if (pref) {
+			lastFirst = pref.GetBoolPref("mail.addr_book.lastnamefirst");
+		}
+		prefInitialized = true;
 	}
 
 	/* todo:  mscott says there was a pref in 4.5 that would */