Browse Source

[1396] Rename method to getLength()

At first I thought getLength() may already be in use by RR type-specific
classes, but it's not in use. getLength() is more consistent with naming
of other classes' wire data length methods.
Mukund Sivaraman 11 years ago
parent
commit
7f9bc84501
3 changed files with 7 additions and 7 deletions
  1. 1 1
      src/lib/dns/rdata.cc
  2. 3 3
      src/lib/dns/rdata.h
  3. 3 3
      src/lib/dns/tests/rdata_unittest.cc

+ 1 - 1
src/lib/dns/rdata.cc

@@ -47,7 +47,7 @@ namespace dns {
 namespace rdata {
 
 uint16_t
-Rdata::getWireLength() const {
+Rdata::getLength() const {
     OutputBuffer obuffer(0);
 
     toWire(obuffer);

+ 3 - 3
src/lib/dns/rdata.h

@@ -227,15 +227,15 @@ public:
     /// IMPLEMENTATION NOTE: Currently this base class implementation is
     /// non-optimal as it renders the wire data to a buffer and returns
     /// the buffer's length. What would perform better is to add
-    /// implementations of \c getWireLength() method to every RDATA
+    /// implementations of \c getLength() method to every RDATA
     /// type. This is why this method is virtual. Once all Rdata types
-    /// have \c getWireLength() implementations, this base class
+    /// have \c getLength() implementations, this base class
     /// implementation must be removed and the method should become a
     /// pure interface.
     ///
     /// \return The length of the wire format representation of the
     /// RDATA.
-    virtual uint16_t getWireLength() const;
+    virtual uint16_t getLength() const;
 };
 
 namespace generic {

+ 3 - 3
src/lib/dns/tests/rdata_unittest.cc

@@ -211,12 +211,12 @@ TEST_F(RdataTest, createRdataWithLexer) {
                    "file does not end with newline");
 }
 
-TEST_F(RdataTest, getWireLength) {
+TEST_F(RdataTest, getLength) {
     const in::AAAA aaaa_rdata("2001:db8::1");
-    EXPECT_EQ(16, aaaa_rdata.getWireLength());
+    EXPECT_EQ(16, aaaa_rdata.getLength());
 
     const generic::TXT txt_rdata("Hello World");
-    EXPECT_EQ(12, txt_rdata.getWireLength());
+    EXPECT_EQ(12, txt_rdata.getLength());
 }
 
 }