Project

General

Profile

Feature #12386

Move definition of ONIG_CASE_MAPPING compilation switch outside onigumo files

Added by duerst (Martin Dürst) over 3 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Normal
Target version:
-
[ruby-core:75525]

Description

At r55018, in include/ruby/oniguruma.h, at line 131, I introduced
the compilation switch ONIG_CASE_MAPPING. The purpose of this switch
is to isolate the case-mapping functionality that is introduced into
Ruby but is not used with regular expressions.

The definition on line 131 should be moved outside oniguruma.h,
to a location that makes sure it is seen in oniguruma.h and all files
that include it, but will not be part of backwards pull requests
to onigumo. Such a location may either be an option setting in some
make file or a suitable .h file included into oniguruma.h. I trust
Nobu to find the best location for this.

Associated revisions

Revision 6ed393ad
Added by duerst (Martin Dürst) about 3 years ago

  • regenc.h/c, include/ruby/oniguruma.h, enc/ascii.c, big5.c, cp949.c, emacs_mule.c, euc_jp.c, euc_kr.c, euc_tw.c, gb18030.c, gbk.c, iso_8859_1|2|3|4|5|6|7|8|9|10|11|13|14|15|16.c, koi8_r.c, koi8_u.c, shift_jis.c, unicode.c, us_ascii.c, utf_16|32be|le.c, utf_8.c, windows_1250|51|52|53|54|57.c, windows_31j.c, unicode.c: Remove conditional compilation macro ONIG_CASE_MAPPING. [Feature #12386].

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@55740 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

Revision 55740
Added by duerst (Martin Dürst) about 3 years ago

  • regenc.h/c, include/ruby/oniguruma.h, enc/ascii.c, big5.c, cp949.c, emacs_mule.c, euc_jp.c, euc_kr.c, euc_tw.c, gb18030.c, gbk.c, iso_8859_1|2|3|4|5|6|7|8|9|10|11|13|14|15|16.c, koi8_r.c, koi8_u.c, shift_jis.c, unicode.c, us_ascii.c, utf_16|32be|le.c, utf_8.c, windows_1250|51|52|53|54|57.c, windows_31j.c, unicode.c: Remove conditional compilation macro ONIG_CASE_MAPPING. [Feature #12386].

Revision 55740
Added by duerst (Martin Dürst) about 3 years ago

  • regenc.h/c, include/ruby/oniguruma.h, enc/ascii.c, big5.c, cp949.c, emacs_mule.c, euc_jp.c, euc_kr.c, euc_tw.c, gb18030.c, gbk.c, iso_8859_1|2|3|4|5|6|7|8|9|10|11|13|14|15|16.c, koi8_r.c, koi8_u.c, shift_jis.c, unicode.c, us_ascii.c, utf_16|32be|le.c, utf_8.c, windows_1250|51|52|53|54|57.c, windows_31j.c, unicode.c: Remove conditional compilation macro ONIG_CASE_MAPPING. [Feature #12386].

Revision 55740
Added by duerst (Martin Dürst) about 3 years ago

  • regenc.h/c, include/ruby/oniguruma.h, enc/ascii.c, big5.c, cp949.c, emacs_mule.c, euc_jp.c, euc_kr.c, euc_tw.c, gb18030.c, gbk.c, iso_8859_1|2|3|4|5|6|7|8|9|10|11|13|14|15|16.c, koi8_r.c, koi8_u.c, shift_jis.c, unicode.c, us_ascii.c, utf_16|32be|le.c, utf_8.c, windows_1250|51|52|53|54|57.c, windows_31j.c, unicode.c: Remove conditional compilation macro ONIG_CASE_MAPPING. [Feature #12386].

Revision 55740
Added by duerst (Martin Dürst) about 3 years ago

  • regenc.h/c, include/ruby/oniguruma.h, enc/ascii.c, big5.c, cp949.c, emacs_mule.c, euc_jp.c, euc_kr.c, euc_tw.c, gb18030.c, gbk.c, iso_8859_1|2|3|4|5|6|7|8|9|10|11|13|14|15|16.c, koi8_r.c, koi8_u.c, shift_jis.c, unicode.c, us_ascii.c, utf_16|32be|le.c, utf_8.c, windows_1250|51|52|53|54|57.c, windows_31j.c, unicode.c: Remove conditional compilation macro ONIG_CASE_MAPPING. [Feature #12386].

History

Updated by naruse (Yui NARUSE) about 3 years ago

  • Assignee changed from nobu (Nobuyoshi Nakada) to duerst (Martin Dürst)
  • Status changed from Open to Assigned

The upstream of Ruby's regexp is Onigmo's ruby branch.
It is much different from master.
Therefore an extra ifdef, ONIG_CASE_MAPPING, is not needed.
Just write without ifdefs.

#2

Updated by duerst (Martin Dürst) about 3 years ago

  • Status changed from Assigned to Closed

Applied in changeset r55740.


  • regenc.h/c, include/ruby/oniguruma.h, enc/ascii.c, big5.c, cp949.c, emacs_mule.c, euc_jp.c, euc_kr.c, euc_tw.c, gb18030.c, gbk.c, iso_8859_1|2|3|4|5|6|7|8|9|10|11|13|14|15|16.c, koi8_r.c, koi8_u.c, shift_jis.c, unicode.c, us_ascii.c, utf_16|32be|le.c, utf_8.c, windows_1250|51|52|53|54|57.c, windows_31j.c, unicode.c: Remove conditional compilation macro ONIG_CASE_MAPPING. [Feature #12386].

Also available in: Atom PDF