Browse Source

[2387] trivial (unrelated) cleanups: constify and indentation fix.

JINMEI Tatuya 12 years ago
parent
commit
d124a5b6f1

+ 1 - 2
src/lib/dns/rdata/generic/detail/nsec3param_common.cc

@@ -60,8 +60,7 @@ parseNSEC3ParamFromLexer(const char* const rrtype_name,
         lexer.getNextToken(MasterToken::NUMBER).getNumber();
         lexer.getNextToken(MasterToken::NUMBER).getNumber();
     if (iterations > 0xffff) {
     if (iterations > 0xffff) {
         isc_throw(InvalidRdataText, rrtype_name <<
         isc_throw(InvalidRdataText, rrtype_name <<
-                  " iterations out of range: " <<
-            iterations);
+                  " iterations out of range: " << iterations);
     }
     }
 
 
     const string salthex =
     const string salthex =

+ 10 - 10
src/lib/dns/rdata/generic/dnskey_48.cc

@@ -93,9 +93,9 @@ DNSKEY::DNSKEY(InputBuffer& buffer, size_t rdata_len) {
         isc_throw(InvalidRdataLength, "DNSKEY too short: " << rdata_len);
         isc_throw(InvalidRdataLength, "DNSKEY too short: " << rdata_len);
     }
     }
 
 
-    uint16_t flags = buffer.readUint16();
-    uint16_t protocol = buffer.readUint8();
-    uint16_t algorithm = buffer.readUint8();
+    const uint16_t flags = buffer.readUint16();
+    const uint16_t protocol = buffer.readUint8();
+    const uint16_t algorithm = buffer.readUint8();
 
 
     rdata_len -= 4;
     rdata_len -= 4;
     vector<uint8_t> keydata(rdata_len);
     vector<uint8_t> keydata(rdata_len);
@@ -234,11 +234,11 @@ DNSKEY::compare(const Rdata& other) const {
         return (impl_->algorithm_ < other_dnskey.impl_->algorithm_ ? -1 : 1);
         return (impl_->algorithm_ < other_dnskey.impl_->algorithm_ ? -1 : 1);
     }
     }
 
 
-    size_t this_len = impl_->keydata_.size();
-    size_t other_len = other_dnskey.impl_->keydata_.size();
-    size_t cmplen = min(this_len, other_len);
-    int cmp = memcmp(&impl_->keydata_[0], &other_dnskey.impl_->keydata_[0],
-                     cmplen);
+    const size_t this_len = impl_->keydata_.size();
+    const size_t other_len = other_dnskey.impl_->keydata_.size();
+    const size_t cmplen = min(this_len, other_len);
+    const int cmp = memcmp(&impl_->keydata_[0],
+                           &other_dnskey.impl_->keydata_[0], cmplen);
     if (cmp != 0) {
     if (cmp != 0) {
         return (cmp);
         return (cmp);
     } else {
     } else {
@@ -249,7 +249,7 @@ DNSKEY::compare(const Rdata& other) const {
 uint16_t
 uint16_t
 DNSKEY::getTag() const {
 DNSKEY::getTag() const {
     if (impl_->algorithm_ == 1) {
     if (impl_->algorithm_ == 1) {
-        int len = impl_->keydata_.size();
+        const int len = impl_->keydata_.size();
         return ((impl_->keydata_[len - 3] << 8) + impl_->keydata_[len - 2]);
         return ((impl_->keydata_[len - 3] << 8) + impl_->keydata_[len - 2]);
     }
     }
 
 
@@ -257,7 +257,7 @@ DNSKEY::getTag() const {
     ac += (impl_->protocol_ << 8);
     ac += (impl_->protocol_ << 8);
     ac += impl_->algorithm_;
     ac += impl_->algorithm_;
 
 
-    size_t size = impl_->keydata_.size();
+    const size_t size = impl_->keydata_.size();
     for (size_t i = 0; i < size; i ++) {
     for (size_t i = 0; i < size; i ++) {
         ac += (i & 1) ? impl_->keydata_[i] : (impl_->keydata_[i] << 8);
         ac += (i & 1) ? impl_->keydata_[i] : (impl_->keydata_[i] << 8);
     }
     }