aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorEven Rouault <even.rouault@spatialys.com>2021-11-15 18:55:52 +0100
committerEven Rouault <even.rouault@spatialys.com>2021-11-15 18:55:52 +0100
commitdff8131cc1bc27c271c54b0234c1fa3fff8e3062 (patch)
treebc8ae8bde3f9296c347628c726ad72d1fa685b4a
parent48f018f0032f4769e1ab0f09e930f5ac9040b4c9 (diff)
downloadPROJ-dff8131cc1bc27c271c54b0234c1fa3fff8e3062.tar.gz
PROJ-dff8131cc1bc27c271c54b0234c1fa3fff8e3062.zip
Code reformat
-rw-r--r--test/unit/gie_self_tests.cpp95
-rw-r--r--test/unit/test_c_api.cpp4
2 files changed, 54 insertions, 45 deletions
diff --git a/test/unit/gie_self_tests.cpp b/test/unit/gie_self_tests.cpp
index 23684e68..71df8fe1 100644
--- a/test/unit/gie_self_tests.cpp
+++ b/test/unit/gie_self_tests.cpp
@@ -892,7 +892,7 @@ TEST(gie, proj_trans_generic) {
TEST(gie, proj_create_crs_to_crs_from_pj_force_over) {
- PJ_CONTEXT* ctx;
+ PJ_CONTEXT *ctx;
ctx = proj_context_create();
ASSERT_TRUE(ctx != nullptr);
@@ -902,26 +902,27 @@ TEST(gie, proj_create_crs_to_crs_from_pj_force_over) {
auto epsg4326 = proj_create(ctx, "EPSG:4326");
ASSERT_TRUE(epsg4326 != nullptr);
-
+
auto epsg3857 = proj_create(ctx, "EPSG:3857");
ASSERT_TRUE(epsg3857 != nullptr);
{
- const char* const options[] = { "FORCE_OVER=YES", nullptr };
- auto P = proj_create_crs_to_crs_from_pj(ctx, epsg4326, epsg3857, nullptr, options);
+ const char *const options[] = {"FORCE_OVER=YES", nullptr};
+ auto P = proj_create_crs_to_crs_from_pj(ctx, epsg4326, epsg3857,
+ nullptr, options);
ASSERT_TRUE(P != nullptr);
ASSERT_TRUE(P->over);
PJ_COORD input;
PJ_COORD input_over;
- //Test a point along the equator.
- //The same point, but in two different representations.
- input.xyz.x = 0; // Lat in deg
- input.xyz.y = 140; // Long in deg
+ // Test a point along the equator.
+ // The same point, but in two different representations.
+ input.xyz.x = 0; // Lat in deg
+ input.xyz.y = 140; // Long in deg
input.xyz.z = 0;
- input_over.xyz.x = 0; // Lat in deg
- input_over.xyz.y = -220; // Long in deg
+ input_over.xyz.x = 0; // Lat in deg
+ input_over.xyz.y = -220; // Long in deg
input_over.xyz.z = 0;
auto output = proj_trans(P, PJ_FWD, input);
@@ -930,22 +931,24 @@ TEST(gie, proj_create_crs_to_crs_from_pj_force_over) {
auto input_inv = proj_trans(P, PJ_INV, output);
auto input_over_inv = proj_trans(P, PJ_INV, output_over);
- //Web Mercator x's between 0 and 180 longitude come out positive.
- //But when forcing the over flag, the -220 calculation makes it flip.
+ // Web Mercator x's between 0 and 180 longitude come out positive.
+ // But when forcing the over flag, the -220 calculation makes it flip.
EXPECT_GT(output.xyz.x, 0);
EXPECT_LT(output_over.xyz.x, 0);
EXPECT_NEAR(output.xyz.x, 15584728.711058298, 1e-8);
EXPECT_NEAR(output_over.xyz.x, -24490287.974520184, 1e-8);
- //The distance from 140 to 180 and -220 to -180 should be pretty much the same.
+ // The distance from 140 to 180 and -220 to -180 should be pretty much
+ // the same.
auto dx_o = fabs(output.xyz.x - 20037508.342789244);
auto dx_over = fabs(output_over.xyz.x + 20037508.342789244);
auto dx = fabs(dx_o - dx_over);
EXPECT_NEAR(dx, 0, 1e-8);
- //Check the inverse operations get us back close to our original input values.
+ // Check the inverse operations get us back close to our original input
+ // values.
EXPECT_NEAR(input.xyz.x, input_inv.xyz.x, 1e-8);
EXPECT_NEAR(input.xyz.y, input_inv.xyz.y, 1e-8);
EXPECT_NEAR(input_over.xyz.x, input_over_inv.xyz.x, 1e-8);
@@ -955,20 +958,22 @@ TEST(gie, proj_create_crs_to_crs_from_pj_force_over) {
}
{
- //Try again with force over set to anything but YES to verify it didn't do anything.
- const char* const options[] = { "FORCE_OVER=NO", nullptr };
- auto P = proj_create_crs_to_crs_from_pj(ctx, epsg4326, epsg3857, nullptr, options);
+ // Try again with force over set to anything but YES to verify it didn't
+ // do anything.
+ const char *const options[] = {"FORCE_OVER=NO", nullptr};
+ auto P = proj_create_crs_to_crs_from_pj(ctx, epsg4326, epsg3857,
+ nullptr, options);
ASSERT_TRUE(P != nullptr);
ASSERT_FALSE(P->over);
PJ_COORD input;
PJ_COORD input_notOver;
- input.xyz.x = 0; // Lat in deg
- input.xyz.y = 140; // Long in deg
+ input.xyz.x = 0; // Lat in deg
+ input.xyz.y = 140; // Long in deg
input.xyz.z = 0;
- input_notOver.xyz.x = 0; // Lat in deg
- input_notOver.xyz.y = -220; // Long in deg
+ input_notOver.xyz.x = 0; // Lat in deg
+ input_notOver.xyz.y = -220; // Long in deg
input_notOver.xyz.z = 0;
auto output = proj_trans(P, PJ_FWD, input);
@@ -984,19 +989,20 @@ TEST(gie, proj_create_crs_to_crs_from_pj_force_over) {
}
{
- //Try again with no options to verify it didn't do anything.
- auto P = proj_create_crs_to_crs_from_pj(ctx, epsg4326, epsg3857, nullptr, nullptr);
+ // Try again with no options to verify it didn't do anything.
+ auto P = proj_create_crs_to_crs_from_pj(ctx, epsg4326, epsg3857,
+ nullptr, nullptr);
ASSERT_TRUE(P != nullptr);
ASSERT_FALSE(P->over);
PJ_COORD input;
PJ_COORD input_notOver;
- input.xyz.x = 0; // Lat in deg
- input.xyz.y = 140; // Long in deg
+ input.xyz.x = 0; // Lat in deg
+ input.xyz.y = 140; // Long in deg
input.xyz.z = 0;
- input_notOver.xyz.x = 0; // Lat in deg
- input_notOver.xyz.y = -220; // Long in deg
+ input_notOver.xyz.x = 0; // Lat in deg
+ input_notOver.xyz.y = -220; // Long in deg
input_notOver.xyz.z = 0;
auto output = proj_trans(P, PJ_FWD, input);
@@ -1012,49 +1018,52 @@ TEST(gie, proj_create_crs_to_crs_from_pj_force_over) {
}
{
- //EPSG:4326 -> EPSG:27700 has more than one coordinate operation candidate.
- const char* const options[] = { "FORCE_OVER=YES", nullptr };
- auto P = proj_create_crs_to_crs_from_pj(ctx, epsg4326, epsg27700, nullptr, options);
+ // EPSG:4326 -> EPSG:27700 has more than one coordinate operation
+ // candidate.
+ const char *const options[] = {"FORCE_OVER=YES", nullptr};
+ auto P = proj_create_crs_to_crs_from_pj(ctx, epsg4326, epsg27700,
+ nullptr, options);
ASSERT_TRUE(P != nullptr);
ASSERT_TRUE(P->over);
PJ_COORD input;
PJ_COORD input_over;
- input.xyz.x = 0; // Lat in deg
- input.xyz.y = 140; // Long in deg
+ input.xyz.x = 0; // Lat in deg
+ input.xyz.y = 140; // Long in deg
input.xyz.z = 0;
- input_over.xyz.x = 0; // Lat in deg
- input_over.xyz.y = -220; // Long in deg
+ input_over.xyz.x = 0; // Lat in deg
+ input_over.xyz.y = -220; // Long in deg
input_over.xyz.z = 0;
auto output = proj_trans(P, PJ_FWD, input);
auto output_over = proj_trans(P, PJ_FWD, input_over);
- //Doesn't actually change the result for this tmerc transformation.
+ // Doesn't actually change the result for this tmerc transformation.
EXPECT_NEAR(output.xyz.x, 4980122.749364435, 1e-8);
EXPECT_NEAR(output.xyz.y, 14467212.882603768, 1e-8);
EXPECT_NEAR(output_over.xyz.x, 4980122.749364435, 1e-8);
EXPECT_NEAR(output_over.xyz.y, 14467212.882603768, 1e-8);
-
+
proj_destroy(P);
}
{
- //Negative test for 27700.
- const char* const options[] = { "FORCE_OVER=NO", nullptr };
- auto P = proj_create_crs_to_crs_from_pj(ctx, epsg4326, epsg27700, nullptr, options);
+ // Negative test for 27700.
+ const char *const options[] = {"FORCE_OVER=NO", nullptr};
+ auto P = proj_create_crs_to_crs_from_pj(ctx, epsg4326, epsg27700,
+ nullptr, options);
ASSERT_TRUE(P != nullptr);
ASSERT_FALSE(P->over);
PJ_COORD input;
PJ_COORD input_over;
- input.xyz.x = 0; // Lat in deg
- input.xyz.y = 140; // Long in deg
+ input.xyz.x = 0; // Lat in deg
+ input.xyz.y = 140; // Long in deg
input.xyz.z = 0;
- input_over.xyz.x = 0; // Lat in deg
- input_over.xyz.y = -220; // Long in deg
+ input_over.xyz.x = 0; // Lat in deg
+ input_over.xyz.y = -220; // Long in deg
input_over.xyz.z = 0;
auto output = proj_trans(P, PJ_FWD, input);
diff --git a/test/unit/test_c_api.cpp b/test/unit/test_c_api.cpp
index 3548293d..291d40bc 100644
--- a/test/unit/test_c_api.cpp
+++ b/test/unit/test_c_api.cpp
@@ -6116,8 +6116,8 @@ TEST_F(CApi, open_plenty_of_contexts) {
// On some platforms fopen returned nullptrs before reaching limit - 50, we
// can avoid this by capping the limit to 1024.
if (open_max.rlim_cur > 1024) {
- open_max.rlim_cur = 1024;
- setrlimit(RLIMIT_NOFILE, &open_max);
+ open_max.rlim_cur = 1024;
+ setrlimit(RLIMIT_NOFILE, &open_max);
}
for (rlim_t i = 0; i < open_max.rlim_cur - 50; i++) {
FILE *f = fopen("/dev/null", "rb");