Bug fix so search path order is preserved

[Levi-Armstrong]

When debugging PR #34 I found that std::set does not preserve order which I believe is not what we want for the search paths and library names. @marip8 do you agree?

[marip8]

I found that std::set does not preserve order which I believe is not what we want for the search paths and library names.

Generally, I think we would like to maintain the order in which libraries and paths are added (although I can't think of specific application that I'm currently working on where this is actually important)

[Levi-Armstrong]

I found that std::set does not preserve order which I believe is not what we want for the search paths and library names.

Generally, I think we would like to maintain the order in which libraries and paths are added (although I can't think of specific application that I'm currently working on where this is actually important)

I think this is problematic when it comes to the search paths. I am surprised that there is not a std object that does this.

[rjoomen]

But there's std::set and std::unordered_set. The former does preserve ordering, but it sorts the entries, I think. Overloading the compare function might allow you to disable the sorting. Or use std::vector

[Levi-Armstrong]

But there's std::set and std::unordered_set. The former does preserve ordering, but it sorts the entries, I think. Overloading the compare function might allow you to disable the sorting. Or use std::vector

The std::set iterator is based on the sorted order and not the inserted order which is the issue. It also looks like unordered_set does not iterate based on insertion order.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.41%. Comparing base (0168d42) to head (7fdfd67).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main      #35   +/-   ##
=======================================
  Coverage   98.41%   98.41%           
=======================================
  Files           4        2    -2     
  Lines         189      189           
=======================================
  Hits          186      186           
  Misses          3        3           

Files with missing lines	Coverage Δ
include/boost_plugin_loader/plugin_loader.hpp	99.09% <100.00%> (ø)
src/utils.cpp	97.43% <100.00%> (+0.06%)	⬆️

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.