[pdal-commits] [PDAL/PDAL] 2969f8: Add sort order option, with new descending sort #1...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/PDAL/PDAL
  Commit: 2969f8931fd4ceb626978310ccbb859ea959194d
      https://github.com/PDAL/PDAL/commit/2969f8931fd4ceb626978310ccbb859ea959194d
  Author: Weston Renoud <wrenoud at gmail.com>
  Date:   2017-03-27 (Mon, 27 Mar 2017)

  Changed paths:
    M doc/stages/filters.sort.rst
    M filters/SortFilter.cpp
    M filters/SortFilter.hpp
    M pdal/PointViewIter.hpp
    M test/unit/filters/SortFilterTest.cpp

  Log Message:
  -----------
  Add sort order option, with new descending sort #1121 (#1530)

* Add sort order Option, with new descending (largest to smallest) sort #1121

Default behavior remains unchanged.
This includes unittests and documentation updates.

* reformat sort filter to follow style guide

* undue change to sort filter test interations

I'm unsure if there is a reason for the number of iterations.

* better handling of unknown options for sort

per suggestion from @abellgithub

* wrong out text for SortOrder enum

* switch sort filter to std::stable_sort

needed a default constructor for pdal::PointViewIter

* update sort filter description in documentation

* bug exposed when trying to use std::stable_sort with the SortFilter