ashatora/openlayers
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

remove unneeded parseFloat when building Geometry.Point

Browse Source
This commit is contained in:
Frederic Junod
2011-10-06 14:32:38 +02:00
parent 00e2e1bd59
commit 54e5db90ae
3 changed files with 12 additions and 31 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lib/OpenLayers/Format/ArcXML.js
View File

@@ -885,7 +885,7 @@ OpenLayers.Format.ArcXML = OpenLayers.Class(OpenLayers.Format.XML, {
coordArr = coordArr.split(/;/);
for (var cn = 0; cn < coordArr.length; cn++) {
var coordItems = coordArr[cn].split(/ /);
ringPoints.push(new OpenLayers.Geometry.Point(parseFloat(coordItems[0]), parseFloat(coordItems[1])));
ringPoints.push(new OpenLayers.Geometry.Point(coordItems[0], coordItems[1]));
}
coords = null;
} else {

17
lib/OpenLayers/Format/Atom.js
View File

@@ -607,12 +607,7 @@ OpenLayers.Format.Atom = OpenLayers.Class(OpenLayers.Format.XML, {
point[i].firstChild.nodeValue
).split(/\s*,\s*/);
}
components.push(
new OpenLayers.Geometry.Point(
parseFloat(xy[1]),
parseFloat(xy[0])
)
);
components.push(new OpenLayers.Geometry.Point(xy[1], xy[0]));
}
}
@@ -627,10 +622,7 @@ OpenLayers.Format.Atom = OpenLayers.Class(OpenLayers.Format.XML, {
).split(/\s+/);
points = [];
for (var j=0, jj=coords.length; j<jj; j+=2) {
p = new OpenLayers.Geometry.Point(
parseFloat(coords[j+1]),
parseFloat(coords[j])
);
p = new OpenLayers.Geometry.Point(coords[j+1], coords[j]);
points.push(p);
}
components.push(
@@ -650,10 +642,7 @@ OpenLayers.Format.Atom = OpenLayers.Class(OpenLayers.Format.XML, {
).split(/\s+/);
points = [];
for (var j=0, jj=coords.length; j<jj; j+=2) {
p = new OpenLayers.Geometry.Point(
parseFloat(coords[j+1]),
parseFloat(coords[j])
);
p = new OpenLayers.Geometry.Point(coords[j+1], coords[j]);
points.push(p);
}
components.push(

24
lib/OpenLayers/Format/GeoRSS.js
View File

@@ -126,16 +126,14 @@ OpenLayers.Format.GeoRSS = OpenLayers.Class(OpenLayers.Format.XML, {
parseFloat(lon[0].firstChild.nodeValue)];
}
var geometry = new OpenLayers.Geometry.Point(parseFloat(location[1]),
parseFloat(location[0]));
var geometry = new OpenLayers.Geometry.Point(location[1], location[0]);
} else if (line.length > 0) {
var coords = OpenLayers.String.trim(this.concatChildValues(line[0])).split(/\s+/);
var components = [];
var point;
for (var i=0, len=coords.length; i<len; i+=2) {
point = new OpenLayers.Geometry.Point(parseFloat(coords[i+1]),
parseFloat(coords[i]));
point = new OpenLayers.Geometry.Point(coords[i+1], coords[i]);
components.push(point);
}
geometry = new OpenLayers.Geometry.LineString(components);
@@ -144,8 +142,7 @@ OpenLayers.Format.GeoRSS = OpenLayers.Class(OpenLayers.Format.XML, {
var components = [];
var point;
for (var i=0, len=coords.length; i<len; i+=2) {
point = new OpenLayers.Geometry.Point(parseFloat(coords[i+1]),
parseFloat(coords[i]));
point = new OpenLayers.Geometry.Point(coords[i+1], coords[i]);
components.push(point);
}
geometry = new OpenLayers.Geometry.Polygon([new OpenLayers.Geometry.LinearRing(components)]);
@@ -160,20 +157,15 @@ OpenLayers.Format.GeoRSS = OpenLayers.Class(OpenLayers.Format.XML, {
var components = [];
var point;
if (coords.length > 3) {
point = new OpenLayers.Geometry.Point(parseFloat(coords[1]),
parseFloat(coords[0]));
point = new OpenLayers.Geometry.Point(coords[1], coords[0]);
components.push(point);
point = new OpenLayers.Geometry.Point(parseFloat(coords[1]),
parseFloat(coords[2]));
point = new OpenLayers.Geometry.Point(coords[1], coords[2]);
components.push(point);
point = new OpenLayers.Geometry.Point(parseFloat(coords[3]),
parseFloat(coords[2]));
point = new OpenLayers.Geometry.Point(coords[3], coords[2]);
components.push(point);
point = new OpenLayers.Geometry.Point(parseFloat(coords[3]),
parseFloat(coords[0]));
point = new OpenLayers.Geometry.Point(coords[3], coords[0]);
components.push(point);
point = new OpenLayers.Geometry.Point(parseFloat(coords[1]),
parseFloat(coords[0]));
point = new OpenLayers.Geometry.Point(coords[1], coords[0]);
components.push(point);
}
geometry = new OpenLayers.Geometry.Polygon([new OpenLayers.Geometry.LinearRing(components)]);