Changeset View
Changeset View
Standalone View
Standalone View
source/blender/makesdna/DNA_curve_types.h
| Show First 20 Lines • Show All 285 Lines • ▼ Show 20 Lines | typedef struct Curve { | ||||
| /** | /** | ||||
| * A pointer to curve data from evaluation. Owned by the object's #geometry_set_eval, either as a | * A pointer to curve data from evaluation. Owned by the object's #geometry_set_eval, either as a | ||||
| * geometry instance or the data of the evaluated #CurveComponent. The curve may also contain | * geometry instance or the data of the evaluated #CurveComponent. The curve may also contain | ||||
| * data in the #nurb list, but for evaluated curves this is the proper place to retrieve data, | * data in the #nurb list, but for evaluated curves this is the proper place to retrieve data, | ||||
| * since it also contains the result of geometry nodes evaluation, and isn't just a copy of the | * since it also contains the result of geometry nodes evaluation, and isn't just a copy of the | ||||
| * original object data. | * original object data. | ||||
| */ | */ | ||||
| const struct Curves *curve_eval; | const struct Curves *curve_eval; | ||||
sergey: The description mentions removal of `curve_eval`, but it is still here?
It also feels that it… | |||||
HooglyBooglyAuthorUnsubmitted Done Inline ActionsAh, totally right, sorry. That was left over from an earlier attempted fix. I updated the description. HooglyBoogly: Ah, totally right, sorry. That was left over from an earlier attempted fix. I updated the… | |||||
| /** | |||||
| * If non-zero, the #editfont and #editnurb pointers are not owned by this #Curve. That means | |||||
| * this curve is a container for the result of object geometry evaluation. This only works | |||||
| * because evaluated object data never outlives original data. | |||||
| */ | |||||
| char edit_data_not_owned; | |||||
sergeyUnsubmitted Done Inline ActionsWhat about edit_data_shared to avoid double negations (if not edit_data_not_owned)? sergey: What about `edit_data_shared` to avoid double negations (if not edit_data_not_owned)? | |||||
JacquesLuckeUnsubmitted Done Inline ActionsGuess the main reason for this name is that it has minimal impact on existing code? JacquesLucke: Guess the main reason for this name is that it has minimal impact on existing code? | |||||
| char _pad3[7]; | |||||
| void *batch_cache; | void *batch_cache; | ||||
| } Curve; | } Curve; | ||||
| #define CURVE_VFONT_ANY(cu) ((cu)->vfont), ((cu)->vfontb), ((cu)->vfonti), ((cu)->vfontbi) | #define CURVE_VFONT_ANY(cu) ((cu)->vfont), ((cu)->vfontb), ((cu)->vfonti), ((cu)->vfontbi) | ||||
| /* **************** CURVE ********************* */ | /* **************** CURVE ********************* */ | ||||
| ▲ Show 20 Lines • Show All 297 Lines • Show Last 20 Lines | |||||
The description mentions removal of curve_eval, but it is still here?
It also feels that it should be in the runtime.