[#44036] [ruby-trunk - Feature #6242][Open] Ruby should support lists — "shugo (Shugo Maeda)" <redmine@...>

20 messages 2012/04/01

[#44084] [ruby-trunk - Bug #6246][Open] 1.9.3-p125 intermittent segfault — "jshow (Jodi Showers)" <jodi@...>

22 messages 2012/04/02

[#44156] [ruby-trunk - Feature #6265][Open] Remove 'useless' 'concatenation' syntax — "rosenfeld (Rodrigo Rosenfeld Rosas)" <rr.rosas@...>

45 messages 2012/04/06

[#44163] [ruby-trunk - Bug #6266][Open] encoding related exception with recent integrated psych — "jonforums (Jon Forums)" <redmine@...>

10 messages 2012/04/06

[#44303] [ruby-trunk - Feature #6284][Open] Add composition for procs — "pabloh (Pablo Herrero)" <pablodherrero@...>

57 messages 2012/04/12

[#44349] [ruby-trunk - Feature #6293][Open] new queue / blocking queues — "tenderlovemaking (Aaron Patterson)" <aaron@...>

10 messages 2012/04/13

[#44402] [ruby-trunk - Feature #6308][Open] Eliminate delegation from WeakRef — "headius (Charles Nutter)" <headius@...>

20 messages 2012/04/17

[#44403] [ruby-trunk - Feature #6309][Open] Add a reference queue for weak references — "headius (Charles Nutter)" <headius@...>

15 messages 2012/04/17

[#44533] [ruby-trunk - Bug #6341][Open] SIGSEGV: Thread.new { fork { GC.start } }.join — "rudolf (r stu3)" <redmine@...>

24 messages 2012/04/22

[#44630] [ruby-trunk - Feature #6361][Open] Bitwise string operations — "MartinBosslet (Martin Bosslet)" <Martin.Bosslet@...>

31 messages 2012/04/26

[#44648] [ruby-trunk - Feature #6367][Open] #same? for Enumerable — "prijutme4ty (Ilya Vorontsov)" <prijutme4ty@...>

16 messages 2012/04/26

[#44704] [ruby-trunk - Feature #6373][Open] public #self — "trans (Thomas Sawyer)" <transfire@...>

61 messages 2012/04/27

[#44748] [ruby-trunk - Feature #6376][Open] Feature lookup and checking if feature is loaded — "trans (Thomas Sawyer)" <transfire@...>

13 messages 2012/04/28

[ruby-core:44210] Re: [ruby-trunk - Feature #5716][Open] duplicated when clause is ignored -> Report which clause it is specifically.

From: SASADA Koichi <ko1@...>
Date: 2012-04-09 05:11:24 UTC
List: ruby-core #44210
Thank you for your comment.

I added a small hint on it because if duplicated object was too big, the
output should be ugly.

Does it help you?

# sample code
case nil
when 1
when 1, 1
when 1, 1, 1, 1
when 1, 1
end

#=>
../trunk/test.rb:1: warning: duplicated when clause (#1) is ignored
../trunk/test.rb:1: warning: duplicated when clause (#2) is ignored
../trunk/test.rb:1: warning: duplicated when clause (#3) is ignored
../trunk/test.rb:1: warning: duplicated when clause (#4) is ignored
../trunk/test.rb:1: warning: duplicated when clause (#5) is ignored
../trunk/test.rb:1: warning: duplicated when clause (#6) is ignored
../trunk/test.rb:1: warning: duplicated when clause (#7) is ignored
../trunk/test.rb:1: warning: duplicated when clause (#8) is ignored


Patch:
Index: compile.c
===================================================================
--- compile.c	(revision 35267)
+++ compile.c	(working copy)
@@ -1454,8 +1454,9 @@
 				    rb_hash_aset(map, obj, INT2FIX(lobj->position - (pos+len)));
 				}
 				else {
+				    int n = i/2;
 				    rb_compile_warning(RSTRING_PTR(iseq->filename), iobj->line_no,
-						       "duplicated when clause is ignored");
+						       "duplicated when clause (#%d) is ignored", n);
 				}
 			    }
 			    hide_obj(map);


(2011/12/06 19:07), markus heiler wrote:
> 
> Issue #5716 has been reported by markus heiler.
> 
> ----------------------------------------
> Feature #5716: duplicated when clause is ignored -> Report which clause it is specifically.
> http://redmine.ruby-lang.org/issues/5716
> 
> Author: markus heiler
> Status: Open
> Priority: Low
> Assignee: 
> Category: 
> Target version: 
> 
> 
> In latest Ruby 1.9.3, if you have warnings enabled, you get information like this here:
> 
> foobar.rb:24: warning: duplicated when clause is ignored
> 
> This error comes from the file:
> 
>   compile.c
> 
> 
> rb_compile_warning(RSTRING_PTR(iseq->filename), iobj->line_no,
>     "duplicated when clause is ignored");
> 
> 
> This is called, it seems, when the prior function fails:
> 
>   rb_hash_aset(map, obj, INT2FIX(lobj->position - (pos+len)));
> 
> 
> As a user of Ruby, the warning should be better.
> 
> Rather than:
> 
>   foobar.rb:24: warning: duplicated when clause is ignored
> 
> I would like to see:
> 
>   foobar.rb:24: warning: duplicated when clause 'foo' is ignored.
> 
> This way, I can scan through the file with the case/when structure
> and correct this error. But right now, if you have a very long 
> case/when structure, you have no idea WHICH clause is duplicated.
> 
> The warning message just vaguely says that there is, and this is
> not very helpful information.
> 
> 


-- 
// SASADA Koichi at atdot dot net

In This Thread