[llvm/DWARFDebugLine] Remove spurious full stop from warning messages
authorPavel Labath <pavel@labath.sk>
Thu, 11 Jun 2020 11:13:31 +0000 (13:13 +0200)
committerPavel Labath <pavel@labath.sk>
Thu, 11 Jun 2020 11:14:21 +0000 (13:14 +0200)
Other warnings messages don't have a trailing full stop.

llvm/lib/DebugInfo/DWARF/DWARFDebugLine.cpp
llvm/test/tools/llvm-dwarfdump/X86/debug_line_invalid.test
llvm/unittests/DebugInfo/DWARF/DWARFDebugLineTest.cpp

index 2faee15..df9a496 100644 (file)
@@ -275,14 +275,14 @@ parseV5DirFileTables(const DWARFDataExtractor &DebugLineData,
         if (!Value.extractValue(DebugLineData, OffsetPtr, FormParams, &Ctx, U))
           return createStringError(errc::invalid_argument,
                                    "failed to parse directory entry because "
-                                   "extracting the form value failed.");
+                                   "extracting the form value failed");
         IncludeDirectories.push_back(Value);
         break;
       default:
         if (!Value.skipValue(DebugLineData, OffsetPtr, FormParams))
           return createStringError(errc::invalid_argument,
                                    "failed to parse directory entry because "
-                                   "skipping the form value failed.");
+                                   "skipping the form value failed");
       }
     }
   }
@@ -302,7 +302,7 @@ parseV5DirFileTables(const DWARFDataExtractor &DebugLineData,
       if (!Value.extractValue(DebugLineData, OffsetPtr, FormParams, &Ctx, U))
         return createStringError(errc::invalid_argument,
                                  "failed to parse file entry because "
-                                 "extracting the form value failed.");
+                                 "extracting the form value failed");
       switch (Descriptor.Type) {
       case DW_LNCT_path:
         FileEntry.Name = Value;
index 0b2116a..053063e 100644 (file)
 ## V5 prologue ends during directory table.
 # NONFATAL:      debug_line[0x00000214]
 # SOME-ERR-NEXT: warning: parsing line table prologue at 0x00000214 found an invalid directory or file table description at 0x00000236
-# SOME-ERR-NEXT: warning: failed to parse directory entry because extracting the form value failed.
+# SOME-ERR-NEXT: warning: failed to parse directory entry because extracting the form value failed
 # NONFATAL-NEXT: Line table prologue
 # NONFATAL-NOT:  include_directories
 # VERBOSE:       DW_LNE_set_address (0x4444333322221111)
 ## V5 invalid MD5 hash form when there is still data to be read.
 # NONFATAL:      debug_line[0x00000244]
 # SOME-ERR-NEXT: warning: parsing line table prologue at 0x00000244 found an invalid directory or file table description at 0x00000277
-# SOME-ERR-NEXT: warning: failed to parse file entry because extracting the form value failed.
+# SOME-ERR-NEXT: warning: failed to parse file entry because extracting the form value failed
 # NONFATAL-NEXT: Line table prologue
 # NONFATAL:      include_directories[  0] = "/tmp"
 # NONFATAL-NOT:  file_names
 ## V5 invalid directory content description has unsupported form.
 # NONFATAL:      debug_line[0x000002c0]
 # SOME-ERR-NEXT: warning: parsing line table prologue at 0x000002c0 found an invalid directory or file table description at 0x000002e9
-# SOME-ERR-NEXT: warning: failed to parse directory entry because skipping the form value failed.
+# SOME-ERR-NEXT: warning: failed to parse directory entry because skipping the form value failed
 # NONFATAL-NEXT: Line table prologue
 # NONFATAL:      include_directories[  0] = "/foo"
 # NONFATAL-NOT:  include_directories
index d01c24a..6a99bbf 100644 (file)
@@ -472,7 +472,7 @@ TEST_P(DebugLineParameterisedFixture, ErrorForTooShortPrologueLength) {
                       "of the prologue");
   } else {
     Errs.emplace_back(
-        "failed to parse file entry because extracting the form value failed.");
+        "failed to parse file entry because extracting the form value failed");
   }
   EXPECT_THAT_ERROR(std::move(Recoverable),
                     FailedWithMessageArray(testing::ElementsAreArray(Errs)));