Skip to content
This repository has been archived by the owner on Nov 6, 2022. It is now read-only.

Fix gcc comma at end of enumerator list warning. #531

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Fix gcc comma at end of enumerator list warning. #531

wants to merge 1 commit into from

Conversation

paulbartell
Copy link

@paulbartell paulbartell commented Apr 27, 2021 •
edited

Add dummy "LAST" items to each enum to mask the gcc "comma at end of enumerator list" warning and allow building with -Werror.

@viccpp
Copy link

viccpp commented Apr 29, 2021 •
edited

Trailing comma is permitted since C99 (and C++11)

@paulbartell
Copy link
Author

@viccpp . Very true. The FreeRTOS coreHTTP library that depends on http-parser targets C90 compilers.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@paulbartell @viccpp