search

LLNL / libROM

Model reduction library with an emphasis on large scale parallelism and linear subspace methods
https://www.librom.net
Other
201 stars 36 forks source link

Character limit per line #115

Closed siuwuncheung closed 2 years ago

siuwuncheung commented 2 years ago

This PR simply adds a 80-character limit to the scripts throughout libROM using astyle, similar to MFEM. Short lines are more easily read and reviewed on GitHub.

Long lines can be truncated by running ./scripts/stylize.sh $PATH_TO_ASTYLE.

kevinhkhuynh commented 2 years ago

Can we wait until S-OPT is merged before merging this? I feel like this change will cause a bunch of merge conflicts.

dylan-copeland commented 2 years ago

Since we are already changing a lot of files, some other simple changes could be made in this PR:

  1. Update copyright year from 2021 to 2022 (multiple files).
  2. Change .C to .cpp.
kevinhkhuynh commented 2 years ago

@dylan-copeland Requested updates have been made.