Refactor: extract DocComment::getInnerText(PosTable)

src/libexpr/eval.cc

@@ -572,20 +572,7 @@ std::optional<EvalState::Doc> EvalState::getDoc(Value & v)
         }
 
         if (exprLambda->docComment) {
-            auto begin = positions[exprLambda->docComment.begin];
+            docStr = exprLambda->docComment.getInnerText(positions);
-            auto end = positions[exprLambda->docComment.end];
-            auto docCommentStr = begin.getSnippetUpTo(end);
-
-            // Strip "/**" and "*/"
-            constexpr size_t prefixLen = 3;
-            constexpr size_t suffixLen = 2;
-            docStr = docCommentStr.substr(prefixLen, docCommentStr.size() - prefixLen - suffixLen);
-            if (docStr.empty())
-                return {};
-            // Turn the now missing "/**" into indentation
-            docStr = "   " + docStr;
-            // Strip indentation (for the whole, potentially multi-line string)
-            docStr = stripIndentation(docStr);
         }
 
         if (name.empty()) {

src/libexpr/nixexpr.cc

@@ -641,4 +641,22 @@ size_t SymbolTable::totalSize() const
     return n;
 }
 
+std::string DocComment::getInnerText(const PosTable & positions) const {
+    auto beginPos = positions[begin];
+    auto endPos = positions[end];
+    auto docCommentStr = beginPos.getSnippetUpTo(endPos);
+
+    // Strip "/**" and "*/"
+    constexpr size_t prefixLen = 3;
+    constexpr size_t suffixLen = 2;
+    std::string docStr = docCommentStr.substr(prefixLen, docCommentStr.size() - prefixLen - suffixLen);
+    if (docStr.empty())
+        return {};
+    // Turn the now missing "/**" into indentation
+    docStr = "   " + docStr;
+    // Strip indentation (for the whole, potentially multi-line string)
+    docStr = stripIndentation(docStr);
+    return docStr;
+}
+
 }

src/libexpr/nixexpr.hh

@@ -60,6 +60,8 @@ struct DocComment {
      */
     operator bool() const { return static_cast<bool>(begin); }
 
+    std::string getInnerText(const PosTable & positions) const;
+
 };
 
 /**