Re: [PATCH] [bug #65110] [gropdf] should not include full argv[0] in diagnostic messages

2024-01-03 Thread G. Branden Robinson
At 2024-01-04T01:31:08-0600, Dave Kemper wrote: > On 1/3/24, G. Branden Robinson wrote: > > +{ > > + my ($v, $d, $f) = File::Spec->splitpath($prog); > > + $prog = $f; > > +} > > Since $v and $d are being discarded anyway, perl doesn't require you > to create dummy variables for them. > (http://

Re: [PATCH] [bug #65110] [gropdf] should not include full argv[0] in diagnostic messages

2024-01-03 Thread Dave Kemper
On 1/3/24, G. Branden Robinson wrote: > +{ > + my ($v, $d, $f) = File::Spec->splitpath($prog); > + $prog = $f; > +} Since $v and $d are being discarded anyway, perl doesn't require you to create dummy variables for them. (http://perldoc.perl.org/perlfunc#my-VARLIST) You can say: my (undef,

[PATCH] [bug #65111] "warning: Using nospace mode for font 'EURO'"

2024-01-03 Thread G. Branden Robinson
Hi Deri, I have another patch for gropdf. I tried, but my Perl may not be idiomatic; a couple of attempts to do simple things failed because the language and I appear to have different ideas about where to draw the statement/expression dichotomy, and how to express anonymous list literals. diff

[PATCH] [bug #65110] [gropdf] should not include full argv[0] in diagnostic messages

2024-01-03 Thread G. Branden Robinson
Hi Deri, I tried 3 times with 2 different browsers to update the ticket, but Savannah has fallen over. https://savannah.nongnu.org/support/?111000 I have a patch for gropdf. diff --git a/src/devices/gropdf/gropdf.pl b/src/devices/gropdf/gropdf.pl index 224f0e4b7..a55de5569 100644 --- a/src/devi