Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

2.12 build fails on 32-bit arch (testqgsdistancearea.cpp) #21757

Closed
qgib opened this issue Nov 2, 2015 · 2 comments
Closed

2.12 build fails on 32-bit arch (testqgsdistancearea.cpp) #21757

qgib opened this issue Nov 2, 2015 · 2 comments
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter!
Milestone

Comments

@qgib
Copy link
Contributor

qgib commented Nov 2, 2015

Author Name: sthen - (sthen -)
Original Redmine Issue: 13730
Affected QGIS version: 2.12.0


--- tests/src/core/testqgsdistancearea.cpp.orig	Sun Nov  1 13:45:40 2015
+++ tests/src/core/testqgsdistancearea.cpp	Sun Nov  1 13:46:13 2015
@@ -204,19 +204,19 @@ void TestQgsDistanceArea::collections()
   //collection of polygons
   QgsGeometry polys( QgsGeometryFactory::geomFromWkt( "GeometryCollection( Polygon((0 36.53, 5.76 -48.16, 0 25.54, 0 36.53)), Polygon((10 20, 15 20, 15 10, 10 20)) )" ) );
   result = myDa.measure( &polys ); //should meaure area
-  QVERIFY( qgsDoubleNear( result, 670434859475, 1 ) );
+  QVERIFY( qgsDoubleNear( result, 670434859475LL, 1 ) );
   result = myDa.measureArea( &polys );
-  QVERIFY( qgsDoubleNear( result, 670434859475, 1 ) );
+  QVERIFY( qgsDoubleNear( result, 670434859475LL, 1 ) );
   result = myDa.measureLength( &polys );
   QVERIFY( qgsDoubleNear( result, 0 ) );
 
   //mixed collection
   QgsGeometry mixed( QgsGeometryFactory::geomFromWkt( "GeometryCollection( LineString(0 36.53, 5.76 -48.16), LineString(0 25.54, 24.20 36.70), Polygon((0 36.53, 5.76 -48.16, 0 25.54, 0 36.53)), Polygon((10 20, 15 20, 15 10, 10 20)) )" ) );
   result = myDa.measure( &mixed ); //should measure area
-  QVERIFY( qgsDoubleNear( result, 670434859475, 1 ) );
+  QVERIFY( qgsDoubleNear( result, 670434859475LL, 1 ) );
   //measure area specifically
   result = myDa.measureArea( &mixed );
-  QVERIFY( qgsDoubleNear( result, 670434859475, 1 ) );
+  QVERIFY( qgsDoubleNear( result, 670434859475LL, 1 ) );
   //measure length
   result = myDa.measureLength( &mixed );
   QVERIFY( qgsDoubleNear( result, 12006159, 1 ) );

@qgib
Copy link
Contributor Author

qgib commented Nov 2, 2015

Author Name: landry Landry Breuil (@landryb)


#2413

@qgib
Copy link
Contributor Author

qgib commented Nov 2, 2015

Author Name: landry Landry Breuil (@landryb)


PR merged in 03eaafe


  • done_ratio was changed from 0 to 100
  • fixed_version_id was configured as Version 2.14
  • status_id was changed from Open to Closed

@qgib qgib added the Bug Either a bug report, or a bug fix. Let's hope for the latter! label May 25, 2019
@qgib qgib added this to the Version 2.14 milestone May 25, 2019
@qgib qgib closed this as completed May 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter!
Projects
None yet
Development

No branches or pull requests

1 participant