Add error message to build to make packing type errors clear
Because the base template actually handles things (all integer types),
it needs to ward off non-integer types in a clear way, rather than
relying on the user seeing that a tuple doesn't have an operator <<(),
for example. This provides a clear message if a specialized pack
operation was not hit.
Tested-by: attempted to build with a non-supported pack type
Change-Id: I66280831e88b4eac903b89f523e5f1a56a53cf9d
Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>
diff --git a/include/ipmid/message/pack.hpp b/include/ipmid/message/pack.hpp
index b458af4..8a9abe1 100644
--- a/include/ipmid/message/pack.hpp
+++ b/include/ipmid/message/pack.hpp
@@ -75,6 +75,8 @@
*/
static int op(Payload& p, const T& t)
{
+ static_assert(std::is_integral_v<T>,
+ "Attempt to pack a type that has no IPMI pack operation");
// if not on a byte boundary, must pack values LSbit/LSByte first
if (p.bitCount)
{