ChangeLogTag:Thu Dec 06 14:06:22 2001 Nanbor Wang <nanbor@cs.wustl.edu>

7 files changed, 123 insertions, 80 deletions

diff --git a/ChangeLog b/ChangeLog
index 93a8d132f4c..45de5344288 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,12 +1,24 @@
+Thu Dec 06 14:06:22 2001  Nanbor Wang  <nanbor@cs.wustl.edu>
+
+	* tests/Config_Test.cpp: The string literals used to initialize
+	  ACE_TString need to be enclosed in an ACE_TEXT macro.
+
+	* ace/Registry_Name_Space.h:
+	* ace/Registry_Name_Space.cpp: Changed ACE_WString to
+	  ACE_NS_WString.  I missed these last time.
+
+	* ace/SString.i: Added an ACE_NS_WString constructor that I
+	  missed.
+
 Thu Dec  6 11:19:18 2001  Venkita Subramonian  <venkita@cs.wustl.edu>
 
-	* performance-tests/Server_Concurrency/Queue_Based_Workers/RT_CORBA_Workers.h: 
+	* performance-tests/Server_Concurrency/Queue_Based_Workers/RT_CORBA_Workers.h:
 	* performance-tests/Server_Concurrency/Leader_Follower/RT_CORBA_Leader_Follower.h:
-	  Fixed warnings in gcc 3.0.2 
+	  Fixed warnings in gcc 3.0.2
 
 Thu Dec  6 11:15:28 2001  Balachandran Natarajan  <bala@cs.wustl.edu>
 
-	* performance-tests/Server_Concurrency/Queue_Based_Workers/RT_CORBA_Workers.h: 
+	* performance-tests/Server_Concurrency/Queue_Based_Workers/RT_CORBA_Workers.h:
 	* performance-tests/Server_Concurrency/Leader_Follower/RT_CORBA_Leader_Follower.h:
 	  Fixed Fuzz errors in these files.
 
diff --git a/ChangeLogs/ChangeLog-02a b/ChangeLogs/ChangeLog-02a
index 93a8d132f4c..45de5344288 100644
--- a/ChangeLogs/ChangeLog-02a
+++ b/ChangeLogs/ChangeLog-02a
@@ -1,12 +1,24 @@
+Thu Dec 06 14:06:22 2001  Nanbor Wang  <nanbor@cs.wustl.edu>
+
+	* tests/Config_Test.cpp: The string literals used to initialize
+	  ACE_TString need to be enclosed in an ACE_TEXT macro.
+
+	* ace/Registry_Name_Space.h:
+	* ace/Registry_Name_Space.cpp: Changed ACE_WString to
+	  ACE_NS_WString.  I missed these last time.
+
+	* ace/SString.i: Added an ACE_NS_WString constructor that I
+	  missed.
+
 Thu Dec  6 11:19:18 2001  Venkita Subramonian  <venkita@cs.wustl.edu>
 
-	* performance-tests/Server_Concurrency/Queue_Based_Workers/RT_CORBA_Workers.h: 
+	* performance-tests/Server_Concurrency/Queue_Based_Workers/RT_CORBA_Workers.h:
 	* performance-tests/Server_Concurrency/Leader_Follower/RT_CORBA_Leader_Follower.h:
-	  Fixed warnings in gcc 3.0.2 
+	  Fixed warnings in gcc 3.0.2
 
 Thu Dec  6 11:15:28 2001  Balachandran Natarajan  <bala@cs.wustl.edu>
 
-	* performance-tests/Server_Concurrency/Queue_Based_Workers/RT_CORBA_Workers.h: 
+	* performance-tests/Server_Concurrency/Queue_Based_Workers/RT_CORBA_Workers.h:
 	* performance-tests/Server_Concurrency/Leader_Follower/RT_CORBA_Leader_Follower.h:
 	  Fixed Fuzz errors in these files.
 
diff --git a/ChangeLogs/ChangeLog-03a b/ChangeLogs/ChangeLog-03a
index 93a8d132f4c..45de5344288 100644
--- a/ChangeLogs/ChangeLog-03a
+++ b/ChangeLogs/ChangeLog-03a
@@ -1,12 +1,24 @@
+Thu Dec 06 14:06:22 2001  Nanbor Wang  <nanbor@cs.wustl.edu>
+
+	* tests/Config_Test.cpp: The string literals used to initialize
+	  ACE_TString need to be enclosed in an ACE_TEXT macro.
+
+	* ace/Registry_Name_Space.h:
+	* ace/Registry_Name_Space.cpp: Changed ACE_WString to
+	  ACE_NS_WString.  I missed these last time.
+
+	* ace/SString.i: Added an ACE_NS_WString constructor that I
+	  missed.
+
 Thu Dec  6 11:19:18 2001  Venkita Subramonian  <venkita@cs.wustl.edu>
 
