[pdal-commits] [PDAL/PDAL] 3f1af0: Load only one plugin when guessing plugin names

GitHub noreply at github.com
Fri May 8 08:43:05 PDT 2015


  Branch: refs/heads/master
  Home:   https://github.com/PDAL/PDAL
  Commit: 3f1af0f4350acce47d02422e84d78ca3d569a38d
      https://github.com/PDAL/PDAL/commit/3f1af0f4350acce47d02422e84d78ca3d569a38d
  Author: Pete Gadomski <pete.gadomski at gmail.com>
  Date:   2015-04-26 (Sun, 26 Apr 2015)

  Changed paths:
    M src/PluginManager.cpp

  Log Message:
  -----------
  Load only one plugin when guessing plugin names

In `PluginManager::guessLoadByPath`, the driverName was not being
entirely used when loading plugins. Only the plugin type (filter,
reader, writer) was used. This patch checks the actual name of the
driver (e.g. hexbin) against the filename of the library, and only loads
the plugin if the name matches.

Since a plugin only needs to be loaded once, the return value of the
`loadByPath` call is returned from `guessLoadByPath`. If no plugins are
found, the `guessLoadByPath` now returns -1.


  Commit: 6e210d468195e27c0beae0a7b60489105e0c0a2c
      https://github.com/PDAL/PDAL/commit/6e210d468195e27c0beae0a7b60489105e0c0a2c
  Author: Connor Manning <connor at hobu.co>
  Date:   2015-04-26 (Sun, 26 Apr 2015)

  Changed paths:
    M include/pdal/PluginManager.hpp
    M include/pdal/pdal_macros.hpp
    M src/KernelFactory.cpp
    M src/PluginManager.cpp
    M src/StageFactory.cpp

  Log Message:
  -----------
  Plugin manager tweaks.  Avoid integer return codes and simplify a couple things.


  Commit: d639bd7e00416631da5451db5492eb2d64669221
      https://github.com/PDAL/PDAL/commit/d639bd7e00416631da5451db5492eb2d64669221
  Author: Connor Manning <connor at hobu.co>
  Date:   2015-04-26 (Sun, 26 Apr 2015)

  Changed paths:
    M src/PluginManager.cpp

  Log Message:
  -----------
  Merge remote-tracking branch 'gadomski/missing-plugin-cleanup' into pm-consistency

Conflicts:
	src/PluginManager.cpp


  Commit: 397936b8cf96dea6056ac582564fd311f6ae1180
      https://github.com/PDAL/PDAL/commit/397936b8cf96dea6056ac582564fd311f6ae1180
  Author: Howard Butler <howard at hobu.co>
  Date:   2015-05-08 (Fri, 08 May 2015)

  Changed paths:
    M src/PluginManager.cpp

  Log Message:
  -----------
  loadByPath should load DynamicLibrary if they're not already loaded


  Commit: 87499e59ff74379510145ef06af54b471978ba0b
      https://github.com/PDAL/PDAL/commit/87499e59ff74379510145ef06af54b471978ba0b
  Author: Howard Butler <howard at hobu.co>
  Date:   2015-05-08 (Fri, 08 May 2015)

  Changed paths:
    M include/pdal/PluginManager.hpp
    M include/pdal/pdal_macros.hpp
    M src/KernelFactory.cpp
    M src/PluginManager.cpp
    M src/StageFactory.cpp

  Log Message:
  -----------
  Merge pull request #887 from PDAL/pm-consistency

PluginManager consistency


Compare: https://github.com/PDAL/PDAL/compare/1f96d9459ab7...87499e59ff74


More information about the pdal-commits mailing list