sij/mapnik
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Doh, fixing stupid C++ that I wrote after properly writing a unit test

Browse source
This commit is contained in:
Blake Thompson 2015-05-15 13:21:51 -05:00
parent 90de0d65cb
commit 3e24c5a417
3 changed files with 77 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

49
src/proj_transform.cpp
View file

@ -264,10 +264,14 @@ bool proj_transform::forward (box2d<double> & box) const
if (is_source_equal_dest_)
return true;
double llx, ulx = box.minx();
double lly, lry = box.miny();
double lrx, urx = box.maxx();
double uly, ury = box.maxy();
double llx = box.minx();
double ulx = box.minx();
double lly = box.miny();
double lry = box.miny();
double lrx = box.maxx();
double urx = box.maxx();
double uly = box.maxy();
double ury = box.maxy();
double z = 0.0;
if (!forward(llx,lly,z))
return false;
@ -277,10 +281,15 @@ bool proj_transform::forward (box2d<double> & box) const
return false;
if (!forward(urx,ury,z))
return false;
box.init(std::min(ulx, llx),
std::min(lly, lry),
std::max(urx, lrx),
std::max(uly, ury));
double minx = std::min(ulx, llx);
double miny = std::min(lly, lry);
double maxx = std::max(urx, lrx);
double maxy = std::max(ury, uly);
box.init(minx,
miny,
maxx,
maxy);
return true;
}
@ -289,10 +298,14 @@ bool proj_transform::backward (box2d<double> & box) const
if (is_source_equal_dest_)
return true;
double llx, ulx = box.minx();
double lly, lry = box.miny();
double lrx, urx = box.maxx();
double uly, ury = box.maxy();
double llx = box.minx();
double ulx = box.minx();
double lly = box.miny();
double lry = box.miny();
double lrx = box.maxx();
double urx = box.maxx();
double uly = box.maxy();
double ury = box.maxy();
double z = 0.0;
if (!backward(llx,lly,z))
return false;
@ -302,10 +315,14 @@ bool proj_transform::backward (box2d<double> & box) const
return false;
if (!backward(urx,ury,z))
return false;
box.init(std::min(ulx, llx),
std::min(lly, lry),
std::max(urx, lrx),
std::max(uly, ury));
double minx = std::min(ulx, llx);
double miny = std::min(lly, lry);
double maxx = std::max(urx, lrx);
double maxy = std::max(ury, uly);
box.init(minx,
miny,
maxx,
maxy);
return true;
}

7
test/unit/imaging/image_multiply_alpha.cpp
View file

@ -58,10 +58,15 @@ SECTION("test rgba8 overflow") {
mapnik::image_rgba8 im(4,4);
mapnik::color c(128,128,128,128); // This color is premultiplied
mapnik::fill(im, c); // Because c1 is not premultiplied it will make the image not premultiplied
mapnik::color out;
out = mapnik::get_pixel<mapnik::color>(im, 0, 0);
CHECK(static_cast<int>(out.red()) == 128);
CHECK(static_cast<int>(out.green()) == 128);
CHECK(static_cast<int>(out.blue()) == 128);
CHECK(static_cast<int>(out.alpha()) == 128);
mapnik::multiply_alpha(im, 2.5);
mapnik::color out;
out = mapnik::get_pixel<mapnik::color>(im, 0, 0);
CHECK(static_cast<int>(out.red()) == 128);
CHECK(static_cast<int>(out.green()) == 128);

38
test/unit/projection/proj_transform.cpp Normal file
View file

@ -0,0 +1,38 @@
#include "catch.hpp"
#include <mapnik/projection.hpp>
#include <mapnik/proj_transform.hpp>
#include <mapnik/box2d.hpp>
TEST_CASE("projection transform")
{
SECTION("Test bounding box transforms - 4326 to 3857")
{
mapnik::projection proj_4326("+init=epsg:4326");
mapnik::projection proj_3857("+init=epsg:3857");
mapnik::proj_transform prj_trans(proj_4326, proj_3857);
double minx = -45.0;
double miny = 55.0;
double maxx = -40.0;
double maxy = 75.0;
mapnik::box2d<double> bbox(minx, miny, maxx, maxy);
prj_trans.forward(bbox);
INFO(bbox.to_string());
CHECK(bbox.minx() == Approx(-5009377.085697311));
CHECK(bbox.miny() == Approx(7361866.1130511891));
CHECK(bbox.maxx() == Approx(-4452779.631730943));
CHECK(bbox.maxy() == Approx(12932243.1119920239));
prj_trans.backward(bbox);
CHECK(bbox.minx() == Approx(minx));
CHECK(bbox.miny() == Approx(miny));
CHECK(bbox.maxx() == Approx(maxx));
CHECK(bbox.maxy() == Approx(maxy));
} // END SECTION
} // END TEST CASE