Closed brainiac closed 6 years ago
Thanks for the report.
Seems like the std::map
overload is not chosen, thus it falls back trying to construct a std::pair<**const** std::string, std::string>
...
I'll fix it tomorrow.
@brainiac Could you please check the fix in #1301?
Hi @nlohmann and @theodelrieu, thanks for looking into this. It looks like with the fix in #1301 it is working now.
It looks like the change to fix #1292 (11fecc25af64529599e9a24675651115d6bae322) causes a regression when converting to a container std::pair with a non-trivial type. I didn't see a unit test that covers this case.
Example code that show the failure: https://wandbox.org/permlink/BVgyd1o4MW60bqvS