From: Nobuyoshi Nakada <nobu@...>
Date: 2007-04-18T09:51:40+09:00
Subject: Re: backwards compatibility for 'raise Interrupt'

Hi,

At Wed, 18 Apr 2007 06:49:50 +0900,
Ryan Davis wrote in [ruby-core:10947]:
> % ./ruby -ve 'raise Interrupt'; ruby -ve 'raise Interrupt'
> ruby 1.8.6 (2007-03-24 patchlevel 5000) [i686-darwin8.9.1]
> -e:1:in `initialize': wrong number of arguments (0 for 1)  
> (ArgumentError)
>          from -e:1:in `exception'
>          from -e:1:in `raise'
>          from -e:1
> ruby 1.8.5 (2006-08-25) [i686-darwin8.8.1]
> -e:1: Interrupt (Interrupt)

Sorry it's my fault, and thank you for the report/patch.

> +    rb_scan_args(argc, argv, "01", &mesg);

mesg seems unnecessary, instead you can use args[1] directly.

> +    args[0] = INT2FIX(SIGINT);
> +    args[1] = mesg;
> +    return rb_call_super(2, args);

It feels better to use argc + 1 rather than 2.

-- 
Nobu Nakada