jnthn / oo-monitors

Monitors with condition variables for Raku
12 stars 7 forks source link

Useless use #11

Closed AlexDaniel closed 6 years ago

AlexDaniel commented 6 years ago

When installing this module:

===> Testing: OO::Monitors:ver<1.1>
WARNINGS for /home/travis/.zef/store/oo-monitors.git/b9de07c277f0a9fa805bb18113788228f6acfd3b/t/condition.t:
Useless use of constant value not-empty in sink context (lines 15, 20)
Useless use of constant value not-full in sink context (lines 12, 22)
jnthn commented 6 years ago

That'd be a Rakudo bug. The values in question are arguments to a macro, and the argument is used.

zoffixznet commented 6 years ago

This issue can probably closed, I golfed it down to something that doesn't involve this module at all:

use experimental :macros; macro z($) { quasi {} }; z $;
jnthn commented 6 years ago

Verified that, with the Rakudo warning bug fixed, the warning in this module is also gone.