cmlibs / zinc

Source code repository for OpenCMISS-Zinc
Mozilla Public License 2.0
15 stars 18 forks source link

Remove node/element group fields #267

Closed rchristie closed 1 year ago

rchristie commented 1 year ago

Closes #263. Remove deprecated API: Elementtemplate defineFieldSimpleNodal, defineFieldConstant, setting nodes before in template.

rchristie commented 1 year ago

Use of the de-reference operator when declaring pointers, should it be next to the type or the variable?

I've always thought the full type of the variable should be together, then the name of it, e.g.

cmzn_field*& fieldRef

So fieldRef is a reference to a field pointer. I've recently found my new Visual Studio formats this way by default and I want to go with it. Admittedly, C/C++ associates it with the variable, such as when declaring several variables at once:

cmzn_field *bob, **fred;

However, I no longer bulk declare variables like this -- it's typical of the C code I've been removing.