jsoriano/sql-translator — comment on commit f60322d8fc2391c455c0ecc84bd1a4640f9602a2
File:  line 0
By ribasushi:

This is saner yes. Please also add extra tests to t/30sqlt-diff.t. Also while you are at it it'd be a good idea to rip out the insane @ARGV parsing, and replace it with a call to the core Getopt::Long. This is what I usually use:

```
require Getopt::Long;
my $getopt = Getopt::Long::Parser->new(
config => [qw/gnu_getopt bundling_override no_ignore_case pass_through/]
);
my $args = {
#a hashref with specification of defaults
};
$getopt->getoptions($args, @optionspec );
if (@ARGV) {
warn "\nIgnoring unrecognized option(s): @ARGV\n\n";
}

# everything is in $args at this point