Browse Source

[1603] cleanup: removed unnecessary declaration of maptolower from name.h.

the one declared in name_internal.h is now also used by MessageRenderer, too,
so updated the comment accordingly.
JINMEI Tatuya 13 years ago
parent
commit
1389eb8fa9
2 changed files with 5 additions and 13 deletions
  1. 0 9
      src/lib/dns/name.h
  2. 5 4
      src/lib/dns/name_internal.h

+ 0 - 9
src/lib/dns/name.h

@@ -166,15 +166,6 @@ private:
     NameRelation relation_;
 };
 
-// This is effectively a "private" namespace, but exposed publicly so the
-// definitions in it can be shared with the LabelSequence class.  It's not
-// expected to be used
-namespace name {
-namespace internal {
-extern const unsigned char maptolower[];
-}
-}
-
 ///
 /// The \c Name class encapsulates DNS names.
 ///

+ 5 - 4
src/lib/dns/name_internal.h

@@ -16,10 +16,11 @@
 #define __NAME_INTERNAL_H 1
 
 // This is effectively a "private" namespace for the Name class implementation,
-// but exposed publicly so the definitions in it can be shared with the
-// LabelSequence class.  It's not expected to be used even by other modules
-// of this library, much less by normal applications.  This header file is
-// therefore not expected to be installed as part of the library.
+// but exposed publicly so the definitions in it can be shared with other
+// modules of the library (as of its introduction, used by LabelSequence and
+// MessageRenderer).  It's not expected to be used even by normal applications.
+// This header file is therefore not expected to be installed as part of the
+// library.
 namespace isc {
 namespace dns {
 namespace name {