feat(Corpus): location objects include column number

Author: alandefreitas

include/mrdocs/Metadata/Info/Location.hpp

@@ -39,6 +39,10 @@ struct MRDOCS_DECL
     */
     unsigned LineNumber = 0;
 
+    /** Column number within the line
+    */
+    unsigned ColumnNumber = 0;
+
     /** Whether this location has documentation.
     */
     bool Documented = false;
@@ -51,11 +55,13 @@ struct MRDOCS_DECL
         std::string_view const short_path = {},
         std::string_view const source_path = {},
         unsigned const line = 0,
+        unsigned const col = 0,
         bool const documented = false)
         : FullPath(full_path)
         , ShortPath(short_path)
         , SourcePath(source_path)
         , LineNumber(line)
+        , ColumnNumber(col)
         , Documented(documented)
     {
     }

src/lib/AST/ASTVisitor.cpp

@@ -580,21 +580,28 @@ populate(
     bool const definition,
     bool const documented)
 {
-    unsigned line = source_.getPresumedLoc(
-        loc, false).getLine();
+    auto presLoc = source_.getPresumedLoc(loc, false);
+    unsigned line = presLoc.getLine();
+    unsigned col = presLoc.getColumn();
     FileInfo* file = findFileInfo(loc);
 
     // No file is not an error, it just typically means it has been generated
     // in the virtual filesystem.
     MRDOCS_CHECK_OR(file);
 
+    Location Loc(file->full_path,
+        file->short_path,
+        file->source_path,
+        line,
+        col,
+        documented);
     if (definition)
     {
         if (I.DefLoc)
         {
             return;
         }
-        I.DefLoc.emplace(file->full_path, file->short_path, file->source_path, line, documented);
+        I.DefLoc = std::move(Loc);
     }
     else
     {
@@ -609,7 +616,7 @@ populate(
         {
             return;
         }
-        I.Loc.emplace_back(file->full_path, file->short_path, file->source_path, line, documented);
+        I.Loc.push_back(std::move(Loc));
     }
 }
 

src/lib/Metadata/Source.cpp

@@ -111,6 +111,7 @@ tag_invoke(
     io.map("shortPath", loc.ShortPath);
     io.map("sourcePath", loc.SourcePath);
     io.map("line", loc.LineNumber);
+    io.map("column", loc.ColumnNumber);
     io.map("documented", loc.Documented);
 }