The source project of this merge request has been removed.
lalapps_splitSFTs: drop -i argument, support concatenated SFTs
Description
3 patches to lalapps_splitSFTs
and associated tests:
- Drop support for the
-i
argument for input files; since these always have to been last on the command line, they're really positional arguments, so don't need an optional arguments (although--
is supported as a separator.) - Add support for concatenated SFTs. Code now loops over all SFTs in all input files, rather than just reading the first.
- Whitespace/indentation/style cleanup, delayed from 2nd patch so that those changes are easier to see. Also do style cleanup on
splitSFTs.c
for easier legibility.
Closes https://git.ligo.org/CW/software/lalsuite/issues/77
API Changes and Justification
Backwards Compatible Changes
-
This change introduces no API changes -
This change adds new API calls
Backwards Incompatible Changes
-
This change modifies an existing API -
This change removes an existing API
Removing -i
argument is backwards incompatible, but this code is not widely used in CW (yet) so impact should be minimal.
Review Status
@david-keitel to review.
Edited by Ghost User