timothygrant80 / cisTEM

Other
32 stars 27 forks source link

Some spring cleaning #500

Closed bHimes closed 7 months ago

bHimes commented 7 months ago

Remove footguns from numeric text file, expand tests and remove duplicated defines and replace with constepxr.

Description

NOTE: the behavior of the constructor/OPEN argument "wanted_records_per_line" is a bit misleading as it seems to only be valid when OPEN_TO_WRITE

NOTE: OPEN_TO_APPEND is not implemented - added a debug assert

For expand symmetry stack and par, just removed nested comments to clear a compiler warning and also to reduce clutter when doing repo wide regex.

Fixes # (issue)

I have rebased my feature branch to be current with the master branch using to minimize conflicts and headaches

Which compilers were tested

These changes are isolated to the

How has the functionality been tested?

Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

Checklist: