You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There's a good opportunity to make the code more readable when it comes to serialization/deserialization of many classes (like grids, file source/sinks, coords, etc).
currently, the code looks like this: Rbt::WriteWithThrow(ostr, (const char*)&m_NX, sizeof(m_NX));
improvements:
we should replace this (const char*) casting with a reinterpret_cast
encapsulate all this within a templated function
explicitly set the flags for ostream right after instantiation to automatically raise an exception in case an operation fails(ostr.exceptions(...))
The text was updated successfully, but these errors were encountered:
There's a good opportunity to make the code more readable when it comes to serialization/deserialization of many classes (like grids, file source/sinks, coords, etc).
currently, the code looks like this:
Rbt::WriteWithThrow(ostr, (const char*)&m_NX, sizeof(m_NX));
improvements:
(const char*)
casting with areinterpret_cast
ostr.exceptions(...)
)The text was updated successfully, but these errors were encountered: