Feature #13125: MRI has too much Qtrue : Qfalse;

• Status: Closed
• Assignee: -
• Target version: -
• Tags: build

[shyouhei (Shyouhei Urabe)]

% git grep -P 'Q(true|false|nil|undef)\s+:\s+Q(true|false|nil|undef)' | wc -l
211

Out of those 211 occurrences the most interesting lines include:

complex.c:#define f_boolcast(x) ((x) ? Qtrue : Qfalse)
math.c:#define f_boolcast(x) ((x) ? Qtrue : Qfalse)
range.c:#define RBOOL(v) ((v) ? Qtrue : Qfalse)
rational.c:#define f_boolcast(x) ((x) ? Qtrue : Qfalse)

I strongly believe that we need an orthodox way to do this.

[shyouhei (Shyouhei Urabe)]

We looked at this issue at yesterday's developer meeting. Nobody was against introducing a macro. No consensus over its name was made though.

• RBOOL() seems like a member of RSTRING(), RARRAY() and family so not very good.
• It is in fact a INT2FIX()'s family so I proposed BOOL2BOOL() but this sounds too strange.
• What is needed is a "reverse" of RTEST() so RUNTEST() is proposed (I believe it was not serious).

[rmosolgo (Robert Mosolgo)]

"reverse" of RTEST()

How about ...

• CTEST()
• TEST2BOOL()
• C2BOOL()

Hope those help brainstorm a bit :)

[znz (Kazuhiro NISHIYAMA)]

How about BOOL2RBOOL?

[phluid61 (Matthew Kerwin)]

Seem to me like VALUE2BOOL, except perhaps for the way it casts Qnil to Qtrue

[nobu (Nobuyoshi Nakada)]

Someone suggested RB_BOOL at the meeting.

[jeremyevans0 (Jeremy Evans)]

• Tracker changed from Bug to Feature
• Backport deleted (2.2: UNKNOWN, 2.3: UNKNOWN, 2.4: UNKNOWN)

[jeremyevans0 (Jeremy Evans)]

• Status changed from Open to Closed

RBOOL was recently introduced for this purpose.