added inline to NLOHMANN_DEFINE_TYPE_NON_INTRUSIVE macro

[jwittbrodt]

Hey,

just a tiny pull request here.

I tried out the DEFINE_TYPE macros and noticed that the functions defined by the NLOHMANN_DEFINE_TYPE_NON_INTRUSIVE macro weren't marked inline. I could find no reason why they cannot be inline or why forcing users to manually declare the to/from_json functions would be helpful, so I simply added the inline.

Cheers,
Jonas

---

• Changes are described in the pull request, or an existing issue is referenced.
• The test suite compiles and runs without error.
• Code coverage is 100%. Test cases can be added by editing the test suite.
• The source code is amalgamated; that is, after making changes to the sources in the include/nlohmann directory, run make amalgamate to create the single-header file single_include/nlohmann/json.hpp. The whole process is described here.

[coveralls]

Coverage remained the same at 100.0% when pulling 0f22ed0 on jwittbrodt:develop into 43e07bb on nlohmann:develop.

[nlohmann]

Looks good to me.

[nlohmann]

Thanks!

[nlohmann]

---

🔖 Release item

This issue/PR will be part of the next release of the library. This template helps preparing the release notes.

Type

• ✨ New Feature
• 🐛 Bug Fix
• ⚡️ Improvement
• 🔨 Further Change
• 🔥 Deprecated function

Description

• Made NLOHMANN_DEFINE_TYPE_NON_INTRUSIVE macro inline. added inline to NLOHMANN_DEFINE_TYPE_NON_INTRUSIVE macro #2306

---