Skip to content

descriptor: remove orphaned dc_filter_garmin forward declaration - #141

Merged
mikeller merged 1 commit into
subsurface:Subsurface-DS9from
mikeller:fix/remove-orphaned-garmin-filter-171
Sep 30, 2026
Merged

mikeller merged 1 commit into
subsurface:Subsurface-DS9from
mikeller:fix/remove-orphaned-garmin-filter-171

Conversation

@mikeller

Copy link
Copy Markdown
Member

The function was never defined and never referenced. All Garmin
descriptors use NULL for the filter field, so no runtime filter is
needed. The declaration also had a mismatched signature (non-const
dc_descriptor_t *) compared to the dc_filter_t typedef and every
other filter in the file.

The function was never defined and never referenced. All Garmin
descriptors use NULL for the filter field, so no runtime filter is
needed. The declaration also had a mismatched signature (non-const
dc_descriptor_t *) compared to the dc_filter_t typedef and every
other filter in the file.

Signed-off-by: Michael Keller <github@ike.ch>
Copilot AI balanced review requested due to automatic review settings September 28, 2026 23:29

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@mikeller
mikeller merged commit af4d5f6 into subsurface:Subsurface-DS9 Sep 30, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants