Add more break points for long lines

[gold2718]

Some long lines are still broken incorrectly

Some long lines are still broken into invalid Fortran. This PR addresses this by adding more non-ID characters to the allowed break points (currently only spaces and commas).

User interface changes?: No

Fixes: TBA

Testing:
test removed:
unit tests:
system tests:
manual testing:

[gold2718]

@climbfuji, The latest Codee failure seems to be a mismatch between the default settings of the FortranWriter class and what Codee expects. Am I seeing that correctly? If so, I suggest we either modify Codee's expectations or the FortranWriter class defaults (e.g., __INDENT, __CONTINUE_INDENT, etc.). Thoughts?

Note, linebreak_test.F90 now compiles with no warnings so this is just a formatting issue.

[gold2718]

Another Codee comment. While I can match most of Codee's formatting expectations by setting __INDENT = 2, __CONTINUE_INDENT = 4, lowercasing CONTAINS, and minimizing the spacing before the continue character, Codee expects spaces around operators. Unfortunately, this defeats the purpose of the test which is how to break lines when spaces are not present.

Ironically, if capgen had been generating Codee-approved Fortran, @jimmielin would never have seen the bug he reported in #724 🤷.

[climbfuji]

Another Codee comment. While I can match most of Codee's formatting expectations by setting __INDENT = 2, __CONTINUE_INDENT = 4, lowercasing CONTAINS, and minimizing the spacing before the continue character, Codee expects spaces around operators. Unfortunately, this defeats the purpose of the test which is how to break lines when spaces are not present.

Ironically, if capgen had been generating Codee-approved Fortran, @jimmielin would never have seen the bug he reported in #724 🤷.

We had a similar issue with another test (#722; comment #722 (comment)). If you want to preserve formatting for certain parts of the code, you can wrap those lines in

! codee format off
...
! codee format on

See c221aaf

[climbfuji]

try this please

[climbfuji]

Suggested change

	character(len=7) :: data(100) = (/ 'name000', 'name001', 'name002', 'name003', 'name004', &
	! codee format off
	character(len=7) :: data(100) = (/ 'name000', 'name001', 'name002', 'name003', 'name004', &

[climbfuji]

Suggested change

	end subroutine foo
	end subroutine foo
	! codee format on