[Chapel Merge] compiler/next: Stop using public fields in scope-t

Notifications Commits
1

Branch: refs/heads/main
Revision: a629b61
Author: aconsroe-hpe
Link: Unavailable
Log Message:

Merge pull request #18702 from aconsroe-hpe/next-cleanups-public-fields

compiler/next: Stop using public fields in scope-types

reviewed by @mppf

  • Makes public fields private behind getter methods where appropriate.

  • Adds an Iterable template as a more opaque "thing to be iterated over"

  • Adds doc comments to classes, getters, and enums

    Modified Files:
    A compiler/next/include/chpl/util/iteration.h
    M compiler/next/doc/doxygen-pitfalls.txt
    M compiler/next/include/chpl/resolution/scope-types.h
    M compiler/next/lib/resolution/resolution-queries.cpp
    M compiler/next/lib/resolution/scope-queries.cpp
    M compiler/next/test/resolution/testScopeResolve.cpp
    M doc/rst/meta/compiler-internals-doxygen/base.rst

    Compare: Comparing 78f8784efd2c...a629b6142796 · chapel-lang/chapel · GitHub