Skip to content

wgrib src #445 - #518

Merged
AlysonStahl-NOAA merged 2 commits into
developfrom
wgrib
Jun 9, 2026
Merged

wgrib src #445#518
AlysonStahl-NOAA merged 2 commits into
developfrom
wgrib

Conversation

@webisu

@webisu webisu commented May 16, 2026

Copy link
Copy Markdown
Collaborator

issue #445 change wgrib.c to *.c and *.h

Historically the *.c and *.h files for wgrib were combined to wgrib.c for
convenience of the users. This pull request reverts the source to *.c and *.h
for future development.

Minor change: icx complains about "for (....) ;" even though that is
a legal statement. Icx wants the semicolon on the following line.

@webisu
webisu requested a review from AlysonStahl-NOAA May 16, 2026 22:14
@manfredsc manfredsc mentioned this pull request Jun 8, 2026
@AlysonStahl-NOAA
AlysonStahl-NOAA merged commit 78f43bd into develop Jun 9, 2026
18 checks passed
AlysonStahl-NOAA added a commit that referenced this pull request Jun 11, 2026
…#529)

Once wgrib is cleaned up as well (as follow-up to #518), one could consider
"-Wunused-macros" to be added to the default compile flags.

Co-authored-by: AlysonStahl-NOAA <166434581+AlysonStahl-NOAA@users.noreply.github.com>
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