-	* performance-tests/Server_Concurrency/Queue_Based_Workers/RT_CORBA_Workers.h: 
+	* performance-tests/Server_Concurrency/Queue_Based_Workers/RT_CORBA_Workers.h:
 	* performance-tests/Server_Concurrency/Leader_Follower/RT_CORBA_Leader_Follower.h:
-	  Fixed warnings in gcc 3.0.2 
+	  Fixed warnings in gcc 3.0.2
 
 Thu Dec  6 11:15:28 2001  Balachandran Natarajan  <bala@cs.wustl.edu>
 
-	* performance-tests/Server_Concurrency/Queue_Based_Workers/RT_CORBA_Workers.h: 
+	* performance-tests/Server_Concurrency/Queue_Based_Workers/RT_CORBA_Workers.h:
 	* performance-tests/Server_Concurrency/Leader_Follower/RT_CORBA_Leader_Follower.h:
 	  Fixed Fuzz errors in these files.
 
diff --git a/ace/Registry_Name_Space.cpp b/ace/Registry_Name_Space.cpp
index 56a396bbc73..4293246892e 100644
--- a/ace/Registry_Name_Space.cpp
+++ b/ace/Registry_Name_Space.cpp
@@ -31,8 +31,8 @@ ACE_Registry_Name_Space::open (ACE_Name_Options *name_options)
   ACE_Registry::Naming_Context predefined;
 
   int result = ACE_Predefined_Naming_Contexts::connect (predefined,
-							HKEY_LOCAL_MACHINE,
-							host);
+                                                        HKEY_LOCAL_MACHINE,
+                                                        host);
   if (result != 0)
     ACE_ERROR_RETURN ((LM_ERROR,  ACE_LIB_TEXT ("%p\n"),
                        ACE_LIB_TEXT ("ACE_Predefined_Naming_Context::connect")),
@@ -48,18 +48,18 @@ ACE_Registry_Name_Space::open (ACE_Name_Options *name_options)
 
       // Create new context or bind to existing one
       result = predefined.bind_context (name,
-					this->context_);
+                                        this->context_);
       if (result != 0)
-	ACE_ERROR_RETURN ((LM_ERROR,  ACE_LIB_TEXT ("%p\n"),  ACE_LIB_TEXT ("ACE_Registry::Naming_Context::bind_context")), result);
+        ACE_ERROR_RETURN ((LM_ERROR,  ACE_LIB_TEXT ("%p\n"),  ACE_LIB_TEXT ("ACE_Registry::Naming_Context::bind_context")), result);
     }
   return 0;
 }
 
 
 int
-ACE_Registry_Name_Space::bind (const ACE_WString &name,
-			       const ACE_WString &value,
-			       const char *type)
+ACE_Registry_Name_Space::bind (const ACE_NS_WString &name,
+                               const ACE_NS_WString &value,
+                               const char *type)
 {
   // Pointer to data
   const ACE_USHORT16 *data = value.fast_rep ();
@@ -69,18 +69,18 @@ ACE_Registry_Name_Space::bind (const ACE_WString &name,
 
   // Represent value as an ACE_Registry::Object
   ACE_Registry::Object object ((void *) data,
-			       size,
-			       REG_SZ);
+                               size,
+                               REG_SZ);
   // Add new <key>/<value> pair
   return this->context_.bind (name.fast_rep(),
-			      object);
+                              object);
 }
 
 
 int
-ACE_Registry_Name_Space::rebind (const ACE_WString &name,
-				 const ACE_WString &value,
-				 const char *type)
+ACE_Registry_Name_Space::rebind (const ACE_NS_WString &name,
+                                 const ACE_NS_WString &value,
+                                 const char *type)
 {
   // Pointer to data
   const ACE_USHORT16 *data = value.fast_rep ();
@@ -90,25 +90,25 @@ ACE_Registry_Name_Space::rebind (const ACE_WString &name,
 
   // Represent value as an ACE_Registry::Object
   ACE_Registry::Object object ((void *) data,
-			       size,
-			       REG_SZ);
+                               size,
+                               REG_SZ);
   // Add new <key>/<value> pair
   return this->context_.rebind (name.fast_rep (),
-				object);
+                                object);
 }
 
 
 int
