Bug #18345: Duplicate range in character class warning

• Status: Rejected
• Assignee: -
• Target version: -
• ruby -v: >= 2.5.5
• Backport: 2.6: UNKNOWN, 2.7: UNKNOWN, 3.0: UNKNOWN

[halilsen (Halil Sen)]

#!/usr/bin/env ruby -w
puts RUBY_VERSION
puts "".match?(/[\p{Alnum}|\-|\_]/).inspect
puts '--------------'

trial.rb:3: warning: character class has duplicated range: /[\p{Alnum}|\-|\_]/
2.5.5
false
--------------
trial.rb:3: warning: character class has duplicated range: /[\-|\p{Alnum}|\_]/
2.7.4
false
--------------

If I remove any part of the character class definition [], there is no warning but
I don't think there's a duplicate range here, so there should not be a warning.

[xtkoba (Tee KOBAYASHI)]

| seems to appear twice.

[halilsen (Halil Sen)]

xtkoba (Tee KOBAYASHI) wrote in #note-1:

| seems to appear twice.

Effectively, I didn't see the |s, I must have parse them as logical or. This issue can be closed.

I will open a PR on the grape-swagger side. Cheers xtkoba!

https://github.com/ruby-grape/grape-swagger/blob/4ea61c4c2c0bd88043ebc15041b0eb3967b341ff/lib/grape-swagger.rb#L35

[nobu (Nobuyoshi Nakada)]

• Status changed from Open to Rejected