[Chapel Merge] fix(chpldoc): fix smoketests for linux32

Notifications Commits
1

Branch: refs/heads/main
Revision: 92bb7e2
Author: arezaii
Link: fix(chpldoc): fix smoketests for linux32 by arezaii · Pull Request #20603 · chapel-lang/chapel · GitHub
Log Message:

Merge pull request #20603 from arezaii/fix-chpldoc-linux32

fix(chpldoc): fix smoketests for linux32

This PR fixes broken linux32 tests. The failure was first caught
in smoke-testing.

Fix details:

  • the indentStream function was getting the size of the pointer
    of a string rather than the length of that string, and on 32
    bit systems that size is 4 vs. 8 on 64 bit systems, where the
    code was written and tested. Changing to get the length of the
    string as was surely intended fixes this. Thanks to @mppf for
    the simple solution and verifying it works on Linux32!

TESTING:

  • all chpldoc tests pass
  • make check-chpldoc passes on linux32

Reviewed by @mppf - thank you!

Signed-off-by: arezaii ahmad.rezaii@hpe.com

Modified Files:
M compiler/dyno/tools/chpldoc/chpldoc.cpp

Compare: https://github.com/chapel-lang/chapel/compare/cb934adec463...92bb7e21064c