triangle icon indicating copy to clipboard operation
triangle copied to clipboard

Modernize: replace macros with inline functions

Open GravisZro opened this issue 10 months ago • 3 comments

  • Make sure all macros pass in all variables used (a mesh* type varaible (aka m) needs to be passed in for several)
  • Rewrite macros as always inlined functions to ensure type correctness

GravisZro avatar Jul 01 '25 23:07 GravisZro

Your PRs cannot merged without merge conflicts.

Perhaps a summary PR would be a sensible idea?

robinchrist avatar Jul 03 '25 16:07 robinchrist

@robinchrist Generally people want fine grained PRs (to investigate changes) and then rebase each PR as needed to resolve conflicts. Anyway, if you want one large PR then I can make it that way.

GravisZro avatar Jul 03 '25 16:07 GravisZro

@robinchrist Do you want me to combine these PRs into a single PR?

GravisZro avatar Jul 03 '25 16:07 GravisZro