[pdal-commits] [PDAL/PDAL] 62d67d: Remove KernelFactory and move all kernel plugin lo...

GitHub noreply at github.com
Wed Sep 20 11:21:04 PDT 2017


  Branch: refs/heads/master
  Home:   https://github.com/PDAL/PDAL
  Commit: 62d67d0da5ffae60e6d74bb2d2f9864f7d2a4c8e
      https://github.com/PDAL/PDAL/commit/62d67d0da5ffae60e6d74bb2d2f9864f7d2a4c8e
  Author: Bradley J Chambers <brad.chambers at gmail.com>
  Date:   2017-09-20 (Wed, 20 Sep 2017)

  Changed paths:
    M apps/pdal.cpp
    M kernels/GroundKernel.cpp
    M kernels/TIndexKernel.cpp
    M kernels/TIndexKernel.hpp
    M kernels/TranslateKernel.cpp
    R pdal/KernelFactory.cpp
    R pdal/KernelFactory.hpp
    M pdal/StageFactory.cpp
    M plugins/pcl/kernel/PCLKernel.cpp
    M plugins/pcl/kernel/SmoothKernel.cpp

  Log Message:
  -----------
  Remove KernelFactory and move all kernel plugin loading to StageFactory

Long ago, KernelFactory did more than just load kernel plugins. What's left is
duplicated within StageFactory. Though not functionally any different, kernel
plugin loading can safely be moved to StageFactory to simplify code somewhat.


  Commit: 68c23a3663e2b81073011148571669ddb27fef3f
      https://github.com/PDAL/PDAL/commit/68c23a3663e2b81073011148571669ddb27fef3f
  Author: chambbj <brad.chambers at gmail.com>
  Date:   2017-09-20 (Wed, 20 Sep 2017)

  Changed paths:
    M apps/pdal.cpp
    M kernels/GroundKernel.cpp
    M kernels/TIndexKernel.cpp
    M kernels/TIndexKernel.hpp
    M kernels/TranslateKernel.cpp
    R pdal/KernelFactory.cpp
    R pdal/KernelFactory.hpp
    M pdal/StageFactory.cpp
    M plugins/pcl/kernel/PCLKernel.cpp
    M plugins/pcl/kernel/SmoothKernel.cpp

  Log Message:
  -----------
  Merge pull request #1679 from PDAL/issue/1271-plugin-loading

Remove KernelFactory and move all kernel plugin loading to StageFactory.
Closes #1271


Compare: https://github.com/PDAL/PDAL/compare/123d3a80dc78...68c23a3663e2


More information about the pdal-commits mailing list