Browse Source

[2796] compare with Opcode object instead of a numeric code

Yoshitaka Aharen 12 years ago
parent
commit
bb08c0bcb4
1 changed files with 2 additions and 4 deletions
  1. 2 4
      src/bin/auth/statistics.cc.pre

+ 2 - 4
src/bin/auth/statistics.cc.pre

@@ -138,11 +138,9 @@ Counters::incRequest(const MessageAttributes& msgattrs) {
     // if a short message which does not contain DNS header is received, or
     // if a short message which does not contain DNS header is received, or
     // a response message (i.e. QR bit is set) is received.
     // a response message (i.e. QR bit is set) is received.
     if (opcode) {
     if (opcode) {
-        const int code = opcode.get().getCode();
-        server_msg_counter_.inc(opcode_to_msgcounter[code]);
+        server_msg_counter_.inc(opcode_to_msgcounter[opcode->getCode()]);
 
 
-        // Opcode = 0: Query
-        if (code == Opcode::QUERY_CODE) {
+        if (opcode.get() == Opcode::QUERY()) {
             // Recursion Desired bit
             // Recursion Desired bit
             if (msgattrs.requestHasRD()) {
             if (msgattrs.requestHasRD()) {
                 server_msg_counter_.inc(MSG_QRYRECURSION);
                 server_msg_counter_.inc(MSG_QRYRECURSION);