[Chapel Merge] compiler/next: avoid public fields in FileContents

Notifications Commits
1

Branch: refs/heads/main
Revision: d9d5b18
Author: mppf
Log Message:

Merge pull request #18528 from mppf/next-pub-fields-2

compiler/next: avoid public fields in FileContents

  • move FileContents to its own file, avoid public fields in it
  • move update functions for it to that file
  • move update function for ASTTag to ASTTag.h
  • Adjust BuilderResult to return errors by const ref
  • improve doxygen comments for BuilderResult
  • de-indent parsing-queries.h contents to match pattern of not requiring
    indenting for large namespace blocks

Reviewed by @dlongnecke-cray - thanks!

  • [x] full local testing

    Modified Files:
    A compiler/next/include/chpl/parsing/FileContents.h
    M compiler/next/include/chpl/parsing/parsing-queries.h
    M compiler/next/include/chpl/uast/ASTNode.h
    M compiler/next/include/chpl/uast/ASTTag.h
    M compiler/next/include/chpl/uast/BuilderResult.h
    M compiler/next/lib/parsing/parsing-queries.cpp
    M compiler/next/test/parsing/testParsingQueries.cpp

    Compare: Comparing cc3ad6498baa...d9d5b18b7339 · chapel-lang/chapel · GitHub