Remove doubly defined protocol version

ecmwf · Nov 14, 2024 · d134374 · d134374
1 parent f0ec6f4
commit d134374
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/src/gribjump/remote/GribJumpUser.cc b/src/gribjump/remote/GribJumpUser.cc
@@ -61,7 +61,7 @@ void GribJumpUser::handle_client(eckit::Stream& s, eckit::Timer& timer) {
 
     s >> version;
     if (version != remoteProtocolVersion) {
-        throw eckit::SeriousBug("Gribjump remote-protocol mismatch: Serverside version: " + std::to_string(protocolVersion_) + ", Clientside version: " + std::to_string(version));
+        throw eckit::SeriousBug("Gribjump remote-protocol mismatch: Serverside version: " + std::to_string(remoteProtocolVersion) + ", Clientside version: " + std::to_string(version));
     }
 
     LogContext ctx(s);

diff --git a/src/gribjump/remote/GribJumpUser.h b/src/gribjump/remote/GribJumpUser.h
@@ -39,7 +39,6 @@ class GribJumpUser : public eckit::net::NetUser {
 
 private:  // members
 
-    constexpr static uint16_t protocolVersion_ = 0;
 };
 
 //----------------------------------------------------------------------------------------------------------------------