[Padre-dev] Mistake in padre (executable)

[Gabor Szabo]

On Wed, Apr 1, 2009 at 3:51 PM, Offer Kaye <offer.kaye at gmail.com> wrote:
> On Wed, Apr 1, 2009 at 2:47 PM, Offer Kaye wrote:
>>
>> I think the grep will never return anything since @ARGV is empty at
>> this point (Getopt::Long::GetOptions already ran and empties it).
>>
>
> A similiar problem exists in the run method of Padre.pm which includes
> this code:
>
> {{{
>        # Handle architectural command line options
>        foreach my $M ( grep {/^-M/} @ARGV ) {
>                my $module = substr( $M, 2 );
>                eval "use $module";    ## no critic
>                die $@ if $@;
>        }
>        @ARGV = grep { !/^-M/ } @ARGV;
> }}}
>
> Once again, this code will never run anything assuming Padre is only
> run using "padre"...
>

This to seems to be old and unused.
I removed it.
If anyone is missing it you can revert change 3687

regards
  Gabor