[pdal-commits] [PDAL/PDAL] 34e4d8: Seek to the end of an IStream

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/PDAL/PDAL
  Commit: 34e4d82e63880b4bbe569b5c563550cf2405a180
      https://github.com/PDAL/PDAL/commit/34e4d82e63880b4bbe569b5c563550cf2405a180
  Author: Pete Gadomski <pete.gadomski at gmail.com>
  Date:   2015-03-10 (Tue, 10 Mar 2015)

  Changed paths:
    M include/pdal/util/IStream.hpp

  Log Message:
  -----------
  Seek to the end of an IStream

Adds an overload on IStream::seek that allows seeking from position.


  Commit: 7052f43253eb0446a98f75cfcb8d935fbcb4001f
      https://github.com/PDAL/PDAL/commit/7052f43253eb0446a98f75cfcb8d935fbcb4001f
  Author: Pete Gadomski <pete.gadomski at gmail.com>
  Date:   2015-03-10 (Tue, 10 Mar 2015)

  Changed paths:
    M include/pdal/util/Extractor.hpp
    M include/pdal/util/IStream.hpp

  Log Message:
  -----------
  Add Switchable Extractor and IStream

These switchable variants can be used in the QFit reader, which has to
determine the endianness of its source data at runtime. I could have
done this with virtuals, but I thought I'd make it explicit and not
sully up the existing ILeStream and LeExtractor interface.


  Commit: cf383ad9fdd6fa5f809d6b33e54fc391d376a25e
      https://github.com/PDAL/PDAL/commit/cf383ad9fdd6fa5f809d6b33e54fc391d376a25e
  Author: Pete Gadomski <pete.gadomski at gmail.com>
  Date:   2015-03-10 (Tue, 10 Mar 2015)

  Changed paths:
    M io/qfit/QfitReader.cpp
    M io/qfit/QfitReader.hpp

  Log Message:
  -----------
  Refactor the QFit reader to not use Utils::read_n

Instead, we use the new SwitchableExtractor and ISwitchableStream.
One step along to road for #783.


  Commit: 1839bb272a9b726e7e605d1fc981def494b9e11a
      https://github.com/PDAL/PDAL/commit/1839bb272a9b726e7e605d1fc981def494b9e11a
  Author: Pete Gadomski <pete.gadomski at gmail.com>
  Date:   2015-03-10 (Tue, 10 Mar 2015)

  Changed paths:
    M io/terrasolid/TerrasolidReader.hpp
    M test/unit/CMakeLists.txt
    A test/unit/io/terrasolid/TerrasolidReaderTest.cpp

  Log Message:
  -----------
  Add TerrasolidReaderTest

Note that the tests fail at this point, because the TerrasolidReader is
broken.


  Commit: 98abdda7f50a9b87871681c6a4671193a3cf99d3
      https://github.com/PDAL/PDAL/commit/98abdda7f50a9b87871681c6a4671193a3cf99d3
  Author: Pete Gadomski <pete.gadomski at gmail.com>
  Date:   2015-03-10 (Tue, 10 Mar 2015)

  Changed paths:
    M io/terrasolid/TerrasolidReader.cpp

  Log Message:
  -----------
  Increment counter in TerrasolidReader

The reader was broken, stuck in an infinite loop.


  Commit: 3403025515332b3f6b78abb6825d812b0bc408dd
      https://github.com/PDAL/PDAL/commit/3403025515332b3f6b78abb6825d812b0bc408dd
  Author: Pete Gadomski <pete.gadomski at gmail.com>
  Date:   2015-03-10 (Tue, 10 Mar 2015)

  Changed paths:
    M io/terrasolid/TerrasolidReader.cpp

  Log Message:
  -----------
  Smarten up the Terrasolid return number

The Terrasolid spec doesn't give us real return number information, for
whatever reason, but if it is a first or only return we can
intelligently set those values. Otherwise we're SOL.


  Commit: c6890b916d94a322c925ad7b950149a0068ff694
      https://github.com/PDAL/PDAL/commit/c6890b916d94a322c925ad7b950149a0068ff694
  Author: Pete Gadomski <pete.gadomski at gmail.com>
  Date:   2015-03-10 (Tue, 10 Mar 2015)

  Changed paths:
    M io/terrasolid/TerrasolidReader.cpp
    M io/terrasolid/TerrasolidReader.hpp

  Log Message:
  -----------
  Remove read_n from Terrasolid reader

More progress on #783.


  Commit: 2cecc1dbc9d86d75f86a934e3c572fec86f4f47a
      https://github.com/PDAL/PDAL/commit/2cecc1dbc9d86d75f86a934e3c572fec86f4f47a
  Author: Pete Gadomski <pete.gadomski at gmail.com>
  Date:   2015-03-10 (Tue, 10 Mar 2015)

  Changed paths:
    M include/pdal/Utils.hpp

  Log Message:
  -----------
  Remove read_n and read_field from Utils.hpp

They have been superseded by IStream and Extractor.

Fixes #783.


  Commit: ddf20bd18a7b6f599d72a2d87e8e2086b62faad7
      https://github.com/PDAL/PDAL/commit/ddf20bd18a7b6f599d72a2d87e8e2086b62faad7
  Author: Pete Gadomski <pete.gadomski at gmail.com>
  Date:   2015-03-10 (Tue, 10 Mar 2015)

  Changed paths:
    M include/pdal/Utils.hpp
    M include/pdal/util/Extractor.hpp
    M include/pdal/util/IStream.hpp
    M io/qfit/QfitReader.cpp
    M io/qfit/QfitReader.hpp
    M io/terrasolid/TerrasolidReader.cpp
    M io/terrasolid/TerrasolidReader.hpp
    M test/unit/CMakeLists.txt
    A test/unit/io/terrasolid/TerrasolidReaderTest.cpp

  Log Message:
  -----------
  Merge pull request #791 from gadomski/issues/783-purge-read-n

Purge read_n and read_field from Utils.hpp


Compare: https://github.com/PDAL/PDAL/compare/98adff6c1d0a...ddf20bd18a7b