Page MenuHome
Differential D6379 Diff 20169 source/blender/editors/space_graph/graph_edit.c

Changeset View

Standalone View

View Options

source/blender/editors/space_graph/graph_edit.c

Show First 20 Lines • Show All 992 Lines • ▼ Show 20 Lines
ot->poll = graphop_selected_fcurve_poll; ot->poll = graphop_selected_fcurve_poll;
/* flags */ /* flags */
ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO; ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
// todo: add props for start/end frames // todo: add props for start/end frames
} }
/* ******************** Un-Bake F-Curve Operator *********************** */
/* This operator unbakes the data of the selected F-Points to F-Curves */
sybrenUnsubmitted
Done
Inline Actions

Sentences should end in a period.

sybren: Sentences should end in a period.
/* Bake each F-Curve into a set of samples */
static void unbake_graph_curves(bAnimContext *ac, int start, int end)
sybrenUnsubmitted
Done
Inline Actions

This is exactly the opposite of what this function is doing.

sybren: This is exactly the opposite of what this function is doing.
{
ListBase anim_data = {NULL, NULL};
bAnimListElem *ale;
int filter;
/* filter data */
filter = (ANIMFILTER_DATA_VISIBLE | ANIMFILTER_CURVE_VISIBLE | ANIMFILTER_SEL |
ANIMFILTER_FOREDIT | ANIMFILTER_NODUPLIS);
sybrenUnsubmitted
Done
Inline Actions

filter can be declared and initialised one the same line, and declared const.

sybren: `filter` can be declared and initialised one the same line, and declared `const`.
ANIM_animdata_filter(ac, &anim_data, filter, ac->data, ac->datatype);
/* loop through filtered data and add keys between selected keyframes on every frame */
for (ale = anim_data.first; ale; ale = ale->next) {
FCurve *fcu = (FCurve *)ale->key_data;
/* create keyframes */
fcurve_samples_to_keyframes(fcu, start, end);
sybrenUnsubmitted
Done
Inline Actions

No need to repeat the function name in a comment.

sybren: No need to repeat the function name in a comment.
ale->update |= ANIM_UPDATE_DEPS;
}
ANIM_animdata_update(ac, &anim_data);
ANIM_animdata_freelist(&anim_data);
}
/* ------------------- */
static int graphkeys_unbake_exec(bContext *C, wmOperator *UNUSED(op))
{
bAnimContext ac;
Scene *scene = NULL;
int start, end;
/* get editor data */
if (ANIM_animdata_get_context(C, &ac) == 0) {
sybrenUnsubmitted
Done
Inline Actions

Comments should be full sentences (start with capital, end with period).

sybren: Comments should be full sentences (start with capital, end with period).
return OPERATOR_CANCELLED;
}
/* for now, init start/end from preview-range extents */
// TODO: add properties for this
scene = ac.scene;
sybrenUnsubmitted
Done
Inline Actions

TODOs should have the name of the person who knows more about this.

sybren: TODOs should have the name of the person who knows more about this.
sybrenUnsubmitted
Done
Inline Actions

No C++ comments in C files.

sybren: No C++ comments in C files.
start = PSFRA;
sybrenUnsubmitted
Done
Inline Actions

Let's not copy "For now" comments from 2009.

sybren: Let's not copy "For now" comments from 2009.
end = PEFRA;
/* unbake keyframes */
unbake_graph_curves(&ac, start, end);
/* set notifier that keyframes have changed */
// NOTE: some distinction between order/number of keyframes and type should be made?
WM_event_add_notifier(C, NC_ANIMATION | ND_KEYFRAME | NA_EDITED, NULL);
sybrenUnsubmitted
Done
Inline Actions

No C++ comments in C files.

sybren: No C++ comments in C files.
return OPERATOR_FINISHED;
}
void GRAPH_OT_unbake(wmOperatorType *ot)
{
/* identifiers */
ot->name = "Un-Bake Curve";
ot->idname = "GRAPH_OT_unbake";
ot->description = "Un-Bake selected F-Points to F-Curves";
/* api callbacks */
ot->invoke = WM_operator_confirm; // FIXME...
ot->exec = graphkeys_unbake_exec;
ot->poll = graphop_selected_fcurve_poll;
sybrenUnsubmitted
Done
Inline Actions

I just noted this one, and I think we can remove it. Un-baking is a lossless process (ignoring the start/end frame), and we have an undo system that people can use if they're not happy with the result. I don't see the need for a confirmation pop-up.

sybren: I just noted this one, and I think we can remove it. Un-baking is a lossless process (ignoring…
/* flags */
ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
// todo: add props for start/end frames
}
sybrenUnsubmitted
Done
Inline Actions

TODOs should have a name (and be capitalised).

sybren: TODOs should have a name (and be capitalised).
sybrenUnsubmitted
Done
Inline Actions

This is a new operator, so I don't see why it should have a TODO from 2009 in there.

sybren: This is a new operator, so I don't see why it should have a TODO from 2009 in there.
zeddbAuthorUnsubmitted
Done
Inline Actions

But muh vintage code comments! D:

zeddb: But muh vintage code comments! D:
#ifdef WITH_AUDASPACE #ifdef WITH_AUDASPACE
/* ******************** Sound Bake F-Curve Operator *********************** */ /* ******************** Sound Bake F-Curve Operator *********************** */
/* This operator bakes the given sound to the selected F-Curves */ /* This operator bakes the given sound to the selected F-Curves */
/* ------------------- */ /* ------------------- */
/* Custom data storage passed to the F-Sample-ing function, /* Custom data storage passed to the F-Sample-ing function,
▲ Show 20 Lines • Show All 992 Lines • Show Last 20 Lines