[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