ruby / ruby/cgi

Drop C89 support (= Ruby 2.x support)?

Open
#18 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Ruby
Stars
55
Forks
33
Avg merge
7d 3h
Merged PRs (30d)
1

Description

ext/cgi/escape.c uses C99 features, which cannot use before ruby 3.0.

  • Designated Initializers for html_escape_table
  • Variable declaration after code in optimized_escape_html

The latter is easy to make C89-compliant but the former needs to expand the table.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with ext/cgi/escape.c, especially the html_escape_table designated initializers and the declaration after code in optimized_escape_html. Determine whether C89 compatibility should be retained or Ruby 2.x support dropped; done means the compatibility decision is resolved and the affected code follows the chosen support policy.

Written by the indexing model from the issue text.

Assessment

Tech stack
c, ruby
Domain
web-dev
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.