Skip to content

Remove trailing comma from FL_USER3 #5958

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 26, 2022

Conversation

jemmaissroff
Copy link
Contributor

@jemmaissroff jemmaissroff commented May 26, 2022

The ruby/ruby CI is broken due to trailing comma from this commit

@k0kubun
Copy link
Member

k0kubun commented May 26, 2022

🤔 Do we support C++98? We've explicitly allowed trailing commas in enum declaration because it's supported by "C" compilers we support https://bugs.ruby-lang.org/projects/ruby-master/wiki/C99.

@jemmaissroff
Copy link
Contributor Author

@k0kubun I'm unsure. I'm going solely off of the error message in the broken CI run:

error: commas at the end of enumerator lists are a C++11 extension [-Werror,-Wc++11-extensions]
    RMODULE_IS_REFINEMENT            = RUBY_FL_USER3,

Please advise if I should do something different instead.

@k0kubun
Copy link
Member

k0kubun commented May 26, 2022

No worries, I was just expecting somebody who knows about it suddenly appears 🙂 I also posted the same question to a relevant PR #2991 (review).

@k0kubun
Copy link
Member

k0kubun commented May 26, 2022

We could discuss the policy separately. Meanwhile, let's just merge this to make CI successful. Thank you.

@k0kubun k0kubun merged commit a04d8c9 into ruby:master May 26, 2022
@XrXr
Copy link
Member

XrXr commented May 26, 2022

Do we support C++98?

It looks like we do for extensions from this: http://blade.nagaokaut.ac.jp/cgi-bin/scat.rb/ruby/ruby-core/94996

Each header that extensions could use has a section talking about C++98:

         We assume C99  for ruby itself but we don't  assume languages of
         extension libraries.  They could be written in C++98.

@k0kubun
Copy link
Member

k0kubun commented May 27, 2022

Oh okay. So header files are effectively still C89 and other files are C99. Thank you for finding it.

schneems pushed a commit to schneems/ruby that referenced this pull request Jul 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants