-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GRIDEDIT-1407 add mkernel_mesh2d_get_property api #368
GRIDEDIT-1407 add mkernel_mesh2d_get_property api #368
Conversation
/// Returns the value of the property | ||
// [[nodiscard]] std::vector<double> GetProperty(Property property) const; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess sonarcloud will complain about this, commend out code.
if (static_cast<size_t>(geometryList.num_coordinates) < values.size()) | ||
{ | ||
throw meshkernel::MeshKernelError("GeometryList with wrong dimensions"); | ||
} | ||
std::copy(values.begin(), values.end(), geometryList.values); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this snippet common to both cases?
/// | ||
/// @param [in] property The property to retrive |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You probably already have an email about the typo
/// @param[in] propertyValue The value representing the specific property | ||
/// @param[in] dimension The dimension of the specified property | ||
/// @returns Error code |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
dimension should be an out parameter
No description provided.