Merge pull request #127 from rhansen/delete-unused-varargs
Delete unused varargs from _read_config and parse_assignment(s)
This commit is contained in:
commit
afba2f375e
1 changed files with 7 additions and 13 deletions
20
ddclient
20
ddclient
|
@ -1068,14 +1068,13 @@ sub read_cache {
|
||||||
## parse_assignment(string) return (name, value, rest)
|
## parse_assignment(string) return (name, value, rest)
|
||||||
######################################################################
|
######################################################################
|
||||||
sub parse_assignments {
|
sub parse_assignments {
|
||||||
my $rest = shift;
|
my ($rest) = @_;
|
||||||
my @args = @_;
|
|
||||||
my %variables = ();
|
my %variables = ();
|
||||||
my ($name, $value);
|
my ($name, $value);
|
||||||
|
|
||||||
while (1) {
|
while (1) {
|
||||||
$rest =~ s/^\s+//;
|
$rest =~ s/^\s+//;
|
||||||
($name, $value, $rest) = parse_assignment($rest, @args);
|
($name, $value, $rest) = parse_assignment($rest);
|
||||||
if (defined $name) {
|
if (defined $name) {
|
||||||
$variables{$name} = $value;
|
$variables{$name} = $value;
|
||||||
} else {
|
} else {
|
||||||
|
@ -1085,8 +1084,7 @@ sub parse_assignments {
|
||||||
return ($rest, %variables);
|
return ($rest, %variables);
|
||||||
}
|
}
|
||||||
sub parse_assignment {
|
sub parse_assignment {
|
||||||
my $rest = shift;
|
my ($rest) = @_;
|
||||||
my $stop = @_ ? shift : '[\n\s,]';
|
|
||||||
my ($c, $name, $value);
|
my ($c, $name, $value);
|
||||||
my ($escape, $quote) = (0, '');
|
my ($escape, $quote) = (0, '');
|
||||||
|
|
||||||
|
@ -1104,7 +1102,7 @@ sub parse_assignment {
|
||||||
$quote = ''
|
$quote = ''
|
||||||
} elsif (!$quote && $c =~ /[\'\"]/) {
|
} elsif (!$quote && $c =~ /[\'\"]/) {
|
||||||
$quote = $c;
|
$quote = $c;
|
||||||
} elsif (!$quote && $c =~ /^${stop}/) {
|
} elsif (!$quote && $c =~ /^[\n\s,]/) {
|
||||||
last;
|
last;
|
||||||
} else {
|
} else {
|
||||||
$value .= $c;
|
$value .= $c;
|
||||||
|
@ -1118,19 +1116,15 @@ sub parse_assignment {
|
||||||
## read_config
|
## read_config
|
||||||
######################################################################
|
######################################################################
|
||||||
sub read_config {
|
sub read_config {
|
||||||
my $file = shift;
|
my ($file, $config, $globals) = @_;
|
||||||
my $config = shift;
|
_read_config($config, $globals, '', $file);
|
||||||
my $globals = shift;
|
|
||||||
my %globals = ();
|
|
||||||
|
|
||||||
_read_config($config, $globals, '', $file, %globals);
|
|
||||||
}
|
}
|
||||||
sub _read_config {
|
sub _read_config {
|
||||||
my $config = shift;
|
my $config = shift;
|
||||||
my $globals = shift;
|
my $globals = shift;
|
||||||
my $stamp = shift;
|
my $stamp = shift;
|
||||||
local $file = shift;
|
local $file = shift;
|
||||||
my %globals = @_;
|
my %globals = ();
|
||||||
my %config = ();
|
my %config = ();
|
||||||
my $content = '';
|
my $content = '';
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue