Skip to content

Changed the variable name c_int to coeff_int

Pradipta Samanta requested to merge bugfix_change_c_int into main

What is the bug

The use of c_int as a variable name in the module mo_lib_interpolation_scalar would create conflicts during the C++ implementation of the code. This MR solves the issue #6 (closed)

How do you fix it

The variable name c_int is changed to coeff_int

How urgent is the bugfix

  • I need it as soon as possible
  • I can wait for a couple of days
  • None of my current codes is directly affected

Mandatory steps before review

  • Gitlab CI passes (Hint: use make format for linting)
  • Bugfix is covered by additional unit tests
  • Mark the merge request as ready by removing Draft:

Mandatory steps before merge

  • Reviewed by a maintainer
  • Incorporate review suggestions
  • Remember to edit the commit message and select the proper changelog category (feature/bugfix/other)
  • Prior to merging, please remove any boilerplate from the MR description, retaining only the What is the bug and How do you fix it section to maintain

You are not supposed to merge this request by yourself, the maintainers of libiconmath take care of this action!

Edited by Yen-Chen Chen

Merge request reports

Loading