[pdal-commits] [PDAL/PDAL] 9927a4: Modified not to assume std namespace. (#3023)

cgodkin noreply at github.com
Mon Apr 13 06:12:11 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/PDAL/PDAL
  Commit: 9927a4b6235e286f87db4c59d52b4a3bdb9a6cc3
      https://github.com/PDAL/PDAL/commit/9927a4b6235e286f87db4c59d52b4a3bdb9a6cc3
  Author: cgodkin <carl at dgi.com>
  Date:   2020-04-13 (Mon, 13 Apr 2020)

  Changed paths:
    M tools/lasdump/Dumper.cpp

  Log Message:
  -----------
  Modified not to assume std namespace. (#3023)

* Fixed hiding of local variable (principally the warning)

* Proposed fix to Issue #3008

Here's a simple fix to the issue I just opened.  

Hope these trivial fixes aren't too annoying but we always compile on Windows with -W4 so I'm going to find things that are technically incorrect but don't affect behavior.  Thanks.

* Update Dumper.cpp not to assume std namespace

If a fix to https://github.com/LASzip/LASzip/issues/69 is applied, then Dumper.cpp won't compile since it inherits "using namespace std" from laszip_api.h.  This commit adds "std::" in three places to avoid the dependency.  (Alternatively, "using namespace std" could be added to the top of this source file.)

* Revert proposed change

https://github.com/PDAL/PDAL/issues/3008




More information about the pdal-commits mailing list