Skip to content

Just avoiding two compiler warnings#8958

Merged
seisman merged 1 commit intomasterfrom
stop_warnings
Apr 2, 2026
Merged

Just avoiding two compiler warnings#8958
seisman merged 1 commit intomasterfrom
stop_warnings

Conversation

@remkos
Copy link
Copy Markdown
Contributor

@remkos remkos commented Mar 29, 2026

Description of proposed changes

Changed two source files to avoid compiler warnings.

  • gmt_remote.c: removed unused variable.
  • grdview.c: cast input argument to curl_easy_setopt as long, as is required.

No bug fixes, no new functionalities.

Reminders

  • Make sure that your code follows our style. Use the other functions/files as a basis.
  • Add tests for new features or tests that would have caught the bug that you're fixing.
  • Describe changes to function behavior and arguments in a comment below the function declaration.
  • If adding new functionality, add a detailed description to the documentation and/or an example.

@remkos remkos requested a review from joa-quim March 29, 2026 15:50
@seisman seisman merged commit 0141327 into master Apr 2, 2026
11 of 14 checks passed
@seisman seisman deleted the stop_warnings branch April 2, 2026 07:51
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