[SCM] rel2gpx branch, master, updated. 8d846f1eb5a17fef4b2a164fa995b61f1522fa63
Andreas Tille
tille at debian.org
Sun Sep 15 20:14:16 UTC 2013
The following commit has been merged in the master branch:
commit 8d846f1eb5a17fef4b2a164fa995b61f1522fa63
Author: Andreas Tille <tille at debian.org>
Date: Sun Sep 15 19:24:26 2013 +0200
The defined needs to stay but dereferencing is wrong (thanks to gregor herrmann <gregoa at debian.org> for the hint)
diff --git a/debian/patches/omit_defined.patch b/debian/patches/omit_defined.patch
index 8c2fdb2..24a338b 100644
--- a/debian/patches/omit_defined.patch
+++ b/debian/patches/omit_defined.patch
@@ -1,13 +1,15 @@
-Author: Andreas Tille <tille at debian.org>
-Date: Thu, 12 Sep 2013 21:34:03 +0200
-Description: Prevent useless warnings saying:
- defined(@array) is deprecated at /usr/bin/rel2gpx line 669.
- (Maybe you should just omit the defined()?)
- defined(@array) is deprecated at /usr/bin/rel2gpx line 671.
- (Maybe you should just omit the defined()?)
- defined(@array) is deprecated at /usr/bin/rel2gpx line 677.
- (Maybe you should just omit the defined()?)
- ... but this patch is unfortunately wrong
+Author: gregor herrmann <gregoa at debian.org>
+Date: Sat, 14 Sep 2013 14:28:02 +0200
+Description: Avoid dereferencing
+ Perl says:
+ defined(@array) is deprecated at /usr/bin/rel2gpx line 669.
+ (Maybe you should just omit the defined()?)
+ defined(@array) is deprecated at /usr/bin/rel2gpx line 671.
+ (Maybe you should just omit the defined()?)
+ defined(@array) is deprecated at /usr/bin/rel2gpx line 677.
+ (Maybe you should just omit the defined()?)
+ ... but this hint is wrong. The defined needs to be kept but the
+ dereferencing is wrong
--- rel2gpx.orig/rel2gpx.pl
+++ rel2gpx/rel2gpx.pl
@@ -16,17 +18,17 @@ Description: Prevent useless warnings saying:
@{$self->{NID}} = reverse(@{$self->{NID}});
@{$self->{wayIds}} = reverse(@{$self->{wayIds}});
- if (defined @{$self->{PREV}}){
-+ if (@{$self->{PREV}}){
++ if (defined $self->{PREV}){
my @aTmp = (@{$self->{PREV}});
- if (defined @{$self->{NEXT}}){
-+ if (@{$self->{NEXT}}){
++ if (defined $self->{NEXT}){
@{$self->{PREV}} = (@{$self->{NEXT}});
}else{
@{$self->{PREV}} = ();
}
}else{
- if (defined @{$self->{NEXT}}){
-+ if (@{$self->{NEXT}}){
++ if (defined $self->{NEXT}){
@{$self->{PREV}} = (@{$self->{NEXT}});
}else{
@{$self->{PREV}} = ();
diff --git a/debian/patches/series b/debian/patches/series
index 4baf145..0112f64 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,2 +1,2 @@
-##omit_defined.patch
+omit_defined.patch
properly_handle_distances.patch
--
create GPX-Tracks from OSM relation
More information about the Pkg-grass-devel
mailing list