-ACE_Registry_Name_Space::unbind (const ACE_WString &name)
+ACE_Registry_Name_Space::unbind (const ACE_NS_WString &name)
 {
   return this->context_.unbind (name.fast_rep ());
 }
 
 
 int
-ACE_Registry_Name_Space::resolve (const ACE_WString &name,
-				  ACE_WString &value,
-				  char *&type)
+ACE_Registry_Name_Space::resolve (const ACE_NS_WString &name,
+                                  ACE_NS_WString &value,
+                                  char *&type)
 {
   // This object will be used to query the size of the data.
   // Note: The query_object.data will be null for this invocation.
@@ -123,8 +123,8 @@ ACE_Registry_Name_Space::resolve (const ACE_WString &name,
 
   // Represent new space as an ACE_Registry::Object
   ACE_Registry::Object object ((void *) value.fast_rep (),
-			       query_object.size (),
-			       REG_SZ);
+                               query_object.size (),
+                               REG_SZ);
 
   result = this->context_.resolve (name.fast_rep (), object);
   if (object.size () != query_object.size ())
@@ -138,11 +138,11 @@ ACE_Registry_Name_Space::resolve (const ACE_WString &name,
 
 int
 ACE_Registry_Name_Space:: list_names (ACE_WSTRING_SET &set,
-				      const ACE_WString &pattern)
+                                      const ACE_NS_WString &pattern)
 {
   ACE_BINDING_SET binding_set;
   int result = this->list_name_entries (binding_set,
-					pattern);
+                                        pattern);
   if (result != 0)
     return result;
 
@@ -160,11 +160,11 @@ ACE_Registry_Name_Space:: list_names (ACE_WSTRING_SET &set,
 
 int
 ACE_Registry_Name_Space::list_values (ACE_WSTRING_SET &set,
-				      const ACE_WString &pattern)
+                                      const ACE_NS_WString &pattern)
 {
   ACE_BINDING_SET binding_set;
   int result = this->list_name_entries (binding_set,
-					pattern);
+                                        pattern);
   if (result != 0)
     return result;
 
@@ -182,7 +182,7 @@ ACE_Registry_Name_Space::list_values (ACE_WSTRING_SET &set,
 
 int
 ACE_Registry_Name_Space::list_types (ACE_WSTRING_SET &set,
-				     const ACE_WString &pattern)
+                                     const ACE_NS_WString &pattern)
 {
   return 0;
 }
@@ -190,7 +190,7 @@ ACE_Registry_Name_Space::list_types (ACE_WSTRING_SET &set,
 
 int
 ACE_Registry_Name_Space::list_name_entries (ACE_BINDING_SET &set,
-					    const ACE_WString &pattern)
+                                            const ACE_NS_WString &pattern)
 {
   ACE_Registry::Binding_List list;
   int result = this->context_.list (list);
@@ -206,24 +206,24 @@ ACE_Registry_Name_Space::list_name_entries (ACE_BINDING_SET &set,
       ACE_Registry::Binding &binding = *i;
 
       if (binding.type () == ACE_Registry::OBJECT)
-	{
-	  // Key
-	  ACE_TString string = binding.name ();
-	  ACE_WString key (string.c_str ());
-
-	  // Value
-	  ACE_WString value;
-	  char *type = 0;
-	  result = this->resolve (key,
-				  value,
-				  type);
-	  if (result != 0)
-	    ACE_ERROR_RETURN ((LM_ERROR,  ACE_LIB_TEXT ("%p\n"),  ACE_LIB_TEXT ("ACE_Registry::Naming_Context::resolve")), result);
-
-	  // Complete binding
-	  ACE_Name_Binding binding (key, value, type);
-	  set.insert (binding);
-	}
+        {
+          // Key
+          ACE_TString string = binding.name ();
+          ACE_NS_WString key (string.c_str ());
+
+          // Value
+          ACE_NS_WString value;
+          char *type = 0;
+          result = this->resolve (key,
+                                  value,
+                                  type);
+          if (result != 0)
+            ACE_ERROR_RETURN ((LM_ERROR,  ACE_LIB_TEXT ("%p\n"),  ACE_LIB_TEXT ("ACE_Registry::Naming_Context::resolve")), result);
+
+          // Complete binding
+          ACE_Name_Binding binding (key, value, type);
+          set.insert (binding);
+        }
     }
   return 0;
 }
@@ -231,7 +231,7 @@ ACE_Registry_Name_Space::list_name_entries (ACE_BINDING_SET &set,
 
 int
 ACE_Registry_Name_Space::list_value_entries (ACE_BINDING_SET &set,
-					     const ACE_WString &pattern)
+                                             const ACE_NS_WString &pattern)
 {
   return this->list_name_entries (set, pattern);
 }
@@ -239,7 +239,7 @@ ACE_Registry_Name_Space::list_value_entries (ACE_BINDING_SET &set,
 
 int
 ACE_Registry_Name_Space::list_type_entries (ACE_BINDING_SET &set,
-					    const ACE_WString &pattern)
+                                            const ACE_NS_WString &pattern)
 {
   return this->list_name_entries (set, pattern);
 }
diff --git a/ace/Registry_Name_Space.h b/ace/Registry_Name_Space.h
index 439cd14b488..075c2643f56 100644
--- a/ace/Registry_Name_Space.h
+++ b/ace/Registry_Name_Space.h
@@ -58,8 +58,8 @@ public:
   int open (ACE_Name_Options *name_options);
 
   /// Bind a new name to a naming context (Wide character strings).
-  int bind (const ACE_WString &name_in,
-            const ACE_WString &value_in,
+  int bind (const ACE_NS_WString &name_in,
+            const ACE_NS_WString &value_in,
             const char *type_in = "");
 
   /**
@@ -67,34 +67,34 @@ public:
    * ACE_Name_Space or bind a new name to the context, if it didn't
    * exist yet. (Wide charcter strings interface).
    */
-  int rebind (const ACE_WString &name_in,
-              const ACE_WString &value_in,
+  int rebind (const ACE_NS_WString &name_in,
+              const ACE_NS_WString &value_in,
               const char *type_in = "");
 
   /// Delete a name from a ACE_Name_Space (Wide charcter strings
   /// Interface).
-  int unbind (const ACE_WString &name_in);
+  int unbind (const ACE_NS_WString &name_in);
 
   /// Get value and type of a given name binding (Wide chars).  The
   /// caller is responsible for deleting both <value_out> and <type_out>!
-  int resolve (const ACE_WString &name_in,
-               ACE_WString &value_out,
+  int resolve (const ACE_NS_WString &name_in,
+               ACE_NS_WString &value_out,
                char *&type_out);
 
   /// Get a set of names matching a specified pattern (wchars). Matching
   /// means the names must begin with the pattern string.
   int list_names (ACE_WSTRING_SET &set_out,
-                  const ACE_WString &pattern_in);
+                  const ACE_NS_WString &pattern_in);
 
   /// Get a set of values matching a specified pattern (wchars). Matching
   /// means the values must begin with the pattern string.
   int list_values (ACE_WSTRING_SET &set_out,
-                   const ACE_WString &pattern_in);
+                   const ACE_NS_WString &pattern_in);
 
   /// Get a set of types matching a specified pattern (wchars). Matching
   /// means the types must begin with the pattern string.
   int list_types (ACE_WSTRING_SET &set_out,
-                  const ACE_WString &pattern_in);
+                  const ACE_NS_WString &pattern_in);
 
   /**
    * Get a set of names matching a specified pattern (wchars). Matching
@@ -102,7 +102,7 @@ public:
    * complete binding associated each pattern match.
    */
   int list_name_entries (ACE_BINDING_SET &set,
-                         const ACE_WString &pattern);
+                         const ACE_NS_WString &pattern);
 
   /**
    * Get a set of values matching a specified pattern (wchars). Matching
@@ -110,7 +110,7 @@ public:
    * complete binding associated each pattern match.
    */
   int list_value_entries (ACE_BINDING_SET &set,
-                          const ACE_WString &pattern);
+                          const ACE_NS_WString &pattern);
 
   /**
    * Get a set of types matching a specified pattern (wchars). Matching
@@ -118,7 +118,7 @@ public:
    * complete binding associated each pattern match.
    */
   int list_type_entries (ACE_BINDING_SET &set,
-                         const ACE_WString &pattern);
+                         const ACE_NS_WString &pattern);
 
   /// Dump the state of the object
   void dump (void) const;
diff --git a/ace/SString.i b/ace/SString.i
index ee6201fc74e..247812da5a4 100644
--- a/ace/SString.i
+++ b/ace/SString.i
@@ -18,6 +18,13 @@ ACE_NS_WString::ACE_NS_WString (const ACE_WSTRING_TYPE *s,
 }
 
 ACE_INLINE
+ACE_NS_WString::ACE_NS_WString (const ACE_WSTRING_TYPE *s,
+                                ACE_Allocator *alloc)
+  : ACE_WString (s, alloc)
+{
+}
+
+ACE_INLINE
 ACE_NS_WString::ACE_NS_WString (size_t len, ACE_Allocator *alloc)
   : ACE_WString (len, alloc)
 {
diff --git a/tests/Config_Test.cpp b/tests/Config_Test.cpp
index dae7afd2cd4..d3e798fd65e 100644
--- a/tests/Config_Test.cpp
+++ b/tests/Config_Test.cpp
@@ -96,7 +96,7 @@ test (ACE_Configuration *config)
                                 ACE_TEXT ("stvalue"),
                                 stvalue))
     return -6;
-  else if (stvalue != "second stvaluetest")
+  else if (stvalue != ACE_TEXT ("second stvaluetest"))
     return -7;
 
   u_int intvalue;
@@ -135,7 +135,7 @@ test (ACE_Configuration *config)
                                     name,
                                     type))
     {
-      if (name == "stvalue")
+      if (name == ACE_TEXT ("stvalue"))
         {
           if (type != ACE_Configuration::STRING)
             return -12;
@@ -143,7 +143,7 @@ test (ACE_Configuration *config)
             return -12;
           found[0] = 1;
         }
-      else if (name == "intvalue")
+      else if (name == ACE_TEXT ("intvalue"))
         {
           if (type != ACE_Configuration::INTEGER)
             return -13;
@@ -151,7 +151,7 @@ test (ACE_Configuration *config)
             return -13;
           found[1] = 1;
         }
-      else if (name == "binvalue")
+      else if (name == ACE_TEXT ("binvalue"))
         {
           if (type != ACE_Configuration::BINARY)
             return -14;
@@ -352,11 +352,11 @@ build_config_object (ACE_Configuration& cfg)
     return -2;
   else if (cfg.set_string_value (NetworkSection,
                                  ACE_TEXT ("Delay"),
-                                 ACE_TString ("FALSE")))
+                                 ACE_TString (ACE_TEXT ("FALSE"))))
     return -3;
   else if (cfg.set_string_value (NetworkSection,
                                  ACE_TEXT ("DestIPAddress"),
-                                 ACE_TString ("localhost")))
+                                 ACE_TString (ACE_TEXT ("localhost"))))
     return -4;
   else if (cfg.set_integer_value (NetworkSection,
                                   ACE_TEXT ("DestPort"),
@@ -376,7 +376,7 @@ build_config_object (ACE_Configuration& cfg)
 
   if (cfg.set_string_value (LoggerSection,
                             ACE_TEXT ("Heading"),
-                            ACE_TString ("ACE - Adaptive Communication Environment")))
+                            ACE_TString (ACE_TEXT ("ACE - Adaptive Communication Environment"))))
     return -8;
   else if (cfg.set_integer_value (LoggerSection,
                                   ACE_TEXT ("SeekIndex"),
@@ -388,15 +388,15 @@ build_config_object (ACE_Configuration& cfg)
     return -10;
   else if (cfg.set_string_value (LoggerSection,
                                  ACE_TEXT ("Justification"),
-                                 ACE_TString ("left_justified")))
+                                 ACE_TString (ACE_TEXT ("left_justified"))))
     return -11;
   else if (cfg.set_string_value (LoggerSection,
                                  ACE_TEXT ("LogFilePath"),
-                                 ACE_TString ("log/")))
+                                 ACE_TString (ACE_TEXT ("log/"))))
     return -12;
   else if (cfg.set_string_value (LoggerSection,
                                  ACE_TEXT ("TransactionFilePath"),
-                                 ACE_TString ("data/")))
+                                 ACE_TString (ACE_TEXT ("data/"))))
     return -13;
 
   if (cfg.open_section (root,