Rename k_JOURNALOP_TYPE constant

Signed-off-by: Aleksandr Ivanov <[email protected]>
bloomberg · Dec 3, 2024 · 8e63247 · 8e63247
1 parent 70ffcb2
commit 8e63247
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/src/applications/bmqstoragetool/m_bmqstoragetool_parameters.cpp b/src/applications/bmqstoragetool/m_bmqstoragetool_parameters.cpp
@@ -74,9 +74,9 @@ bool isValidQueueKeyHexRepresentation(const char* queueKeyBuf)
 // class CommandLineArguments
 // ==========================
 
-const char* CommandLineArguments::k_MESSAGE_TYPE = "message";
-const char* CommandLineArguments::k_QUEUEOP_TYPE = "queue-op";
-const char* CommandLineArguments::k_JOURNAL_TYPE = "journal-op";
+const char* CommandLineArguments::k_MESSAGE_TYPE   = "message";
+const char* CommandLineArguments::k_QUEUEOP_TYPE   = "queue-op";
+const char* CommandLineArguments::k_JOURNALOP_TYPE = "journal-op";
 
 CommandLineArguments::CommandLineArguments(bslma::Allocator* allocator)
 : d_recordType(allocator)
@@ -390,9 +390,12 @@ Parameters::Parameters(const CommandLineArguments& arguments,
         else if (*cit == CommandLineArguments::k_QUEUEOP_TYPE) {
             d_processRecordTypes.d_queueOp = true;
         }
-        else {
+        else if (*cit == CommandLineArguments::k_JOURNALOP_TYPE) {
             d_processRecordTypes.d_journalOp = true;
         }
+        else {
+            BSLS_ASSERT(false && "Unknown journal record type");
+        }
     }
 
     // Set search range type and values if present

diff --git a/src/applications/bmqstoragetool/m_bmqstoragetool_parameters.h b/src/applications/bmqstoragetool/m_bmqstoragetool_parameters.h
@@ -56,7 +56,7 @@ struct CommandLineArguments {
     // PUBLIC DATA
     static const char* k_MESSAGE_TYPE;
     static const char* k_QUEUEOP_TYPE;
-    static const char* k_JOURNAL_TYPE;
+    static const char* k_JOURNALOP_TYPE;
     // Record types constants
     bsl::vector<bsl::string> d_recordType;
     // List of record types to process (message, journalOp, queueOp)