[pdal-commits] [PDAL/PDAL] b6474d: Doc fix for Delaunay filter (#1936)

GitHub noreply at github.com
Mon Apr 23 13:00:50 PDT 2018 

  Branch: refs/heads/clang6
  Home:   https://github.com/PDAL/PDAL
  Commit: b6474df9cc58599509ec93e6d97d1ad4b6284541
      https://github.com/PDAL/PDAL/commit/b6474df9cc58599509ec93e6d97d1ad4b6284541
  Author: Peter Limkilde Svendsen <34237736+plimkilde at users.noreply.github.com>
  Date:   2018-04-17 (Tue, 17 Apr 2018)

  Changed paths:
    M doc/stages/filters.delaunay.rst

  Log Message:
  -----------
  Doc fix for Delaunay filter (#1936)


  Commit: e7ed147dec8ffa2a11ec1214b4376c42405007df
      https://github.com/PDAL/PDAL/commit/e7ed147dec8ffa2a11ec1214b4376c42405007df
  Author: Even Rouault <even.rouault at mines-paris.org>
  Date:   2018-04-21 (Sat, 21 Apr 2018)

  Changed paths:
    M io/LasWriter.hpp

  Log Message:
  -----------
  Initialize LasWriter::m_forwardVlrs (#1948)

Fixes the following Valgrind warning on FerryFilterTest

==17720== Conditional jump or move depends on uninitialised value(s)
==17720==    at 0x56E9B3A: pdal::LasWriter::addForwardVlrs() (LasWriter.cpp:441)
==17720==    by 0x56E85B2: pdal::LasWriter::readyTable(pdal::BasePointTable&) (LasWriter.cpp:307)
==17720==    by 0x56A727B: pdal::FlexWriter::ready(pdal::BasePointTable&) (FlexWriter.hpp:89)
==17720==    by 0x577046F: pdal::Stage::execute(pdal::BasePointTable&) (Stage.cpp:209)
==17720==    by 0x5782430: pdal::PipelineManager::execute() (PipelineManager.cpp:207)
==17720==    by 0x457957: FerryFilterTest_test_ferry_copy_json_Test::TestBody() (FerryFilterTest.cpp:104)
==17720==    by 0x493CE7: void testing::internal::HandleSehExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::*)(), char const*) (gtest.cc:2078)
==17720==    by 0x48EC8A: void testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::*)(), char const*) (gtest.cc:2114)
==17720==    by 0x4744B3: testing::Test::Run() (gtest.cc:2151)
==17720==    by 0x474D5B: testing::TestInfo::Run() (gtest.cc:2326)
==17720==    by 0x47544A: testing::TestCase::Run() (gtest.cc:2444)
==17720==    by 0x47C527: testing::internal::UnitTestImpl::RunAllTests() (gtest.cc:4315)
==17720==


  Commit: 2bfe763a96341f1e049dfe279a7051088793f612
      https://github.com/PDAL/PDAL/commit/2bfe763a96341f1e049dfe279a7051088793f612
  Author: Even Rouault <even.rouault at mines-paris.org>
  Date:   2018-04-21 (Sat, 21 Apr 2018)

  Changed paths:
    M plugins/matlab/filters/Script.cpp
    M tools/lasdump/Vlr.hpp

  Log Message:
  -----------
  Fix same expression warnings (#1952)

* plugins/matlab/filters/Script.cpp: mxDOUBLE_CLASS and mxINT32_CLASS are compared twice

* Vlr::matches(): compare userId to m_userId instead of self


  Commit: 88dc6db3a50fce7685f468bef4665433765f4469
      https://github.com/PDAL/PDAL/commit/88dc6db3a50fce7685f468bef4665433765f4469
  Author: Even Rouault <even.rouault at mines-paris.org>
  Date:   2018-04-21 (Sat, 21 Apr 2018)

  Changed paths:
    M io/GeotiffSupport.cpp

  Log Message:
  -----------
  GeotiffSrs constructor: fix memleak of GTIFGetOGISDefn() return (#1949)

This fixes the following Valgrind warning on pdal_filters_reprojection_test

==22366== 1,282 bytes in 2 blocks are definitely lost in loss record 675 of 701
==22366==    at 0x4C2DB8F: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==22366==    by 0x7DE5D24: VSIMalloc (cpl_vsisimple.cpp:557)
==22366==    by 0x7D136F5: CPLMalloc (cpl_conv.cpp:184)
==22366==    by 0x8737AD8: OGR_SRSNode::exportToWkt(char**) const (ogr_srsnode.cpp:466)
==22366==    by 0x8724742: OGRSpatialReference::exportToWkt(char**) const (ogrspatialreference.cpp:737)
==22366==    by 0x75C8C4B: GTIFGetOGISDefn (gt_wkt_srs.cpp:1356)
==22366==    by 0x5696FA9: pdal::GeotiffSrs::GeotiffSrs(std::vector<unsigned char, std::allocator<unsigned char> > const&, std::vector<unsigned char, std::allocator<unsigned char> > const&, std::vector<unsigned char, std::allocator<unsigned char> > const&, std::shared_ptr<pdal::Log>) (GeotiffSupport.cpp:149)
==22366==    by 0x56B3C3B: pdal::LasHeader::setSrsFromGeotiff() (LasHeader.cpp:322)
==22366==    by 0x56B3243: pdal::LasHeader::setSrs() (LasHeader.cpp:219)
==22366==    by 0x56B468F: pdal::operator>>(pdal::ILeStream&, pdal::LasHeader&) (LasHeader.cpp:404)
==22366==    by 0x5680FF4: pdal::LasReader::initializeLocal(pdal::BasePointTable&, pdal::MetadataNode&) (LasReader.cpp:192)
==22366==    by 0x568972E: pdal::LasReader::initialize(pdal::BasePointTable&) (LasReader.hpp:134)


  Commit: 9eb8c5a5a0cf7f76497c2e818072f5fca057ce05
      https://github.com/PDAL/PDAL/commit/9eb8c5a5a0cf7f76497c2e818072f5fca057ce05
  Author: Even Rouault <even.rouault at mines-paris.org>
  Date:   2018-04-21 (Sat, 21 Apr 2018)

  Changed paths:
    M test/unit/filters/FerryFilterTest.cpp
    M test/unit/filters/ReprojectionFilterTest.cpp

  Log Message:
  -----------
  Tests: fix for GDAL with proj.5 (#1947)

There are slight different of behavioru in proj.5 vs proj.4
In particular reprojection from +ellps=GRS80 +towgs84=0,0,0 to
+datum=WGS84 now involves a helmert transform in proj.5
instead of being considered as a no-op in proj.4


  Commit: 9ea259c2b04e1b26361ecbac3b6d5200983afcd1
      https://github.com/PDAL/PDAL/commit/9ea259c2b04e1b26361ecbac3b6d5200983afcd1
  Author: Even Rouault <even.rouault at mines-paris.org>
  Date:   2018-04-21 (Sat, 21 Apr 2018)

  Changed paths:
    M pdal/XMLSchema.cpp

  Log Message:
  -----------
  XMLSchema::load(): fix cppcheck warning 'Statements following return, break, continue, goto or throw will never be executed' (#1953)


  Commit: fff3b84a67a90bd7ceba3086f62a4e2a02c86093
      https://github.com/PDAL/PDAL/commit/fff3b84a67a90bd7ceba3086f62a4e2a02c86093
  Author: Andrew Bell <andrew.bell.ia at gmail.com>
  Date:   2018-04-23 (Mon, 23 Apr 2018)

  Changed paths:
    M doc/stages/filters.delaunay.rst
    M io/GeotiffSupport.cpp
    M io/LasWriter.hpp
    M pdal/XMLSchema.cpp
    M plugins/matlab/filters/Script.cpp
    M test/unit/filters/FerryFilterTest.cpp
    M test/unit/filters/ReprojectionFilterTest.cpp
    M tools/lasdump/Vlr.hpp

  Log Message:
  -----------
  Merge remote-tracking branch 'origin/master'


  Commit: b0b12fb07faddba67e7d5d2b31e4f21808a6c56a
      https://github.com/PDAL/PDAL/commit/b0b12fb07faddba67e7d5d2b31e4f21808a6c56a
  Author: Andrew Bell <andrew.bell.ia at gmail.com>
  Date:   2018-04-23 (Mon, 23 Apr 2018)

  Changed paths:
    M io/GeotiffSupport.cpp

  Log Message:
  -----------
  Remove header that can bring in duplicate macros.


  Commit: 98035ccc3778957332fa363cbdb086d2508985ce
      https://github.com/PDAL/PDAL/commit/98035ccc3778957332fa363cbdb086d2508985ce
  Author: Andrew Bell <andrew.bell.ia at gmail.com>
  Date:   2018-04-23 (Mon, 23 Apr 2018)

  Changed paths:
    M filters/ReprojectionFilter.cpp
    M filters/ReprojectionFilter.hpp

  Log Message:
  -----------
  Remove misleading unused variable.


  Commit: 18c02a77a9d889904f1cb0f78db0e2c7a627fcb3
      https://github.com/PDAL/PDAL/commit/18c02a77a9d889904f1cb0f78db0e2c7a627fcb3
  Author: Andrew Bell <andrew.bell.ia at gmail.com>
  Date:   2018-04-23 (Mon, 23 Apr 2018)

  Changed paths:
    M doc/stages/filters.delaunay.rst
    M filters/ReprojectionFilter.cpp
    M filters/ReprojectionFilter.hpp
    M io/GeotiffSupport.cpp
    M io/LasWriter.hpp
    M pdal/XMLSchema.cpp
    M plugins/matlab/filters/Script.cpp
    M test/unit/filters/FerryFilterTest.cpp
    M test/unit/filters/ReprojectionFilterTest.cpp
    M tools/lasdump/Vlr.hpp

  Log Message:
  -----------
  Merge remote-tracking branch 'origin/master' into clang6


Compare: https://github.com/PDAL/PDAL/compare/0196b6c366e2...18c02a77a9d8

More information about the pdal-commits mailing list