Skip to content

lalframe: remove unused LALStatus functions

Kipp Cannon requested to merge (removed):lalframe_cleanup into master

Description

As time has passed, a number of obsolete LALStatus-based functions in lalframe have finally come to be unused. This merge request deletes them: LALFrSetMode(), LALFrEnd(), LALFrNext(), LALFrRewind(), LALFrTell(), LALFrGetTimeSeriesType(), LALFrGetCOMPLEX16TimeSeries(), LALFrGetCOMPLEX16TimeSeriesMetadata(), LALFrGetREAL4FrequencySeries(), LALFrGetREAL8FrequencySeries(), LALFrGetCOMPLEX16FrequencySeries(), LALFrWriteINT2TimeSeries(), LALFrWriteINT8TimeSeries(), LALFrWriteREAL8TimeSeries(), LALFrWriteCOMPLEX8TimeSeries(), LALFrWriteCOMPLEX16TimeSeries(), LALFrWriteREAL4FrequencySeries(), LALFrWriteREAL8FrequencySeries(), LALFrWriteCOMPLEX8FrequencySeries(), LALFrWriteCOMPLEX16FrequencySeries(). All of these functions live in LALFrStreamLegacy.c. This patch does not touch the non-legacy code in lalframe

API Changes and Justification

Backwards Compatible Changes

  • This change does not modify any class/function/struct/type definitions in a public C header file or any Python class/function definitions
  • This change adds new classes/functions/structs/types to a public C header file or Python module

Backwards Incompatible Changes

  • This change modifies an existing class/function/struct/type definition in a public C header file or Python module
  • This change removes an existing class/function/struct/type from a public C header file or Python module

If any of the Backwards Incompatible check boxes are ticked please provide a justification why this change is necessary and why it needs to be done in a backwards incompatible way.

Review Status

Not reviewed.

Edited by Kipp Cannon

Merge request reports