Cleanup: comment line length (editors)
authorCampbell Barton <ideasman42@gmail.com>
Tue, 15 Jan 2019 12:24:20 +0000 (23:24 +1100)
committerCampbell Barton <ideasman42@gmail.com>
Tue, 15 Jan 2019 12:30:31 +0000 (23:30 +1100)
Prevents clang-format wrapping text before comments.

209 files changed:
source/blender/editors/animation/anim_channels_defines.c
source/blender/editors/animation/anim_channels_edit.c
source/blender/editors/animation/anim_draw.c
source/blender/editors/animation/anim_filter.c
source/blender/editors/animation/anim_markers.c
source/blender/editors/animation/drivers.c
source/blender/editors/animation/fmodifier_ui.c
source/blender/editors/animation/keyframes_draw.c
source/blender/editors/animation/keyframes_edit.c
source/blender/editors/animation/keyframing.c
source/blender/editors/animation/keyingsets.c
source/blender/editors/armature/armature_add.c
source/blender/editors/armature/armature_edit.c
source/blender/editors/armature/armature_relations.c
source/blender/editors/armature/armature_utils.c
source/blender/editors/armature/meshlaplacian.c
source/blender/editors/armature/pose_lib.c
source/blender/editors/armature/pose_select.c
source/blender/editors/armature/pose_slide.c
source/blender/editors/armature/pose_transform.c
source/blender/editors/armature/pose_utils.c
source/blender/editors/curve/editcurve.c
source/blender/editors/curve/editcurve_add.c
source/blender/editors/gpencil/annotate_draw.c
source/blender/editors/gpencil/annotate_paint.c
source/blender/editors/gpencil/drawgpencil.c
source/blender/editors/gpencil/gpencil_brush.c
source/blender/editors/gpencil/gpencil_convert.c
source/blender/editors/gpencil/gpencil_data.c
source/blender/editors/gpencil/gpencil_edit.c
source/blender/editors/gpencil/gpencil_fill.c
source/blender/editors/gpencil/gpencil_intern.h
source/blender/editors/gpencil/gpencil_interpolate.c
source/blender/editors/gpencil/gpencil_paint.c
source/blender/editors/gpencil/gpencil_select.c
source/blender/editors/gpencil/gpencil_utils.c
source/blender/editors/include/ED_anim_api.h
source/blender/editors/include/ED_armature.h
source/blender/editors/include/ED_fileselect.h
source/blender/editors/include/ED_keyframes_draw.h
source/blender/editors/include/ED_keyframes_edit.h
source/blender/editors/include/ED_keyframing.h
source/blender/editors/include/ED_numinput.h
source/blender/editors/include/ED_object.h
source/blender/editors/include/ED_screen_types.h
source/blender/editors/include/ED_transverts.h
source/blender/editors/include/ED_view3d.h
source/blender/editors/include/UI_interface.h
source/blender/editors/interface/interface.c
source/blender/editors/interface/interface_align.c
source/blender/editors/interface/interface_anim.c
source/blender/editors/interface/interface_context_menu.c
source/blender/editors/interface/interface_draw.c
source/blender/editors/interface/interface_eyedropper_datablock.c
source/blender/editors/interface/interface_handlers.c
source/blender/editors/interface/interface_icons.c
source/blender/editors/interface/interface_intern.h
source/blender/editors/interface/interface_layout.c
source/blender/editors/interface/interface_ops.c
source/blender/editors/interface/interface_panel.c
source/blender/editors/interface/interface_region_color_picker.c
source/blender/editors/interface/interface_region_menu_pie.c
source/blender/editors/interface/interface_region_menu_popup.c
source/blender/editors/interface/interface_region_popover.c
source/blender/editors/interface/interface_region_search.c
source/blender/editors/interface/interface_style.c
source/blender/editors/interface/interface_templates.c
source/blender/editors/interface/interface_utils.c
source/blender/editors/interface/interface_widgets.c
source/blender/editors/interface/resources.c
source/blender/editors/interface/view2d.c
source/blender/editors/interface/view2d_ops.c
source/blender/editors/lattice/editlattice_tools.c
source/blender/editors/mask/mask_draw.c
source/blender/editors/mask/mask_ops.c
source/blender/editors/mesh/editface.c
source/blender/editors/mesh/editmesh_add.c
source/blender/editors/mesh/editmesh_bevel.c
source/blender/editors/mesh/editmesh_inset.c
source/blender/editors/mesh/editmesh_intersect.c
source/blender/editors/mesh/editmesh_knife.c
source/blender/editors/mesh/editmesh_loopcut.c
source/blender/editors/mesh/editmesh_path.c
source/blender/editors/mesh/editmesh_rip.c
source/blender/editors/mesh/editmesh_select.c
source/blender/editors/mesh/editmesh_tools.c
source/blender/editors/mesh/editmesh_utils.c
source/blender/editors/mesh/mesh_mirror.c
source/blender/editors/mesh/meshtools.c
source/blender/editors/object/object_add.c
source/blender/editors/object/object_bake.c
source/blender/editors/object/object_bake_api.c
source/blender/editors/object/object_collection.c
source/blender/editors/object/object_constraint.c
source/blender/editors/object/object_data_transfer.c
source/blender/editors/object/object_edit.c
source/blender/editors/object/object_modifier.c
source/blender/editors/object/object_relations.c
source/blender/editors/object/object_select.c
source/blender/editors/object/object_transform.c
source/blender/editors/object/object_vgroup.c
source/blender/editors/physics/particle_edit.c
source/blender/editors/physics/particle_object.c
source/blender/editors/render/render_internal.c
source/blender/editors/render/render_opengl.c
source/blender/editors/render/render_shading.c
source/blender/editors/screen/screen_context.c
source/blender/editors/screen/screen_ops.c
source/blender/editors/screen/screendump.c
source/blender/editors/sculpt_paint/paint_cursor.c
source/blender/editors/sculpt_paint/paint_image.c
source/blender/editors/sculpt_paint/paint_image_2d.c
source/blender/editors/sculpt_paint/paint_image_proj.c
source/blender/editors/sculpt_paint/paint_mask.c
source/blender/editors/sculpt_paint/paint_stroke.c
source/blender/editors/sculpt_paint/paint_vertex.c
source/blender/editors/sculpt_paint/paint_vertex_weight_ops.c
source/blender/editors/sculpt_paint/sculpt.c
source/blender/editors/sculpt_paint/sculpt_uv.c
source/blender/editors/sound/sound_ops.c
source/blender/editors/space_action/action_data.c
source/blender/editors/space_action/action_edit.c
source/blender/editors/space_action/action_select.c
source/blender/editors/space_action/space_action.c
source/blender/editors/space_buttons/buttons_context.c
source/blender/editors/space_buttons/space_buttons.c
source/blender/editors/space_clip/clip_buttons.c
source/blender/editors/space_clip/clip_dopesheet_draw.c
source/blender/editors/space_clip/clip_draw.c
source/blender/editors/space_clip/clip_graph_ops.c
source/blender/editors/space_clip/space_clip.c
source/blender/editors/space_console/console_intern.h
source/blender/editors/space_console/console_ops.c
source/blender/editors/space_file/file_draw.c
source/blender/editors/space_file/file_ops.c
source/blender/editors/space_file/filelist.c
source/blender/editors/space_file/filesel.c
source/blender/editors/space_file/fsmenu.c
source/blender/editors/space_graph/graph_buttons.c
source/blender/editors/space_graph/graph_draw.c
source/blender/editors/space_graph/graph_edit.c
source/blender/editors/space_graph/graph_ops.c
source/blender/editors/space_graph/graph_select.c
source/blender/editors/space_graph/space_graph.c
source/blender/editors/space_image/image_buttons.c
source/blender/editors/space_image/image_ops.c
source/blender/editors/space_image/space_image.c
source/blender/editors/space_info/textview.c
source/blender/editors/space_info/textview.h
source/blender/editors/space_nla/nla_buttons.c
source/blender/editors/space_nla/nla_channels.c
source/blender/editors/space_nla/nla_draw.c
source/blender/editors/space_nla/nla_edit.c
source/blender/editors/space_nla/nla_select.c
source/blender/editors/space_node/drawnode.c
source/blender/editors/space_node/node_draw.c
source/blender/editors/space_node/node_edit.c
source/blender/editors/space_node/node_group.c
source/blender/editors/space_node/node_intern.h
source/blender/editors/space_node/node_relationships.c
source/blender/editors/space_node/node_templates.c
source/blender/editors/space_node/space_node.c
source/blender/editors/space_outliner/outliner_draw.c
source/blender/editors/space_outliner/outliner_edit.c
source/blender/editors/space_outliner/outliner_intern.h
source/blender/editors/space_outliner/outliner_select.c
source/blender/editors/space_outliner/outliner_tools.c
source/blender/editors/space_outliner/outliner_tree.c
source/blender/editors/space_outliner/outliner_utils.c
source/blender/editors/space_sequencer/sequencer_add.c
source/blender/editors/space_sequencer/sequencer_draw.c
source/blender/editors/space_sequencer/sequencer_edit.c
source/blender/editors/space_sequencer/sequencer_select.c
source/blender/editors/space_text/text_autocomplete.c
source/blender/editors/space_text/text_draw.c
source/blender/editors/space_text/text_format.h
source/blender/editors/space_text/text_ops.c
source/blender/editors/space_userpref/space_userpref.c
source/blender/editors/space_view3d/drawobject.c
source/blender/editors/space_view3d/space_view3d.c
source/blender/editors/space_view3d/view3d_buttons.c
source/blender/editors/space_view3d/view3d_camera_control.c
source/blender/editors/space_view3d/view3d_draw.c
source/blender/editors/space_view3d/view3d_draw_legacy.c
source/blender/editors/space_view3d/view3d_edit.c
source/blender/editors/space_view3d/view3d_fly.c
source/blender/editors/space_view3d/view3d_gizmo_navigate_type.c
source/blender/editors/space_view3d/view3d_project.c
source/blender/editors/space_view3d/view3d_select.c
source/blender/editors/space_view3d/view3d_snap.c
source/blender/editors/space_view3d/view3d_utils.c
source/blender/editors/space_view3d/view3d_view.c
source/blender/editors/space_view3d/view3d_walk.c
source/blender/editors/transform/transform.c
source/blender/editors/transform/transform_conversions.c
source/blender/editors/transform/transform_generics.c
source/blender/editors/transform/transform_gizmo_3d.c
source/blender/editors/transform/transform_input.c
source/blender/editors/transform/transform_ops.c
source/blender/editors/transform/transform_snap.c
source/blender/editors/transform/transform_snap_object.c
source/blender/editors/util/ed_util.c
source/blender/editors/util/numinput.c
source/blender/editors/uvedit/uvedit_buttons.c
source/blender/editors/uvedit/uvedit_draw.c
source/blender/editors/uvedit/uvedit_ops.c
source/blender/editors/uvedit/uvedit_parametrizer.c
source/blender/editors/uvedit/uvedit_smart_stitch.c
source/blender/editors/uvedit/uvedit_unwrap_ops.c

index e7416b9..8f052b4 100644 (file)
@@ -3531,7 +3531,8 @@ static bAnimChannelType ACF_NLAACTION =
        "NLA Active Action",            /* type name */
        ACHANNEL_ROLE_CHANNEL,          /* role */
 
-       acf_nlaaction_color,            /* backdrop color (NOTE: the backdrop handles this too, since it needs special hacks) */
+       acf_nlaaction_color,            /* backdrop color (NOTE: the backdrop handles this too,
+                                        * since it needs special hacks) */
        acf_nlaaction_backdrop,         /* backdrop */
        acf_generic_indention_flexible, /* indent level */
        acf_generic_group_offset,       /* offset */           // XXX?
@@ -4413,7 +4414,8 @@ static void draw_setting_widget(bAnimContext *ac, bAnimListElem *ale, const bAni
                                        break;
 
                                /* no flushing */
-                               case ACHANNEL_SETTING_EXPAND: /* expanding - cannot flush, otherwise all would open/close at once */
+                               case ACHANNEL_SETTING_EXPAND: /* expanding - cannot flush,
+                                                              * otherwise all would open/close at once */
                                default:
                                        UI_but_func_set(but, achannel_setting_widget_cb, NULL, NULL);
                                        break;
@@ -4601,7 +4603,8 @@ void ANIM_channel_draw_widgets(const bContext *C, bAnimContext *ac, bAnimListEle
 
                        /* modifiers disable */
                        if (acf->has_setting(ac, ale, ACHANNEL_SETTING_MOD_OFF)) {
-                               offset += ICON_WIDTH * 1.2f; /* hack: extra spacing, to avoid touching the mute toggle */
+                               /* hack: extra spacing, to avoid touching the mute toggle */
+                               offset += ICON_WIDTH * 1.2f;
                                draw_setting_widget(ac, ale, acf, block, (int)v2d->cur.xmax - offset, ymid, ACHANNEL_SETTING_MOD_OFF);
                        }
 
index b232c23..af73003 100644 (file)
@@ -881,7 +881,8 @@ static void rearrange_animchannel_add_to_islands(ListBase *islands, ListBase *sr
                                                  Link *channel, eAnim_ChannelType type,
                                                  const bool is_hidden)
 {
-       tReorderChannelIsland *island = islands->last;  /* always try to add to last island if possible */
+       /* always try to add to last island if possible */
+       tReorderChannelIsland *island = islands->last;
        bool is_sel = false, is_untouchable = false;
 
        /* get flags - selected and untouchable from the channel */
@@ -1561,7 +1562,8 @@ static void ANIM_OT_channels_group(wmOperatorType *ot)
        ot->prop = RNA_def_string(ot->srna, "name", "New Group",
                                  sizeof(((bActionGroup *)NULL)->name),
                                  "Name", "Name of newly created group");
-       /* RNA_def_property_flag(ot->prop, PROP_SKIP_SAVE); */ /* XXX: still not too sure about this - keeping same text is confusing... */
+       /* XXX: still not too sure about this - keeping same text is confusing... */
+       // RNA_def_property_flag(ot->prop, PROP_SKIP_SAVE);
 }
 
 /* ----------------------------------------------------------- */
@@ -2948,7 +2950,8 @@ static int mouse_anim_channels(bContext *C, bAnimContext *ac, int channel_index,
                                BKE_gpencil_layer_setactive(gpd, gpl);
                        }
 
-                       WM_event_add_notifier(C, NC_GPENCIL | ND_DATA | NA_EDITED | ND_SPACE_PROPERTIES, NULL); /* Grease Pencil updates */
+                       /* Grease Pencil updates */
+                       WM_event_add_notifier(C, NC_GPENCIL | ND_DATA | NA_EDITED | ND_SPACE_PROPERTIES, NULL);
                        notifierFlags |= (ND_ANIMCHAN | NA_EDITED); /* Animation Editors updates */
                        break;
                }
@@ -3018,17 +3021,24 @@ static int animchannels_mouseclick_invoke(bContext *C, wmOperator *op, const wmE
        v2d = &ar->v2d;
 
        /* select mode is either replace (deselect all, then add) or add/extend */
-       if (RNA_boolean_get(op->ptr, "extend"))
+       if (RNA_boolean_get(op->ptr, "extend")) {
                selectmode = SELECT_INVERT;
-       else if (RNA_boolean_get(op->ptr, "children_only"))
-               selectmode = -1;  /* this is a bit of a special case for ActionGroups only... should it be removed or extended to all instead? */
-       else
+       }
+       else if (RNA_boolean_get(op->ptr, "children_only")) {
+               /* this is a bit of a special case for ActionGroups only...
+                * should it be removed or extended to all instead? */
+               selectmode = -1;
+       }
+       else {
                selectmode = SELECT_REPLACE;
+       }
 
        /* figure out which channel user clicked in
-        * Note: although channels technically start at (y = ACHANNEL_FIRST), we need to adjust by half a channel's height
-        *       so that the tops of channels get caught ok. Since ACHANNEL_FIRST is really ACHANNEL_HEIGHT, we simply use
-        *       ACHANNEL_HEIGHT_HALF.
+        *
+        * Note:
+        * although channels technically start at (y = ACHANNEL_FIRST),
+        * we need to adjust by half a channel's height so that the tops of channels get caught ok.
+        * Since ACHANNEL_FIRST is really ACHANNEL_HEIGHT, we simply use ACHANNEL_HEIGHT_HALF.
         */
        UI_view2d_region_to_view(v2d, event->mval[0], event->mval[1], &x, &y);
        UI_view2d_listview_view_to_cell(v2d, ACHANNEL_NAMEWIDTH, ACHANNEL_STEP(&ac), 0, (float)ACHANNEL_HEIGHT_HALF(&ac), x, y, NULL, &channel_index);
index 239ab5f..27584c5 100644 (file)
@@ -175,7 +175,8 @@ void ANIM_draw_previewrange(const bContext *C, View2D *v2d, int end_frame_width)
 
                immBindBuiltinProgram(GPU_SHADER_2D_UNIFORM_COLOR);
                immUniformThemeColorShadeAlpha(TH_ANIM_PREVIEW_RANGE, -25, -30);
-               //immUniformColor4f(0.8f, 0.44f, 0.1f, 0.2f); /* XXX: Fix this hardcoded color (anim_active) */
+               /* XXX: Fix this hardcoded color (anim_active) */
+               //immUniformColor4f(0.8f, 0.44f, 0.1f, 0.2f);
 
                /* only draw two separate 'curtains' if there's no overlap between them */
                if (PSFRA < PEFRA + end_frame_width) {
@@ -285,7 +286,8 @@ static short bezt_nlamapping_restore(KeyframeEditData *ked, BezTriple *bezt)
        return 0;
 }
 
-/* helper function for ANIM_nla_mapping_apply_fcurve() -> "apply", i.e. mapping points to NLA-mapped global time */
+/* helper function for ANIM_nla_mapping_apply_fcurve() -> "apply",
+ * i.e. mapping points to NLA-mapped global time */
 static short bezt_nlamapping_apply(KeyframeEditData *ked, BezTriple *bezt)
 {
        /* AnimData block providing scaling is stored in 'data', only_keys option is stored in i1 */
index d066d13..db887db 100644 (file)
@@ -908,7 +908,8 @@ static bAnimListElem *make_new_animlistelem(void *data, short datatype, ID *owne
                                break;
                        }
                        case ANIMTYPE_FCURVE:
-                       case ANIMTYPE_NLACURVE: /* practically the same as ANIMTYPE_FCURVE. Differences are applied post-creation */
+                       case ANIMTYPE_NLACURVE: /* practically the same as ANIMTYPE_FCURVE.
+                                                * Differences are applied post-creation */
                        {
                                FCurve *fcu = (FCurve *)data;
 
@@ -1001,7 +1002,8 @@ static bool skip_fcurve_selected_data(bDopeSheet *ads, FCurve *fcu, ID *owner_id
        if (fcu->grp != NULL && fcu->grp->flag & ADT_CURVES_ALWAYS_VISIBLE) {
                return false;
        }
-       /* hidden items should be skipped if we only care about visible data, but we aren't interested in hidden stuff */
+       /* hidden items should be skipped if we only care about visible data,
+        * but we aren't interested in hidden stuff */
        const bool skip_hidden = (filter_mode & ANIMFILTER_DATA_VISIBLE) && !(ads->filterflag & ADS_FILTER_INCL_HIDDEN);
 
        if (GS(owner_id->name) == ID_OB) {
@@ -1266,7 +1268,8 @@ static size_t animfilter_fcurves(ListBase *anim_data, bDopeSheet *ads,
         */
        for (fcu = first; ( (fcu = animfilter_fcurve_next(ads, fcu, fcurve_type, filter_mode, owner, owner_id)) ); fcu = fcu->next) {
                if (UNLIKELY(fcurve_type == ANIMTYPE_NLACURVE)) {
-                       /* NLA Control Curve - Basically the same as normal F-Curves, except we need to set some stuff differently */
+                       /* NLA Control Curve - Basically the same as normal F-Curves,
+                        * except we need to set some stuff differently */
                        ANIMCHANNEL_NEW_CHANNEL_FULL(fcu, ANIMTYPE_NLACURVE, owner_id, fcurve_owner_id, {
                                ale->owner = owner; /* strip */
                                ale->adt = NULL;    /* to prevent time mapping from causing problems */
@@ -1295,8 +1298,11 @@ static size_t animfilter_act_group(bAnimContext *ac, ListBase *anim_data, bDopeS
         *     - Hierarchy matters: this hack should be applied
         *     - Hierarchy ignored: cases like [#21276] won't work properly, unless we skip this hack
         */
-       if ( ((filter_mode & ANIMFILTER_LIST_VISIBLE) && EXPANDED_AGRP(ac, agrp) == 0) &&     /* care about hierarchy but group isn't expanded */
-            (filter_mode & (ANIMFILTER_SEL | ANIMFILTER_UNSEL)) )                          /* care about selection status */
+       if (
+               /* care about hierarchy but group isn't expanded */
+               ((filter_mode & ANIMFILTER_LIST_VISIBLE) && EXPANDED_AGRP(ac, agrp) == 0) &&
+               /* care about selection status */
+               (filter_mode & (ANIMFILTER_SEL | ANIMFILTER_UNSEL)) )
        {
                /* if the group itself isn't selected appropriately, we shouldn't consider it's children either */
                if (ANIMCHANNEL_SELOK(SEL_AGRP(agrp)) == 0)
@@ -1318,9 +1324,9 @@ static size_t animfilter_act_group(bAnimContext *ac, ListBase *anim_data, bDopeS
        {
                /* special filter so that we can get just the F-Curves within the active group */
                if (!(filter_mode & ANIMFILTER_ACTGROUPED) || (agrp->flag & AGRP_ACTIVE)) {
-                       /* for the Graph Editor, curves may be set to not be visible in the view to lessen clutter,
-                        * but to do this, we need to check that the group doesn't have it's not-visible flag set preventing
-                        * all its sub-curves to be shown
+                       /* for the Graph Editor, curves may be set to not be visible in the view to lessen
+                        * clutter, but to do this, we need to check that the group doesn't have it's
+                        * not-visible flag set preventing all its sub-curves to be shown
                         */
                        if (!(filter_mode & ANIMFILTER_CURVE_VISIBLE) || !(agrp->flag & AGRP_NOTVISIBLE)) {
                                /* group must be editable for its children to be editable (if we care about this) */
@@ -1343,7 +1349,8 @@ static size_t animfilter_act_group(bAnimContext *ac, ListBase *anim_data, bDopeS
                        /* restore original filter mode so that this next step works ok... */
                        //filter_mode = ofilter;
 
-                       /* filter selection of channel specially here again, since may be open and not subject to previous test */
+                       /* filter selection of channel specially here again,
+                        * since may be open and not subject to previous test */
                        if (ANIMCHANNEL_SELOK(SEL_AGRP(agrp)) ) {
                                ANIMCHANNEL_NEW_CHANNEL(agrp, ANIMTYPE_GROUP, owner_id, NULL);
                        }
@@ -1759,7 +1766,8 @@ static size_t animdata_filter_gpencil(bAnimContext *ac, ListBase *anim_data, voi
        else {
                bGPdata *gpd;
 
-               /* Grab all Grease Pencil datablocks directly from main, but only those that seem to be useful somewhere */
+               /* Grab all Grease Pencil datablocks directly from main,
+                * but only those that seem to be useful somewhere */
                for (gpd = ac->bmain->gpencil.first; gpd; gpd = gpd->id.next) {
                        /* only show if gpd is used by something... */
                        if (ID_REAL_USERS(gpd) < 1)
@@ -3260,7 +3268,8 @@ size_t ANIM_animdata_filter(bAnimContext *ac, ListBase *anim_data, eAnimFilter_F
                        case ANIMCONT_DRIVERS: /* Graph Editor -> Drivers Editing */
                        case ANIMCONT_NLA:     /* NLA Editor */
                        {
-                               /* all of these editors use the basic DopeSheet data for filtering options, but don't have all the same features */
+                               /* all of these editors use the basic DopeSheet data for filtering options,
+                                * but don't have all the same features */
                                items = animdata_filter_dopesheet(ac, anim_data, data, filter_mode);
                                break;
                        }
index 7416912..6b68704 100644 (file)
@@ -616,7 +616,8 @@ static int ed_markers_opwrap_invoke_custom(bContext *C, wmOperator *op, const wm
                BKE_report(op->reports, RPT_ERROR, "Programming error: operator does not actually have code to do anything!");
 
 
-       /* unless successful, must add "pass-through" to let normal operator's have a chance at tackling this event */
+       /* unless successful, must add "pass-through"
+        * to let normal operator's have a chance at tackling this event */
        if ((retval & (OPERATOR_FINISHED | OPERATOR_INTERFACE)) == 0) {
                retval |= OPERATOR_PASS_THROUGH;
        }
index ac57f77..85662ee 100644 (file)
@@ -329,7 +329,8 @@ int ANIM_add_driver_with_target(
 
        /* handle curve-property mappings based on mapping_type */
        switch (mapping_type) {
-               case CREATEDRIVER_MAPPING_N_N: /* N-N - Try to match as much as possible, then use the first one */
+               case CREATEDRIVER_MAPPING_N_N: /* N-N - Try to match as much as possible,
+                                               * then use the first one */
                {
                        /* Use the shorter of the two (to avoid out of bounds access) */
                        int dst_len = (RNA_property_array_check(prop)) ? RNA_property_array_length(&ptr, prop) : 1;
index 87c9d9c..a588c80 100644 (file)
@@ -358,9 +358,11 @@ static void fmod_envelope_addpoint_cb(bContext *C, void *fcm_dv, void *UNUSED(ar
                bool exists;
                int i = BKE_fcm_envelope_find_index(env->data, (float)(scene->r.cfra), env->totvert, &exists);
 
-               /* binarysearch_...() will set exists by default to 0, so if it is non-zero, that means that the point exists already */
-               if (exists)
+               /* binarysearch_...() will set exists by default to 0,
+                * so if it is non-zero, that means that the point exists already */
+               if (exists) {
                        return;
+               }
 
                /* add new */
                fedn = MEM_callocN((env->totvert + 1) * sizeof(FCM_EnvelopeData), "FCM_EnvelopeData");
index 72da71a..9fe4bc9 100644 (file)
@@ -557,7 +557,8 @@ void draw_keyframe_shape(float x, float y, float size, bool sel, short key_type,
                        size *= 0.85f;
                        break;
 
-               case BEZT_KEYTYPE_MOVEHOLD:  /* slightly smaller than normal keyframes (but by less than for breakdowns) */
+               case BEZT_KEYTYPE_MOVEHOLD:  /* slightly smaller than normal keyframes
+                                             * (but by less than for breakdowns) */
                        size *= 0.925f;
                        break;
 
@@ -942,9 +943,10 @@ void summary_to_keylist(bAnimContext *ac, DLRBT_Tree *keys, int saction_flag)
                /* loop through each F-Curve, grabbing the keyframes */
                for (ale = anim_data.first; ale; ale = ale->next) {
                        /* Why not use all #eAnim_KeyType here?
-                        * All of the other key types are actually "summaries" themselves, and will just end up duplicating stuff
-                        * that comes up through standard filtering of just F-Curves.
-                        * Given the way that these work, there isn't really any benefit at all from including them. - Aligorith */
+                        * All of the other key types are actually "summaries" themselves,
+                        * and will just end up duplicating stuff that comes up through
+                        * standard filtering of just F-Curves. Given the way that these work,
+                        * there isn't really any benefit at all from including them. - Aligorith */
 
                        switch (ale->datatype) {
                                case ALE_FCURVE:
index e733bb0..f85335a 100644 (file)
@@ -369,7 +369,8 @@ short ANIM_animchannel_keyframes_loop(KeyframeEditData *ked, bDopeSheet *ads, bA
        return 0;
 }
 
-/* This function is used to apply operation to all keyframes, regardless of the type without needed an AnimListElem wrapper */
+/* This function is used to apply operation to all keyframes,
+ * regardless of the type without needed an AnimListElem wrapper */
 short ANIM_animchanneldata_keyframes_loop(KeyframeEditData *ked, bDopeSheet *ads, void *data, int keytype, KeyframeEditFunc key_ok, KeyframeEditFunc key_cb, FcuEditFunc fcu_cb)
 {
        /* sanity checks */
@@ -670,25 +671,35 @@ KeyframeEditFunc ANIM_editkeyframes_ok(short mode)
 {
        /* eEditKeyframes_Validate */
        switch (mode) {
-               case BEZT_OK_FRAME: /* only if bezt falls on the right frame (float) */
+               case BEZT_OK_FRAME:
+                       /* only if bezt falls on the right frame (float) */
                        return ok_bezier_frame;
-               case BEZT_OK_FRAMERANGE: /* only if bezt falls within the specified frame range (floats) */
+               case BEZT_OK_FRAMERANGE:
+                       /* only if bezt falls within the specified frame range (floats) */
                        return ok_bezier_framerange;
-               case BEZT_OK_SELECTED:  /* only if bezt is selected (self) */
+               case BEZT_OK_SELECTED:
+                       /* only if bezt is selected (self) */
                        return ok_bezier_selected;
-               case BEZT_OK_VALUE: /* only if bezt value matches (float) */
+               case BEZT_OK_VALUE:
+                       /* only if bezt value matches (float) */
                        return ok_bezier_value;
-               case BEZT_OK_VALUERANGE: /* only if bezier falls within the specified value range (floats) */
+               case BEZT_OK_VALUERANGE:
+                       /* only if bezier falls within the specified value range (floats) */
                        return ok_bezier_valuerange;
-               case BEZT_OK_REGION: /* only if bezier falls within the specified rect (data -> rectf) */
+               case BEZT_OK_REGION:
+                       /* only if bezier falls within the specified rect (data -> rectf) */
                        return ok_bezier_region;
-               case BEZT_OK_REGION_LASSO: /* only if the point falls within KeyframeEdit_LassoData defined data */
+               case BEZT_OK_REGION_LASSO:
+                       /* only if the point falls within KeyframeEdit_LassoData defined data */
                        return ok_bezier_region_lasso;
-               case BEZT_OK_REGION_CIRCLE: /* only if the point falls within KeyframeEdit_CircleData defined data */
+               case BEZT_OK_REGION_CIRCLE:
+                       /* only if the point falls within KeyframeEdit_CircleData defined data */
                        return ok_bezier_region_circle;
-               case BEZT_OK_CHANNEL_LASSO: /* same as BEZT_OK_REGION_LASSO, but we're only using the x-value of the points */
+               case BEZT_OK_CHANNEL_LASSO:
+                       /* same as BEZT_OK_REGION_LASSO, but we're only using the x-value of the points */
                        return ok_bezier_channel_lasso;
-               case BEZT_OK_CHANNEL_CIRCLE: /* same as BEZT_OK_REGION_CIRCLE, but we're only using the x-value of the points */
+               case BEZT_OK_CHANNEL_CIRCLE:
+                       /* same as BEZT_OK_REGION_CIRCLE, but we're only using the x-value of the points */
                        return ok_bezier_channel_circle;
                default: /* nothing was ok */
                        return NULL;
@@ -698,7 +709,10 @@ KeyframeEditFunc ANIM_editkeyframes_ok(short mode)
 /* ******************************************* */
 /* Assorted Utility Functions */
 
-/* helper callback for <animeditor>_cfrasnap_exec() -> used to help get the average time of all selected beztriples */
+/**
+ * Helper callback for <animeditor>_cfrasnap_exec() ->
+ * used to help get the average time of all selected beztriples
+ */
 short bezt_calc_average(KeyframeEditData *ked, BezTriple *bezt)
 {
        /* only if selected */
@@ -718,7 +732,8 @@ short bezt_calc_average(KeyframeEditData *ked, BezTriple *bezt)
        return 0;
 }
 
-/* helper callback for columnselect_<animeditor>_keys() -> populate list CfraElems with frame numbers from selected beztriples */
+/* helper callback for columnselect_<animeditor>_keys() -> populate
+ * list CfraElems with frame numbers from selected beztriples */
 short bezt_to_cfraelem(KeyframeEditData *ked, BezTriple *bezt)
 {
        /* only if selected */
index 84d9116..d10147f 100644 (file)
@@ -512,11 +512,15 @@ int insert_vert_fcurve(FCurve *fcu, float x, float y, eBezTriple_KeyframeType ke
         *       to these later, we want these to work in a sane way out of
         *       the box.
         */
-       beztr.back = 1.70158f;     /* "back" easing - this value used to be used when overshoot=0, but that        */
-                                  /*                 introduced discontinuities in how the param worked           */
 
-       beztr.amplitude = 0.8f;    /* "elastic" easing - values here were hand-optimised for a default duration of */
-       beztr.period = 4.1f;       /*                    ~10 frames (typical mograph motion length)                */
+       /* "back" easing - this value used to be used when overshoot=0, but that
+        *                 introduced discontinuities in how the param worked. */
+       beztr.back = 1.70158f;
+
+       /* "elastic" easing - values here were hand-optimised for a default duration of
+        *                    ~10 frames (typical mograph motion length) */
+       beztr.amplitude = 0.8f;
+       beztr.period = 4.1f;
 
        /* add temp beztriple to keyframes */
        a = insert_bezt_fcurve(fcu, &beztr, flag);
index d980e43..83f454c 100644 (file)
@@ -1026,7 +1026,8 @@ int ANIM_apply_keyingset(bContext *C, ListBase *dsources, bAction *act, KeyingSe
                        RNA_id_pointer_create(ksp->id, &id_ptr);
                        if (RNA_path_resolve_property(&id_ptr, ksp->rna_path, &ptr, &prop)) {
                                arraylen = RNA_property_array_length(&ptr, prop);
-                               i = 0;  /* start from start of array, instead of the previously specified index - T48020 */
+                               /* start from start of array, instead of the previously specified index - T48020 */
+                               i = 0;
                        }
                }
 
index fb0766d..aa95530 100644 (file)
@@ -491,7 +491,9 @@ static int armature_duplicate_selected_exec(bContext *C, wmOperator *op)
        Object **objects = BKE_view_layer_array_from_objects_in_edit_mode_unique_data(view_layer, CTX_wm_view3d(C), &objects_len);
        for (uint ob_index = 0; ob_index < objects_len; ob_index++) {
                EditBone *ebone_iter;
-               EditBone *ebone_first_dupe = NULL;  /* The beginning of the duplicated bones in the edbo list */
+               /* The beginning of the duplicated bones in the edbo list */
+               EditBone *ebone_first_dupe = NULL;
+
                Object *ob = objects[ob_index];
                bArmature *arm = ob->data;
 
@@ -664,7 +666,8 @@ static int armature_symmetrize_exec(bContext *C, wmOperator *op)
                bArmature *arm = obedit->data;
 
                EditBone *ebone_iter;
-               EditBone *ebone_first_dupe = NULL;  /* The beginning of the duplicated mirrored bones in the edbo list */
+               /* The beginning of the duplicated mirrored bones in the edbo list */
+               EditBone *ebone_first_dupe = NULL;
 
                ED_armature_edit_sync_selection(arm->edbo); // XXX why is this needed?
 
index 7aa6b57..c262cbd 100644 (file)
@@ -87,7 +87,7 @@ void ED_armature_transform_apply(Main *bmain, Object *ob, float mat[4][4], const
 void ED_armature_transform_bones(struct bArmature *arm, float mat[4][4], const bool do_props)
 {
        EditBone *ebone;
-       float scale = mat4_to_scale(mat);   /* store the scale of the matrix here to use on envelopes */
+       float scale = mat4_to_scale(mat);  /* store the scale of the matrix here to use on envelopes */
        float mat3[3][3];
 
        copy_m3_m4(mat3, mat);
index aadb805..e48c2df 100644 (file)
@@ -144,7 +144,8 @@ static void joined_armature_fix_animdata_cb(ID *id, FCurve *fcu, void *user_data
                        const char *old_name = BLI_ghashIterator_getKey(&gh_iter);
                        const char *new_name = BLI_ghashIterator_getValue(&gh_iter);
 
-                       /* only remap if changed; this still means there will be some waste if there aren't many drivers/keys */
+                       /* only remap if changed; this still means there will be some
+                        * waste if there aren't many drivers/keys */
                        if (!STREQ(old_name, new_name) && strstr(fcu->rna_path, old_name)) {
                                fcu->rna_path = BKE_animsys_fix_rna_path_rename(id, fcu->rna_path, "pose.bones",
                                                                                old_name, new_name, 0, 0, false);
@@ -542,7 +543,8 @@ static void separate_armature_bones(Main *bmain, Object *ob, short sel)
                        for (ebo = arm->edbo->first; ebo; ebo = ebo->next) {
                                if (ebo->parent == curbone) {
                                        ebo->parent = NULL;
-                                       ebo->temp.p = NULL; /* this is needed to prevent random crashes with in ED_armature_from_edit */
+                                       /* this is needed to prevent random crashes with in ED_armature_from_edit */
+                                       ebo->temp.p = NULL;
                                        ebo->flag &= ~BONE_CONNECTED;
                                }
                        }
@@ -623,7 +625,10 @@ static int separate_armature_exec(bContext *C, wmOperator *op)
                ED_armature_edit_free(obedit->data);
 
                /* 2) duplicate base */
-               newbase = ED_object_add_duplicate(bmain, scene, view_layer, oldbase, USER_DUP_ARM); /* only duplicate linked armature */
+
+               /* only duplicate linked armature */
+               newbase = ED_object_add_duplicate(bmain, scene, view_layer, oldbase, USER_DUP_ARM);
+
                DEG_relations_tag_update(bmain);
 
                newob = newbase->object;
index 275d00d..6409d54 100644 (file)
@@ -672,7 +672,9 @@ void ED_armature_from_edit(Main *bmain, bArmature *arm)
                if (eBone == arm->act_edbone) {
                        /* don't change active selection, this messes up separate which uses
                         * editmode toggle and can separate active bone which is de-selected originally */
-                       /* newBone->flag |= BONE_SELECTED; */ /* important, editbones can be active with only 1 point selected */
+
+                       /* important, editbones can be active with only 1 point selected */
+                       /* newBone->flag |= BONE_SELECTED; */
                        arm->act_bone = newBone;
                }
                newBone->roll = 0.0f;
@@ -707,8 +709,9 @@ void ED_armature_from_edit(Main *bmain, bArmature *arm)
        }
 
        /* Fix parenting in a separate pass to ensure ebone->bone connections are valid at this point.
-        * Do not set bone->head/tail here anymore, using EditBone data for that is not OK since our later fiddling
-        * with parent's arm_mat (for roll conversion) may have some small but visible impact on locations (T46010). */
+        * Do not set bone->head/tail here anymore,
+        * using EditBone data for that is not OK since our later fiddling with parent's arm_mat
+        * (for roll conversion) may have some small but visible impact on locations (T46010). */
        for (eBone = arm->edbo->first; eBone; eBone = eBone->next) {
                newBone = eBone->temp.bone;
                if (eBone->parent) {
index 9679930..b427843 100644 (file)
@@ -1466,7 +1466,8 @@ static void harmonic_coordinates_bind(MeshDeformModifierData *mmd, MeshDeformBin
                mdb->cagemesh_cache.mpoly = me->mpoly;
                mdb->cagemesh_cache.mloop = me->mloop;
                mdb->cagemesh_cache.looptri = BKE_mesh_runtime_looptri_ensure(me);
-               mdb->cagemesh_cache.poly_nors = CustomData_get_layer(&me->pdata, CD_NORMAL);  /* can be NULL */
+               /* can be NULL */
+               mdb->cagemesh_cache.poly_nors = CustomData_get_layer(&me->pdata, CD_NORMAL);
        }
 
        /* make bounding box equal size in all directions, add padding, and compute
index c8ce077..30f0b82 100644 (file)
@@ -498,7 +498,10 @@ static int poselib_add_exec(bContext *C, wmOperator *op)
        BLI_uniquename(&act->markers, marker, DATA_("Pose"), '.', offsetof(TimeMarker, name), sizeof(marker->name));
 
        /* use Keying Set to determine what to store for the pose */
-       ks = ANIM_builtin_keyingset_get_named(NULL, ANIM_KS_WHOLE_CHARACTER_SELECTED_ID); /* this includes custom props :)*/
+
+       /* this includes custom props :)*/
+       ks = ANIM_builtin_keyingset_get_named(NULL, ANIM_KS_WHOLE_CHARACTER_SELECTED_ID);
+
        ANIM_apply_keyingset(C, NULL, act, ks, MODIFYKEY_MODE_INSERT, (float)frame);
 
        /* store new 'active' pose number */
@@ -730,7 +733,8 @@ void POSELIB_OT_pose_rename(wmOperatorType *ot)
        ot->flag = OPTYPE_REGISTER | OPTYPE_UNDO;
 
        /* properties */
-       /* NOTE: name not pose is the operator's "main" property, so that it will get activated in the popup for easy renaming */
+       /* NOTE: name not pose is the operator's "main" property,
+        * so that it will get activated in the popup for easy renaming */
        ot->prop = RNA_def_string(ot->srna, "name", "RenamedPose", 64, "New Pose Name", "New name for pose");
        prop = RNA_def_enum(ot->srna, "pose", DummyRNA_NULL_items, 0, "Pose", "The pose to rename");
        RNA_def_enum_funcs(prop, poselib_stored_pose_itemf);
@@ -963,7 +967,8 @@ static void poselib_backup_restore(tPoseLib_PreviewData *pld)
                if (plb->oldprops)
                        IDP_SyncGroupValues(plb->pchan->prop, plb->oldprops);
 
-               /* TODO: constraints settings aren't restored yet, even though these could change (though not that likely) */
+               /* TODO: constraints settings aren't restored yet,
+                * even though these could change (though not that likely) */
        }
 }
 
@@ -1785,7 +1790,11 @@ void POSELIB_OT_browse_interactive(wmOperatorType *ot)
 
        // XXX: percentage vs factor?
        /* not used yet */
-       /* RNA_def_float_factor(ot->srna, "blend_factor", 1.0f, 0.0f, 1.0f, "Blend Factor", "Amount that the pose is applied on top of the existing poses", 0.0f, 1.0f); */
+#if 0
+       RNA_def_float_factor(
+               ot->srna, "blend_factor", 1.0f, 0.0f, 1.0f, "Blend Factor",
+               "Amount that the pose is applied on top of the existing poses", 0.0f, 1.0f);
+#endif
 }
 
 void POSELIB_OT_apply_pose(wmOperatorType *ot)
index 184c09f..1916495 100644 (file)
@@ -239,7 +239,8 @@ bool ED_armature_pose_select_pick_with_buffer(
 }
 
 /* 'select_mode' is usual SEL_SELECT/SEL_DESELECT/SEL_TOGGLE/SEL_INVERT.
- * When true, 'ignore_visibility' makes this func also affect invisible bones (hidden or on hidden layers). */
+ * When true, 'ignore_visibility' makes this func also affect invisible bones
+ * (hidden or on hidden layers). */
 bool ED_pose_deselect_all(Object *ob, int select_mode, const bool ignore_visibility)
 {
        bArmature *arm = ob->data;
index 52e580a..7ba22fc 100644 (file)
 
 /* Temporary data shared between these operators */
 typedef struct tPoseSlideOp {
-       Scene *scene;       /* current scene */
-       ScrArea *sa;        /* area that we're operating in (needed for modal()) */
-       ARegion *ar;        /* region that we're operating in (needed for modal()) */
-       uint objects_len;   /* len of the PoseSlideObject array. */
-
-       ListBase pfLinks;   /* links between posechannels and f-curves for all the pose objects. */
-       DLRBT_Tree keys;    /* binary tree for quicker searching for keyframes (when applicable) */
+       /** current scene */
+       Scene *scene;
+       /** area that we're operating in (needed for modal()) */
+       ScrArea *sa;
+       /** region that we're operating in (needed for modal()) */
+       ARegion *ar;
+       /** len of the PoseSlideObject array. */
+       uint objects_len;
+
+       /** links between posechannels and f-curves for all the pose objects. */
+       ListBase pfLinks;
+       /** binary tree for quicker searching for keyframes (when applicable) */
+       DLRBT_Tree keys;
 
-       int cframe;         /* current frame number - global time */
+       /** current frame number - global time */
+       int cframe;
 
-       int prevFrame;      /* frame before current frame (blend-from) - global time */
-       int nextFrame;      /* frame after current frame (blend-to)    - global time */
+       /** frame before current frame (blend-from) - global time */
+       int prevFrame;
+       /** frame after current frame (blend-to)    - global time */
+       int nextFrame;
 
-       short mode;         /* sliding mode (ePoseSlide_Modes) */
-       short flag;         /* unused for now, but can later get used for storing runtime settings.... */
+       /** sliding mode (ePoseSlide_Modes) */
+       short mode;
+       /** unused for now, but can later get used for storing runtime settings.... */
+       short flag;
 
-       short channels;     /* which transforms/channels are affected (ePoseSlide_Channels) */
-       short axislock;     /* axis-limits for transforms (ePoseSlide_AxisLock) */
+       /** which transforms/channels are affected (ePoseSlide_Channels) */
+       short channels;
+       /** axis-limits for transforms (ePoseSlide_AxisLock) */
+       short axislock;
 
-       float percentage;   /* 0-1 value for determining the influence of whatever is relevant */
+       /** 0-1 value for determining the influence of whatever is relevant */
+       float percentage;
 
-       NumInput num;       /* numeric input */
+       /** numeric input */
+       NumInput num;
 
        struct tPoseSlideObject *ob_data_array;
 } tPoseSlideOp;
@@ -361,7 +376,8 @@ static void pose_slide_apply_val(
                         * - numerator should be larger than denominator to 'expand' the result
                         * - perform this weighting a number of times given by the percentage...
                         */
-                       int iters = (int)ceil(10.0f * pso->percentage); /* TODO: maybe a sensitivity ctrl on top of this is needed */
+                       /* TODO: maybe a sensitivity ctrl on top of this is needed */
+                       int iters = (int)ceil(10.0f * pso->percentage);
 
                        while (iters-- > 0) {
                                (*val) = (-((sVal * w2) + (eVal * w1)) + ((*val) * 6.0f) ) / 5.0f;
@@ -374,7 +390,8 @@ static void pose_slide_apply_val(
                         * - numerator should be smaller than denominator to 'relax' the result
                         * - perform this weighting a number of times given by the percentage...
                         */
-                       int iters = (int)ceil(10.0f * pso->percentage); /* TODO: maybe a sensitivity ctrl on top of this is needed */
+                       /* TODO: maybe a sensitivity ctrl on top of this is needed */
+                       int iters = (int)ceil(10.0f * pso->percentage);
 
                        while (iters-- > 0) {
                                (*val) = ( ((sVal * w2) + (eVal * w1)) + ((*val) * 5.0f) ) / 6.0f;
@@ -579,7 +596,8 @@ static void pose_slide_apply_quat(tPoseSlideOp *pso, tPChanFCurveLink *pfl)
                }
                else {
                        float quat_interp[4], quat_orig[4];
-                       int iters = (int)ceil(10.0f * pso->percentage); /* TODO: maybe a sensitivity ctrl on top of this is needed */
+                       /* TODO: maybe a sensitivity ctrl on top of this is needed */
+                       int iters = (int)ceil(10.0f * pso->percentage);
 
                        /* perform this blending several times until a satisfactory result is reached */
                        while (iters-- > 0) {
index ffbe8b5..ad48532 100644 (file)
@@ -72,7 +72,8 @@
 /* ********************************************** */
 /* Pose Apply */
 
-/* helper for apply_armature_pose2bones - fixes parenting of objects that are bone-parented to armature */
+/* helper for apply_armature_pose2bones - fixes parenting of objects
+ * that are bone-parented to armature */
 static void applyarmature_fix_boneparents(const bContext *C, Scene *scene, Object *armob)
 {
        Depsgraph *depsgraph = CTX_data_depsgraph(C);
@@ -100,7 +101,8 @@ static int apply_armature_pose2bones_exec(bContext *C, wmOperator *op)
        Main *bmain = CTX_data_main(C);
        Depsgraph *depsgraph = CTX_data_depsgraph(C);
        Scene *scene = CTX_data_scene(C);
-       Object *ob = BKE_object_pose_armature_get(CTX_data_active_object(C)); // must be active object, not edit-object
+       // must be active object, not edit-object
+       Object *ob = BKE_object_pose_armature_get(CTX_data_active_object(C));
        const Object *ob_eval = DEG_get_evaluated_object(depsgraph, ob);
        bArmature *arm = BKE_armature_from_object(ob);
        bPose *pose;
@@ -111,7 +113,7 @@ static int apply_armature_pose2bones_exec(bContext *C, wmOperator *op)
        if (ob->type != OB_ARMATURE)
                return OPERATOR_CANCELLED;
        if (BKE_object_obdata_is_libdata(ob)) {
-               BKE_report(op->reports, RPT_ERROR, "Cannot apply pose to lib-linked armature"); /* error_libdata(); */
+               BKE_report(op->reports, RPT_ERROR, "Cannot apply pose to lib-linked armature");
                return OPERATOR_CANCELLED;
        }
 
@@ -745,7 +747,8 @@ static void pchan_clear_rot(bPoseChannel *pchan)
                }
        }
 
-       /* Clear also Bendy Bone stuff - Roll is obvious, but Curve X/Y stuff is also kindof rotational in nature... */
+       /* Clear also Bendy Bone stuff - Roll is obvious,
+        * but Curve X/Y stuff is also kindof rotational in nature... */
        pchan->roll1 = 0.0f;
        pchan->roll2 = 0.0f;
 
index 051940d..90208c4 100644 (file)
@@ -244,7 +244,8 @@ void poseAnim_mapping_refresh(bContext *C, Scene *scene, Object *ob)
        else
                BKE_pose_where_is(depsgraph, scene, ob);
 
-       DEG_id_tag_update(&ob->id, ID_RECALC_COPY_ON_WRITE); /* otherwise animation doesn't get updated */
+       /* otherwise animation doesn't get updated */
+       DEG_id_tag_update(&ob->id, ID_RECALC_COPY_ON_WRITE);
        WM_event_add_notifier(C, NC_OBJECT | ND_POSE, ob);
 }
 
index 84f8f17..df9527e 100644 (file)
@@ -160,8 +160,9 @@ static void init_editNurb_keyIndex(EditNurb *editnurb, ListBase *origBase)
                        pt_index = 0;
                        while (a--) {
                                /* We cannot keep *any* reference to curve obdata,
-                                * it might be replaced and freed while editcurve remain in use (in viewport render case e.g.).
-                                * Note that we could use a pool to avoid lots of malloc's here, but... not really a problem for now. */
+                                * it might be replaced and freed while editcurve remain in use
+                                * (in viewport render case e.g.). Note that we could use a pool to avoid
+                                * lots of malloc's here, but... not really a problem for now. */
                                BezTriple *origbezt_cpy = MEM_mallocN(sizeof(*origbezt), __func__);
                                *origbezt_cpy = *origbezt;
                                keyIndex = init_cvKeyIndex(origbezt_cpy, key_index, nu_index, pt_index, vertex_index);
@@ -180,8 +181,9 @@ static void init_editNurb_keyIndex(EditNurb *editnurb, ListBase *origBase)
                        pt_index = 0;
                        while (a--) {
                                /* We cannot keep *any* reference to curve obdata,
-                                * it might be replaced and freed while editcurve remain in use (in viewport render case e.g.).
-                                * Note that we could use a pool to avoid lots of malloc's here, but... not really a problem for now. */
+                                * it might be replaced and freed while editcurve remain in use
+                                * (in viewport render case e.g.). Note that we could use a pool to avoid
+                                * lots of malloc's here, but... not really a problem for now. */
                                BPoint *origbp_cpy = MEM_mallocN(sizeof(*origbp_cpy), __func__);
                                *origbp_cpy = *origbp;
                                keyIndex = init_cvKeyIndex(origbp_cpy, key_index, nu_index, pt_index, vertex_index);
@@ -1272,8 +1274,9 @@ void ED_curve_editnurb_make(Object *obedit)
 
                if (actkey) {
                        editnurb->shapenr = obedit->shapenr;
-                       /* Apply shapekey to new nurbs of editnurb, not those of original curve (and *after* we generated keyIndex),
-                        * else we do not have valid 'original' data to properly restore curve when leaving editmode. */
+                       /* Apply shapekey to new nurbs of editnurb, not those of original curve
+                        * (and *after* we generated keyIndex), else we do not have valid 'original' data
+                        * to properly restore curve when leaving editmode. */
                        BKE_keyblock_convert_to_curve(actkey, cu, &editnurb->nurbs);
                }
        }
@@ -1336,7 +1339,8 @@ static int separate_exec(bContext *C, wmOperator *op)
                }
 
                /* 2. Duplicate the object and data. */
-               newbase = ED_object_add_duplicate(bmain, scene, view_layer, oldbase, 0); /* 0 = fully linked. */
+               newbase = ED_object_add_duplicate(
+                       bmain, scene, view_layer, oldbase, 0); /* 0 = fully linked. */
                DEG_relations_tag_update(bmain);
 
                newob = newbase->object;
@@ -2225,7 +2229,8 @@ static void adduplicateflagNurb(Object *obedit, View3D *v3d, ListBase *newnurb,
                                                                        newu++;
                                                                        for (c = a / nu->pntsu, bp3 = bp2; c < nu->pntsv; c++, bp3 += nu->pntsu) {
                                                                                if (bp3->f1 & flag) {
-                                                                                       bp3->f1 |= SURF_SEEN; /* flag as seen so skipped on future iterations */
+                                                                                       /* flag as seen so skipped on future iterations */
+                                                                                       bp3->f1 |= SURF_SEEN;
                                                                                        if (newu == 1) newv++;
                                                                                }
                                                                                else {
@@ -2722,7 +2727,8 @@ static void curve_smooth_value(ListBase *editnurb,
                        for (last_sel = 0; last_sel < nu->pntsu; last_sel++) {
                                /* loop over selection segments of a curve, smooth each */
 
-                               /* Start BezTriple code, this is duplicated below for points, make sure these functions stay in sync */
+                               /* Start BezTriple code,
+                                * this is duplicated below for points, make sure these functions stay in sync */
                                start_sel = -1;
                                for (bezt = &nu->bezt[last_sel], a = last_sel; a < nu->pntsu; a++, bezt++) {
                                        if (bezt->f2 & SELECT) {
@@ -2792,7 +2798,8 @@ static void curve_smooth_value(ListBase *editnurb,
                        for (last_sel = 0; last_sel < nu->pntsu; last_sel++) {
                                /* loop over selection segments of a curve, smooth each */
 
-                               /* Start BezTriple code, this is duplicated below for points, make sure these functions stay in sync */
+                               /* Start BezTriple code,
+                                * this is duplicated below for points, make sure these functions stay in sync */
                                start_sel = -1;
                                for (bp = &nu->bp[last_sel], a = last_sel; a < nu->pntsu; a++, bp++) {
                                        if (bp->f1 & SELECT) {
@@ -5499,7 +5506,8 @@ static bool curve_toggle_cyclic(View3D *v3d, ListBase *editnurb, int direction)
                                        while (a--) {
                                                if (bp->f1 & SELECT) {
                                                        nu->flagu ^= CU_NURB_CYCLIC;
-                                                       BKE_nurb_knot_calc_u(nu);   /* 1==u  type is ignored for cyclic curves */
+                                                       /* 1==u  type is ignored for cyclic curves */
+                                                       BKE_nurb_knot_calc_u(nu);
                                                        changed = true;
                                                        break;
                                                }
@@ -5515,12 +5523,14 @@ static bool curve_toggle_cyclic(View3D *v3d, ListBase *editnurb, int direction)
                                        if (bp->f1 & SELECT) {
                                                if (direction == 0 && nu->pntsu > 1) {
                                                        nu->flagu ^= CU_NURB_CYCLIC;
-                                                       BKE_nurb_knot_calc_u(nu);   /* 1==u  type is ignored for cyclic curves */
+                                                       /* 1==u  type is ignored for cyclic curves */
+                                                       BKE_nurb_knot_calc_u(nu);
                                                        changed = true;
                                                }
                                                if (direction == 1 && nu->pntsv > 1) {
                                                        nu->flagv ^= CU_NURB_CYCLIC;
-                                                       BKE_nurb_knot_calc_v(nu);   /* 2==v  type is ignored for cyclic curves */
+                                                       /* 2==v  type is ignored for cyclic curves */
+                                                       BKE_nurb_knot_calc_v(nu);
                                                        changed = true;
                                                }
                                                break;
index 5308660..3f53fe8 100644 (file)
@@ -343,7 +343,7 @@ Nurb *ED_curve_add_nurbs_primitive(bContext *C, Object *obedit, float mat[4][4],
                        break;
                case CU_PRIM_TUBE: /* Cylinder */
                        if (cutype == CU_NURBS) {
-                               nu = ED_curve_add_nurbs_primitive(C, obedit, mat, CU_NURBS | CU_PRIM_CIRCLE, 0); /* circle */
+                               nu = ED_curve_add_nurbs_primitive(C, obedit, mat, CU_NURBS | CU_PRIM_CIRCLE, 0);
                                nu->resolu = cu->resolu;
                                nu->flag = CU_SMOOTH;
                                BLI_addtail(editnurb, nu); /* temporal for extrude and translate */
@@ -420,7 +420,7 @@ Nurb *ED_curve_add_nurbs_primitive(bContext *C, Object *obedit, float mat[4][4],
                                float tmp_vec[3] = {0.f, 0.f, 1.f};
 
                                xzproj = 1;
-                               nu = ED_curve_add_nurbs_primitive(C, obedit, mat, CU_NURBS | CU_PRIM_CIRCLE, 0); /* circle */
+                               nu = ED_curve_add_nurbs_primitive(C, obedit, mat, CU_NURBS | CU_PRIM_CIRCLE, 0);
                                xzproj = 0;
                                nu->resolu = cu->resolu;
                                nu->resolv = cu->resolv;
index e674785..7c31527 100644 (file)
 /* ----- General Defines ------ */
 /* flags for sflag */
 typedef enum eDrawStrokeFlags {
-       GP_DRAWDATA_NOSTATUS    = (1 << 0),   /* don't draw status info */
-       GP_DRAWDATA_ONLY3D      = (1 << 1),   /* only draw 3d-strokes */
-       GP_DRAWDATA_ONLYV2D     = (1 << 2),   /* only draw 'canvas' strokes */
-       GP_DRAWDATA_ONLYI2D     = (1 << 3),   /* only draw 'image' strokes */
-       GP_DRAWDATA_IEDITHACK   = (1 << 4),   /* special hack for drawing strokes in Image Editor (weird coordinates) */
-       GP_DRAWDATA_NO_XRAY     = (1 << 5),   /* don't draw xray in 3D view (which is default) */
-       GP_DRAWDATA_NO_ONIONS   = (1 << 6),       /* no onionskins should be drawn (for animation playback) */
+       /** don't draw status info */
+       GP_DRAWDATA_NOSTATUS    = (1 << 0),
+       /** only draw 3d-strokes */
+       GP_DRAWDATA_ONLY3D      = (1 << 1),
+       /** only draw 'canvas' strokes */
+       GP_DRAWDATA_ONLYV2D     = (1 << 2),
+       /** only draw 'image' strokes */
+       GP_DRAWDATA_ONLYI2D     = (1 << 3),
+       /** special hack for drawing strokes in Image Editor (weird coordinates) */
+       GP_DRAWDATA_IEDITHACK   = (1 << 4),
+       /** don't draw xray in 3D view (which is default) */
+       GP_DRAWDATA_NO_XRAY     = (1 << 5),
+       /** no onionskins should be drawn (for animation playback) */
+       GP_DRAWDATA_NO_ONIONS   = (1 << 6),
 } eDrawStrokeFlags;
 
 
@@ -272,7 +279,8 @@ static void gp_draw_stroke_3d(
                 * Note: we want more visible levels of pressures when thickness is bigger.
                 */
                if (fabsf(pt->pressure - curpressure) > 0.2f / (float)thickness) {
-                       /* if the pressure changes before get at least 2 vertices, need to repeat last point to avoid assert in immEnd() */
+                       /* if the pressure changes before get at least 2 vertices,
+                        * need to repeat last point to avoid assert in immEnd() */
                        if (draw_points < 2) {
                                const bGPDspoint *pt2 = pt - 1;
                                immVertex3fv(pos, &pt2->x);
@@ -954,7 +962,8 @@ static void gp_draw_data_all(
                        gpd_source = (scene->gpd ? scene->gpd : NULL);
                }
                else if (spacetype == SPACE_CLIP && scene->clip) {
-                       /* currently drawing only gpencil data from either clip or track, but not both - XXX fix logic behind */
+                       /* currently drawing only gpencil data from either clip or track,
+                        * but not both - XXX fix logic behind */
                        gpd_source = (scene->clip->gpd ? scene->clip->gpd : NULL);
                }
 
index 7991c31..a5382eb 100644 (file)
@@ -2370,7 +2370,8 @@ void GPENCIL_OT_annotate(wmOperatorType *ot)
        prop = RNA_def_collection_runtime(ot->srna, "stroke", &RNA_OperatorStrokeElement, "Stroke", "");
        RNA_def_property_flag(prop, PROP_HIDDEN | PROP_SKIP_SAVE);
 
-       /* NOTE: wait for input is enabled by default, so that all UI code can work properly without needing users to know about this */
+       /* NOTE: wait for input is enabled by default,
+        * so that all UI code can work properly without needing users to know about this */
        prop = RNA_def_boolean(ot->srna, "wait_for_input", true, "Wait for Input", "Wait for first click instead of painting immediately");
        RNA_def_property_flag(prop, PROP_SKIP_SAVE);
 }
index 392c670..87c7017 100644 (file)
 /* ----- General Defines ------ */
 /* flags for sflag */
 typedef enum eDrawStrokeFlags {
-       GP_DRAWDATA_NOSTATUS    = (1 << 0),   /* don't draw status info */
-       GP_DRAWDATA_ONLY3D      = (1 << 1),   /* only draw 3d-strokes */
-       GP_DRAWDATA_ONLYV2D     = (1 << 2),   /* only draw 'canvas' strokes */
-       GP_DRAWDATA_ONLYI2D     = (1 << 3),   /* only draw 'image' strokes */
-       GP_DRAWDATA_IEDITHACK   = (1 << 4),   /* special hack for drawing strokes in Image Editor (weird coordinates) */
-       GP_DRAWDATA_NO_XRAY     = (1 << 5),   /* don't draw xray in 3D view (which is default) */
-       GP_DRAWDATA_NO_ONIONS   = (1 << 6),       /* no onionskins should be drawn (for animation playback) */
-       GP_DRAWDATA_VOLUMETRIC  = (1 << 7),   /* draw strokes as "volumetric" circular billboards */
-       GP_DRAWDATA_FILL        = (1 << 8)    /* fill insides/bounded-regions of strokes */
+       /** don't draw status info */
+       GP_DRAWDATA_NOSTATUS    = (1 << 0),
+       /** only draw 3d-strokes */
+       GP_DRAWDATA_ONLY3D      = (1 << 1),
+       /** only draw 'canvas' strokes */
+       GP_DRAWDATA_ONLYV2D     = (1 << 2),
+       /** only draw 'image' strokes */
+       GP_DRAWDATA_ONLYI2D     = (1 << 3),
+       /** special hack for drawing strokes in Image Editor (weird coordinates) */
+       GP_DRAWDATA_IEDITHACK   = (1 << 4),
+       /** don't draw xray in 3D view (which is default) */
+       GP_DRAWDATA_NO_XRAY     = (1 << 5),
+       /** no onionskins should be drawn (for animation playback) */
+       GP_DRAWDATA_NO_ONIONS   = (1 << 6),
+       /** draw strokes as "volumetric" circular billboards */
+       GP_DRAWDATA_VOLUMETRIC  = (1 << 7),
+       /** fill insides/bounded-regions of strokes */
+       GP_DRAWDATA_FILL        = (1 << 8)
 } eDrawStrokeFlags;
 
 
@@ -342,7 +351,10 @@ static void gp_draw_stroke_volumetric_buffer(
        const tGPspoint *pt = points;
        for (int i = 0; i < totpoints; i++, pt++) {
                gp_set_tpoint_varying_color(pt, ink, color);
-               immAttr1f(size, pt->pressure * thickness); /* TODO: scale based on view transform (zoom level) */
+
+               /* TODO: scale based on view transform (zoom level) */
+               immAttr1f(size, pt->pressure * thickness);
+
                immVertex2f(pos, pt->x, pt->y);
        }
 
@@ -403,8 +415,10 @@ static void gp_draw_stroke_volumetric_3d(
        const bGPDspoint *pt = points;
        for (int i = 0; i < totpoints && pt; i++, pt++) {
                gp_set_point_varying_color(pt, ink, color);
-               immAttr1f(size, pt->pressure * thickness); /* TODO: scale based on view transform */
-               immVertex3fv(pos, &pt->x);                   /* we can adjust size in vertex shader based on view/projection! */
+               /* TODO: scale based on view transform */
+               immAttr1f(size, pt->pressure * thickness);
+               /* we can adjust size in vertex shader based on view/projection! */
+               immVertex3fv(pos, &pt->x);
        }
 
        immEnd();
index 412330c..3f21a07 100644 (file)
@@ -707,7 +707,8 @@ static bool gp_brush_twist_apply(
                axis_angle_normalized_to_mat3(rmat, axis, angle);
 
                /* Rotate point (no matrix-space transforms needed, as GP points are in world space) */
-               sub_v3_v3v3(vec, &pt->x, gso->dvec); /* make relative to center (center is stored in dvec) */
+               sub_v3_v3v3(vec, &pt->x, gso->dvec); /* make relative to center
+                                                     * (center is stored in dvec) */
                mul_m3_v3(rmat, vec);
                add_v3_v3v3(&pt->x, vec, gso->dvec); /* restore */
 
index cb3fcba..a880e6c 100644 (file)
@@ -1129,7 +1129,7 @@ static int gp_camera_view_subrect(bContext *C, rctf *subrect)
                if (rv3d->persp == RV3D_CAMOB) {
                        Scene *scene = CTX_data_scene(C);
                        Depsgraph *depsgraph = CTX_data_depsgraph(C);
-                       ED_view3d_calc_camera_border(scene, depsgraph, ar, v3d, rv3d, subrect, true); /* no shift */
+                       ED_view3d_calc_camera_border(scene, depsgraph, ar, v3d, rv3d, subrect, true);
                        return 1;
                }
        }
@@ -1137,7 +1137,8 @@ static int gp_camera_view_subrect(bContext *C, rctf *subrect)
        return 0;
 }
 
-/* convert a given grease-pencil layer to a 3d-curve representation (using current view if appropriate) */
+/* convert a given grease-pencil layer to a 3d-curve representation
+ * (using current view if appropriate) */
 static void gp_layer_to_curve(
         bContext *C, ReportList *reports, bGPdata *gpd, bGPDlayer *gpl, const int mode,
         const bool norm_weights, const float rad_fac, const bool link_strokes, tGpTimingData *gtd)
@@ -1190,7 +1191,8 @@ static void gp_layer_to_curve(
                const bool add_start_point = (link_strokes && !(prev_gps));
                const bool add_end_point = (link_strokes && !(gps->next));
 
-               /* Detect new strokes created because of GP_STROKE_BUFFER_MAX reached, and stitch them to previous one. */
+               /* Detect new strokes created because of GP_STROKE_BUFFER_MAX reached,
+                * and stitch them to previous one. */
                bool stitch = false;
                if (prev_gps) {
                        bGPDspoint *pt1 = &prev_gps->points[prev_gps->totpoints - 1];
index 8afb0ca..33521d7 100644 (file)
@@ -112,7 +112,8 @@ static int gp_data_add_exec(bContext *C, wmOperator *op)
        }
        else {
                /* decrement user count and add new datablock */
-               /* TODO: if a datablock exists, we should make a copy of it instead of starting fresh (as in other areas) */
+               /* TODO: if a datablock exists,
+                * we should make a copy of it instead of starting fresh (as in other areas) */
                Main *bmain = CTX_data_main(C);
 
                /* decrement user count of old GP datablock */
@@ -1887,7 +1888,8 @@ static void joined_gpencil_fix_animdata_cb(ID *id, FCurve *fcu, void *user_data)
                        const char *old_name = BLI_ghashIterator_getKey(&gh_iter);
                        const char *new_name = BLI_ghashIterator_getValue(&gh_iter);
 
-                       /* only remap if changed; this still means there will be some waste if there aren't many drivers/keys */
+                       /* only remap if changed;
+                        * this still means there will be some waste if there aren't many drivers/keys */
                        if (!STREQ(old_name, new_name) && strstr(fcu->rna_path, old_name)) {
                                fcu->rna_path = BKE_animsys_fix_rna_path_rename(
                                        id, fcu->rna_path, "layers",
index 5c6f7de..a71545c 100644 (file)
@@ -629,7 +629,9 @@ static void gp_duplicate_points(const bGPDstroke *gps, ListBase *new_strokes, co
 
                                /* make a stupid copy first of the entire stroke (to get the flags too) */
                                gpsd = MEM_dupallocN(gps);
-                               BLI_strncpy(gpsd->runtime.tmp_layerinfo, layername, sizeof(gpsd->runtime.tmp_layerinfo)); /* saves original layer name */
+
+                               /* saves original layer name */
+                               BLI_strncpy(gpsd->runtime.tmp_layerinfo, layername, sizeof(gpsd->runtime.tmp_layerinfo));
 
                                /* initialize triangle memory - will be calculated on next redraw */
                                gpsd->triangles = NULL;
@@ -930,7 +932,8 @@ static int gp_strokes_copy_exec(bContext *C, wmOperator *op)
 
                                        /* make direct copies of the stroke and its points */
                                        gpsd = MEM_dupallocN(gps);
-                                       BLI_strncpy(gpsd->runtime.tmp_layerinfo, gpl->info, sizeof(gpsd->runtime.tmp_layerinfo)); /* saves original layer name */
+                                       /* saves original layer name */
+                                       BLI_strncpy(gpsd->runtime.tmp_layerinfo, gpl->info, sizeof(gpsd->runtime.tmp_layerinfo));
                                        gpsd->points = MEM_dupallocN(gps->points);
                                        if (gps->dvert != NULL) {
                                                gpsd->dvert = MEM_dupallocN(gps->dvert);
@@ -3061,13 +3064,15 @@ static int gp_strokes_reproject_exec(bContext *C, wmOperator *op)
                                }
                                else {
                                        /* Geometry - Snap to surfaces of visible geometry */
-                                       /* XXX: There will be precision loss (possible stairstep artifacts) from this conversion to satisfy the API's */
+                                       /* XXX: There will be precision loss (possible stairstep artifacts)
+                                        * from this conversion to satisfy the API's */
                                        const int screen_co[2] = {(int)xy[0], (int)xy[1]};
 
                                        int depth_margin = 0; // XXX: 4 for strokes, 0 for normal
                                        float depth;
 
-                                       /* XXX: The proper procedure computes the depths into an array, to have smooth transitions when all else fails... */
+                                       /* XXX: The proper procedure computes the depths into an array,
+                                        * to have smooth transitions when all else fails... */
                                        if (ED_view3d_autodist_depth(gsc.ar, screen_co, depth_margin, &depth)) {
                                                ED_view3d_autodist_simple(gsc.ar, screen_co, &pt->x, 0, &depth);
                                        }
index aa1a3a6..dceb1b4 100644 (file)
@@ -91,39 +91,67 @@ typedef struct tGPDfill {
        bContext *C;
        struct Main *bmain;
        struct Depsgraph *depsgraph;
-       struct wmWindow *win;               /* window where painting originated */
-       struct Scene *scene;                /* current scene from context */
-       struct Object *ob;                  /* current active gp object */
-       struct ScrArea *sa;                 /* area where painting originated */
-       struct RegionView3D *rv3d;          /* region where painting originated */
-       struct View3D *v3d;                 /* view3 where painting originated */
-       struct ARegion *ar;                 /* region where painting originated */
-       struct bGPdata *gpd;                /* current GP datablock */
-       struct Material *mat;               /* current material */
-       struct bGPDlayer *gpl;              /* layer */
-       struct bGPDframe *gpf;              /* frame */
-
-       short flag;                         /* flags */
-       short oldkey;                       /* avoid too fast events */
-       bool on_back;                       /* send to back stroke */
-
-       int center[2];                      /* mouse fill center position */
-       int sizex;                          /* windows width */
-       int sizey;                          /* window height */
-       int lock_axis;                      /* lock to viewport axis */
-
-       short fill_leak;                    /* number of pixel to consider the leak is too small (x 2) */
-       float fill_threshold;               /* factor for transparency */
-       int fill_simplylvl;                 /* number of simplify steps */
-       int fill_draw_mode;                 /* boundary limits drawing mode */
-
-       short sbuffer_size;                 /* number of elements currently in cache */
-       void *sbuffer;                      /* temporary points */
-       float *depth_arr;                   /* depth array for reproject */
-
-       Image *ima;                         /* temp image */
-       BLI_Stack *stack;                   /* temp points data */
-       void *draw_handle_3d;               /* handle for drawing strokes while operator is running 3d stuff */
+       /** window where painting originated */
+       struct wmWindow *win;
+       /** current scene from context */
+       struct Scene *scene;
+       /** current active gp object */
+       struct Object *ob;
+       /** area where painting originated */
+       struct ScrArea *sa;
+       /** region where painting originated */
+       struct RegionView3D *rv3d;
+       /** view3 where painting originated */
+       struct View3D *v3d;
+       /** region where painting originated */
+       struct ARegion *ar;
+       /** current GP datablock */
+       struct bGPdata *gpd;
+       /** current material */
+       struct Material *mat;
+       /** layer */
+       struct bGPDlayer *gpl;
+       /** frame */
+       struct bGPDframe *gpf;
+
+       /** flags */
+       short flag;
+       /** avoid too fast events */
+       short oldkey;
+       /** send to back stroke */
+       bool on_back;
+
+       /** mouse fill center position */
+       int center[2];
+       /** windows width */
+       int sizex;
+       /** window height */
+       int sizey;
+       /** lock to viewport axis */
+       int lock_axis;
+
+       /** number of pixel to consider the leak is too small (x 2) */
+       short fill_leak;
+       /** factor for transparency */
+       float fill_threshold;
+       /** number of simplify steps */
+       int fill_simplylvl;
+       /** boundary limits drawing mode */
+       int fill_draw_mode;
+
+       /** number of elements currently in cache */
+       short sbuffer_size;
+       /** temporary points */
+       void *sbuffer;
+       /** depth array for reproject */
+       float *depth_arr;
+
+       /** temp image */
+       Image *ima;
+       /** temp points data */
+       BLI_Stack *stack;
+       /** handle for drawing strokes while operator is running 3d stuff */
+       void *draw_handle_3d;
 } tGPDfill;
 
 
index 56ddf81..0ccc879 100644 (file)
@@ -107,79 +107,135 @@ typedef struct tGPDdraw {
 typedef struct tGPDinterpolate_layer {
        struct tGPDinterpolate_layer *next, *prev;
 
-       struct bGPDlayer *gpl;            /* layer */
-       struct bGPDframe *prevFrame;      /* frame before current frame (interpolate-from) */
-       struct bGPDframe *nextFrame;      /* frame after current frame (interpolate-to) */
-       struct bGPDframe *interFrame;     /* interpolated frame */
-       float factor;                     /* interpolate factor */
+       /** layer */
+       struct bGPDlayer *gpl;
+       /** frame before current frame (interpolate-from) */
+       struct bGPDframe *prevFrame;
+       /** frame after current frame (interpolate-to) */
+       struct bGPDframe *nextFrame;
+       /** interpolated frame */
+       struct bGPDframe *interFrame;
+       /** interpolate factor */
+       float factor;
 
 } tGPDinterpolate_layer;
 
 typedef struct tGPDinterpolate {
-       struct Scene *scene;       /* current scene from context */
-       struct ScrArea *sa;        /* area where painting originated */
-       struct ARegion *ar;        /* region where painting originated */
-       struct bGPdata *gpd;       /* current GP datablock */
-       struct Material *mat;      /* current material */
-
-       int cframe;                /* current frame number */
-       ListBase ilayers;          /* (tGPDinterpolate_layer) layers to be interpolated */
-       float shift;               /* value for determining the displacement influence */
-       float init_factor;         /* initial interpolation factor for active layer */
-       float low_limit;           /* shift low limit (-100%) */
-       float high_limit;          /* shift upper limit (200%) */
-       int flag;                  /* flag from toolsettings */
+       /** current scene from context */
+       struct Scene *scene;
+       /** area where painting originated */
+       struct ScrArea *sa;
+       /** region where painting originated */
+       struct ARegion *ar;
+       /** current GP datablock */
+       struct bGPdata *gpd;
+       /** current material */
+       struct Material *mat;
+
+       /** current frame number */
+       int cframe;
+       /** (tGPDinterpolate_layer) layers to be interpolated */
+       ListBase ilayers;
+       /** value for determining the displacement influence */
+       float shift;
+       /** initial interpolation factor for active layer */
+       float init_factor;
+       /** shift low limit (-100%) */
+       float low_limit;
+       /** shift upper limit (200%) */
+       float high_limit;
+       /** flag from toolsettings */
+       int flag;
 
        NumInput num;              /* numeric input */
-       void *draw_handle_3d;      /* handle for drawing strokes while operator is running 3d stuff */
-       void *draw_handle_screen;  /* handle for drawing strokes while operator is running screen stuff */
+       /** handle for drawing strokes while operator is running 3d stuff */
+       void *draw_handle_3d;
+       /** handle for drawing strokes while operator is running screen stuff */
+       void *draw_handle_screen;
 } tGPDinterpolate;
 
 
 /* Temporary primitive operation data */
 typedef struct tGPDprimitive {
-       struct Main *bmain;               /* main database pointer */
+       /** main database pointer */
+       struct Main *bmain;
        struct Depsgraph *depsgraph;
-       struct wmWindow *win;             /* window where painting originated */
-       struct Scene *scene;              /* current scene from context */
-       struct Object *ob;                /* current active gp object */
-       struct ScrArea *sa;               /* area where painting originated */
-       struct RegionView3D *rv3d;        /* region where painting originated */
-       struct View3D *v3d;               /* view3d where painting originated */
-       struct ARegion *ar;               /* region where painting originated */
-       struct bGPdata *gpd;              /* current GP datablock */
-       struct Material *mat;             /* current material */
-       struct Brush *brush;              /* current brush */
-
-       int cframe;                       /* current frame number */
-       struct bGPDlayer *gpl;            /* layer */
-       struct bGPDframe *gpf;            /* frame */
-       int type;                         /* type of primitive */
-       int orign_type;                   /* original type of primitive */
-       bool curve;                       /* type of primitive is a curve */
-       int brush_size;                   /* brush size */
-       float brush_strength;             /* brush strength */
-       short flip;                       /* flip option */
-       tGPspoint *points;                /* array of data-points for stroke */
-       int point_count;                  /* number of edges allocated */
-       int tot_stored_edges;             /* stored number of polygon edges */
-       int tot_edges;                    /* number of polygon edges */
-       float move[2];                    /* move distance */
-       float origin[2];                  /* initial box corner */
-       float start[2];                   /* first box corner */
-       float end[2];                     /* last box corner */
-       float midpoint[2];                /* midpoint box corner */
-       float cp1[2];                     /* first control point */
-       float cp2[2];                     /* second control point */
-       int sel_cp;                       /* flag to determine control point is selected */
-       int flag;                         /* flag to determine operations in progress */
-       float mval[2];                    /* recorded mouse-position */
-       float mvalo[2];                   /* previous recorded mouse-position */
-
-       int lock_axis;                    /* lock to viewport axis */
+       /** window where painting originated */
+       struct wmWindow *win;
+       /** current scene from context */
+       struct Scene *scene;
+       /** current active gp object */
+       struct Object *ob;
+       /** area where painting originated */
+       struct ScrArea *sa;
+       /** region where painting originated */
+       struct RegionView3D *rv3d;
+       /** view3d where painting originated */
+       struct View3D *v3d;
+       /** region where painting originated */
+       struct ARegion *ar;
+       /** current GP datablock */
+       struct bGPdata *gpd;
+       /** current material */
+       struct Material *mat;
+       /** current brush */
+       struct Brush *brush;
+
+       /** current frame number */
+       int cframe;
+       /** layer */
+       struct bGPDlayer *gpl;
+       /** frame */
+       struct bGPDframe *gpf;
+       /** type of primitive */
+       int type;
+       /** original type of primitive */
+       int orign_type;
+       /** type of primitive is a curve */
+       bool curve;
+       /** brush size */
+       int brush_size;
+       /** brush strength */
+       float brush_strength;
+       /** flip option */
+       short flip;
+       /** array of data-points for stroke */
+       tGPspoint *points;
+       /** number of edges allocated */
+       int point_count;
+       /** stored number of polygon edges */
+       int tot_stored_edges;
+       /** number of polygon edges */
+       int tot_edges;
+       /** move distance */
+       float move[2];
+       /** initial box corner */
+       float origin[2];
+       /** first box corner */
+       float start[2];
+       /** last box corner */
+       float end[2];
+       /** midpoint box corner */
+       float midpoint[2];
+       /** first control point */
+       float cp1[2];
+       /** second control point */
+       float cp2[2];
+       /** flag to determine control point is selected */
+       int sel_cp;
+       /** flag to determine operations in progress */
+       int flag;
+       /** recorded mouse-position */
+       float mval[2];
+       /** previous recorded mouse-position */
+       float mvalo[2];
+
+       /** lock to viewport axis */
+       int lock_axis;
        struct RNG *rng;
 
-       NumInput num;                     /* numeric input */
+       /** numeric input */
+       NumInput num;
 } tGPDprimitive;
 
 
index 86a8e34..4b1706f 100644 (file)
@@ -1112,7 +1112,8 @@ static int gpencil_interpolate_reverse_exec(bContext *C, wmOperator *UNUSED(op))
                                gpf = gpf->prev;
                        }
                        else {
-                               /* Not a breakdown (may be a key, or an extreme, or something else that wasn't generated)... stop */
+                               /* Not a breakdown (may be a key, or an extreme,
+                                * or something else that wasn't generated)... stop */
                                break;
                        }
                }
index baf8ca4..46b3d53 100644 (file)
@@ -2931,7 +2931,8 @@ static void gpencil_draw_apply_event(bContext *C, wmOperator *op, const wmEvent
        gpencil_draw_apply(C, op, p, depsgraph);
 
        /* force refresh */
-       ED_region_tag_redraw(p->ar); /* just active area for now, since doing whole screen is too slow */
+       /* just active area for now, since doing whole screen is too slow */
+       ED_region_tag_redraw(p->ar);
 }
 
 /* ------------------------------- */
@@ -3761,7 +3762,8 @@ void GPENCIL_OT_draw(wmOperatorType *ot)
        prop = RNA_def_collection_runtime(ot->srna, "stroke", &RNA_OperatorStrokeElement, "Stroke", "");
        RNA_def_property_flag(prop, PROP_HIDDEN | PROP_SKIP_SAVE);
 
-       /* NOTE: wait for input is enabled by default, so that all UI code can work properly without needing users to know about this */
+       /* NOTE: wait for input is enabled by default,
+        * so that all UI code can work properly without needing users to know about this */
        prop = RNA_def_boolean(ot->srna, "wait_for_input", true, "Wait for Input", "Wait for first click instead of painting immediately");
        RNA_def_property_flag(prop, PROP_HIDDEN | PROP_SKIP_SAVE);
 
index fb5efb0..7d0d71b 100644 (file)
@@ -998,7 +998,8 @@ static int gpencil_circle_select_exec(bContext *C, wmOperator *op)
        bool select = !RNA_boolean_get(op->ptr, "deselect");
 
        GP_SpaceConversion gsc = {NULL};
-       rcti rect = {0};            /* for bounding rect around circle (for quicky intersection testing) */
+       /* for bounding rect around circle (for quicky intersection testing) */
+       rcti rect = {0};
 
        bool changed = false;
 
index 36ae1a2..226a2bc 100644 (file)
@@ -140,7 +140,8 @@ bGPdata **ED_gpencil_data_get_pointers_direct(ID *screen_id, ScrArea *sa, Scene
 
                                /* return the GP data for the active node block/node */
                                if (snode && snode->nodetree) {
-                                       /* for now, as long as there's an active node tree, default to using that in the Nodes Editor */
+                                       /* for now, as long as there's an active node tree,
+                                        * default to using that in the Nodes Editor */
                                        if (r_ptr) RNA_id_pointer_create(&snode->nodetree->id, r_ptr);
                                        return &snode->nodetree->gpd;
                                }
@@ -546,7 +547,7 @@ void gp_point_conversion_init(bContext *C, GP_SpaceConversion *r_gsc)
 
                /* for camera view set the subrect */
                if (rv3d->persp == RV3D_CAMOB) {
-                       ED_view3d_calc_camera_border(scene, CTX_data_depsgraph(C), ar, v3d, rv3d, &r_gsc->subrect_data, true); /* no shift */
+                       ED_view3d_calc_camera_border(scene, CTX_data_depsgraph(C), ar, v3d, rv3d, &r_gsc->subrect_data, true);
                        r_gsc->subrect = &r_gsc->subrect_data;
                }
        }
@@ -1215,7 +1216,8 @@ void ED_gpencil_parent_location(
                                mul_m4_m4m4(diff_mat, tmp_mat, gpl->inverse);
                        }
                        else {
-                               mul_m4_m4m4(diff_mat, obparent_eval->obmat, gpl->inverse); /* if bone not found use object (armature) */
+                               /* if bone not found use object (armature) */
+                               mul_m4_m4m4(diff_mat, obparent_eval->obmat, gpl->inverse);
                        }
                        return;
                }
index bbc9f25..ec7c5f9 100644 (file)
@@ -69,29 +69,46 @@ struct PropertyRNA;
  * 'context' information
  */
 typedef struct bAnimContext {
-       void *data;             /* data to be filtered for use in animation editor */
-       short datatype;         /* type of data eAnimCont_Types */
-
-       short mode;             /* editor->mode */
-       short spacetype;        /* sa->spacetype */
-       short regiontype;       /* active region -> type (channels or main) */
-
-       struct ScrArea *sa;     /* editor host */
-       struct SpaceLink *sl;   /* editor data */
-       struct ARegion *ar;     /* region within editor */
-
-       struct bDopeSheet *ads; /* dopesheet data for editor (or which is being used) */
-
-       struct Depsgraph *depsgraph; /* active dependency graph */
-       struct Main *bmain;     /* Current Main */
-       struct Scene *scene;    /* active scene */
-       struct ViewLayer *view_layer; /* active scene layer */
-       struct Object *obact;   /* active object */
-       ListBase *markers;      /* active set of markers */
-
-       struct ReportList *reports; /* pointer to current reports list */
-
-       float yscale_fac;       /* scale factor for height of channels (i.e. based on the size of keyframes) */
+       /** data to be filtered for use in animation editor */
+       void *data;
+       /** type of data eAnimCont_Types */
+       short datatype;
+
+       /** editor->mode */
+       short mode;
+       /** sa->spacetype */
+       short spacetype;
+       /** active region -> type (channels or main) */
+       short regiontype;
+
+       /** editor host */
+       struct ScrArea *sa;
+       /** editor data */
+       struct SpaceLink *sl;
+       /** region within editor */
+       struct ARegion *ar;
+
+       /** dopesheet data for editor (or which is being used) */
+       struct bDopeSheet *ads;
+
+       /** active dependency graph */
+       struct Depsgraph *depsgraph;
+       /** Current Main */
+       struct Main *bmain;
+       /** active scene */
+       struct Scene *scene;
+       /** active scene layer */
+       struct ViewLayer *view_layer;
+       /** active object */
+       struct Object *obact;
+       /** active set of markers */
+       ListBase *markers;
+
+       /** pointer to current reports list */
+       struct ReportList *reports;
+
+       /** Scale factor for height of channels (i.e. based on the size of keyframes). */
+       float yscale_fac;
 } bAnimContext;
 
 /* Main Data container types */
@@ -117,31 +134,44 @@ typedef enum eAnimCont_Types {
 typedef struct bAnimListElem {
        struct bAnimListElem *next, *prev;
 
-       void    *data;          /* source data this elem represents */
-       int     type;           /* (eAnim_ChannelType) one of the ANIMTYPE_* values */
-       int     flag;           /* copy of elem's flags for quick access */
-       int     index;          /* for un-named data, the index of the data in its collection */
-
-       char    update;         /* (eAnim_Update_Flags)  tag the element for updating */
-       char    tag;            /* tag the included data. Temporary always */
-
-       short   datatype;       /* (eAnim_KeyType) type of motion data to expect */
-       void   *key_data;       /* motion data - mostly F-Curves, but can be other types too */
-
-
-       /* NOTE: id here is the "IdAdtTemplate"-style datablock (e.g. Object, Material, Texture, NodeTree)
-        *       from which evaluation of the RNA-paths takes place. It's used to figure out how deep
-        *       channels should be nested (e.g. for Textures/NodeTrees) in the tree, and allows property
-        *       lookups (e.g. for sliders and for inserting keyframes) to work. If we had instead used
-        *       bAction or something similar, none of this would be possible: although it's trivial
-        *       to use an IdAdtTemplate type to find the source action a channel (e.g. F-Curve) comes from
-        *       (i.e. in the AnimEditors, it *must* be the active action, as only that can be edited),
-        *       it's impossible to go the other way (i.e. one action may be used in multiple places).
+       /** source data this elem represents */
+       void    *data;
+       /** (eAnim_ChannelType) one of the ANIMTYPE_* values */
+       int     type;
+       /** copy of elem's flags for quick access */
+       int     flag;
+       /** for un-named data, the index of the data in its collection */
+       int     index;
+
+       /** (eAnim_Update_Flags)  tag the element for updating */
+       char    update;
+       /** tag the included data. Temporary always */
+       char    tag;
+
+       /** (eAnim_KeyType) type of motion data to expect */
+       short   datatype;
+       /** motion data - mostly F-Curves, but can be other types too */
+       void   *key_data;
+
+
+       /**
+        * \note
+        * id here is the "IdAdtTemplate"-style datablock (e.g. Object, Material, Texture, NodeTree)
+        * from which evaluation of the RNA-paths takes place. It's used to figure out how deep
+        * channels should be nested (e.g. for Textures/NodeTrees) in the tree, and allows property
+        * lookups (e.g. for sliders and for inserting keyframes) to work. If we had instead used
+        * bAction or something similar, none of this would be possible: although it's trivial
+        * to use an IdAdtTemplate type to find the source action a channel (e.g. F-Curve) comes from
+        * (i.e. in the AnimEditors, it *must* be the active action, as only that can be edited),
+        * it's impossible to go the other way (i.e. one action may be used in multiple places).
         */
-       struct ID *id;          /* ID block that channel is attached to */
-       struct AnimData *adt;   /* source of the animation data attached to ID block (for convenience) */
+       /** ID block that channel is attached to */
+       struct ID *id;
+       /** source of the animation data attached to ID block (for convenience) */
+       struct AnimData *adt;
 
-       /* For list element which corresponds to a f-curve, this is an ID which
+       /**
+        * For list element which corresponds to a f-curve, this is an ID which
         * owns the f-curve.
         *
         * For example, if the f-curve is coming from Action, this id will be set to
@@ -153,13 +183,18 @@ typedef struct bAnimListElem {
         * object. */
        struct ID *fcurve_owner_id;
 
-       void   *owner;          /* for per-element F-Curves (e.g. NLA Control Curves), the element that this represents (e.g. NlaStrip) */
+       /**
+        * for per-element F-Curves
+        * (e.g. NLA Control Curves), the element that this represents (e.g. NlaStrip) */
+       void   *owner;
 } bAnimListElem;
 
 
-/* Some types for easier type-testing
- * NOTE: need to keep the order of these synchronized with the channels define code
- *      which is used for drawing and handling channel lists for
+/**
+ * Some types for easier type-testing
+ *
+ * \note need to keep the order of these synchronized with the channels define code
+ * which is used for drawing and handling channel lists for.
  */
 typedef enum eAnim_ChannelType {
        ANIMTYPE_NONE = 0,
@@ -248,42 +283,46 @@ typedef enum eAnim_Update_Flags {
 
 /* filtering flags  - under what circumstances should a channel be returned */
 typedef enum eAnimFilter_Flags {
-       /* data which channel represents is fits the dopesheet filters (i.e. scene visibility criteria) */
+       /** data which channel represents is fits the dopesheet filters
+        * (i.e. scene visibility criteria) */
        // XXX: it's hard to think of any examples where this *ISN'T* the case... perhaps becomes implicit?
        ANIMFILTER_DATA_VISIBLE   = (1 << 0),
-       /* channel is visible within the channel-list hierarchy (i.e. F-Curves within Groups in ActEdit) */
+       /** channel is visible within the channel-list hierarchy
+        * (i.e. F-Curves within Groups in ActEdit) */
        ANIMFILTER_LIST_VISIBLE   = (1 << 1),
-       /* channel has specifically been tagged as visible in Graph Editor (* Graph Editor Only) */
+       /** channel has specifically been tagged as visible in Graph Editor (* Graph Editor Only) */
        ANIMFILTER_CURVE_VISIBLE  = (1 << 2),
 
-       /* include summary channels and "expanders" (for drawing/mouse-selection in channel list) */
+       /** include summary channels and "expanders" (for drawing/mouse-selection in channel list) */
        ANIMFILTER_LIST_CHANNELS  = (1 << 3),
 
-       /* for its type, channel should be "active" one */
+       /** for its type, channel should be "active" one */
        ANIMFILTER_ACTIVE         = (1 << 4),
-       /* channel is a child of the active group (* Actions speciality) */
+       /** channel is a child of the active group (* Actions speciality) */
        ANIMFILTER_ACTGROUPED     = (1 << 5),
 
-       /* channel must be selected/not-selected, but both must not be set together */
+       /** channel must be selected/not-selected, but both must not be set together */
        ANIMFILTER_SEL            = (1 << 6),
        ANIMFILTER_UNSEL          = (1 << 7),
 
-       /* editability status - must be editable to be included */
+       /** editability status - must be editable to be included */
        ANIMFILTER_FOREDIT        = (1 << 8),
-       /* only selected animchannels should be considerable as editable - mainly for Graph Editor's option for keys on select curves only */
+       /** only selected animchannels should be considerable as editable - mainly
+        * for Graph Editor's option for keys on select curves only */
        ANIMFILTER_SELEDIT        = (1 << 9),
 
-       /* flags used to enforce certain data types */
-       // NOTE: the ones for curves and NLA tracks were redundant and have been removed for now...
+       /** flags used to enforce certain data types
+        * \node the ones for curves and NLA tracks were redundant and have been removed for now...
+        */
        ANIMFILTER_ANIMDATA       = (1 << 10),
 
-       /* duplicate entries for animation data attached to multi-user blocks must not occur */
+       /** duplicate entries for animation data attached to multi-user blocks must not occur */
        ANIMFILTER_NODUPLIS       = (1 << 11),
 
-       /* for checking if we should keep some collapsed channel around (internal use only!) */
+       /** for checking if we should keep some collapsed channel around (internal use only!) */
        ANIMFILTER_TMP_PEEK       = (1 << 30),
 
-       /* ignore ONLYSEL flag from filterflag, (internal use only!) */
+       /** ignore ONLYSEL flag from filterflag, (internal use only!) */
        ANIMFILTER_TMP_IGNORE_ONLYSEL = (1u << 31)
 } eAnimFilter_Flags;
 
@@ -429,30 +468,42 @@ void ANIM_animdata_freelist(ListBase *anim_data);
 
 /* role or level of animchannel in the hierarchy */
 typedef enum eAnimChannel_Role {
-       ACHANNEL_ROLE_EXPANDER = -1,    /* datablock expander - a "composite" channel type */
-       ACHANNEL_ROLE_SPECIAL  = 0,     /* special purposes - not generally for hierarchy processing */
-       ACHANNEL_ROLE_CHANNEL  = 1      /* data channel - a channel representing one of the actual building blocks of channels */
+       /** datablock expander - a "composite" channel type */
+       ACHANNEL_ROLE_EXPANDER = -1,
+       /** special purposes - not generally for hierarchy processing */
+       ACHANNEL_ROLE_SPECIAL  = 0,
+       /** data channel - a channel representing one of the actual building blocks of channels */
+       ACHANNEL_ROLE_CHANNEL  = 1
 } eAnimChannel_Role;
 
 /* flag-setting behavior */
 typedef enum eAnimChannels_SetFlag {
-       ACHANNEL_SETFLAG_CLEAR  = 0,     /* turn off */
-       ACHANNEL_SETFLAG_ADD    = 1,     /* turn on */
-       ACHANNEL_SETFLAG_INVERT = 2,     /* on->off, off->on */
-       ACHANNEL_SETFLAG_TOGGLE = 3      /* some on -> all off // all on */
+       /** turn off */
+       ACHANNEL_SETFLAG_CLEAR  = 0,
+       /** turn on */
+       ACHANNEL_SETFLAG_ADD    = 1,
+       /** on->off, off->on */
+       ACHANNEL_SETFLAG_INVERT = 2,
+       /** some on -> all off // all on */
+       ACHANNEL_SETFLAG_TOGGLE = 3
 } eAnimChannels_SetFlag;
 
 /* types of settings for AnimChannels */
 typedef enum eAnimChannel_Settings {
        ACHANNEL_SETTING_SELECT   = 0,
-       ACHANNEL_SETTING_PROTECT  = 1, /* warning: for drawing UI's, need to check if this is off (maybe inverse this later) */
+       /** warning: for drawing UI's, need to check if this is off (maybe inverse this later) */
+       ACHANNEL_SETTING_PROTECT  = 1,
        ACHANNEL_SETTING_MUTE     = 2,
        ACHANNEL_SETTING_EXPAND   = 3,
-       ACHANNEL_SETTING_VISIBLE  = 4,  /* only for Graph Editor */
-       ACHANNEL_SETTING_SOLO     = 5,  /* only for NLA Tracks */
-       ACHANNEL_SETTING_PINNED   = 6,  /* only for NLA Actions */
+       /** only for Graph Editor */
+       ACHANNEL_SETTING_VISIBLE  = 4,
+       /** only for NLA Tracks */
+       ACHANNEL_SETTING_SOLO     = 5,
+       /** only for NLA Actions */
+       ACHANNEL_SETTING_PINNED   = 6,
        ACHANNEL_SETTING_MOD_OFF  = 7,
-       ACHANNEL_SETTING_ALWAYS_VISIBLE = 8,  /* channel is pinned and always visible */
+       /** channel is pinned and always visible */
+       ACHANNEL_SETTING_ALWAYS_VISIBLE = 8,
 } eAnimChannel_Settings;
 
 
@@ -542,7 +593,8 @@ void ANIM_deselect_anim_channels(bAnimContext *ac, void *data, eAnimCont_Types d
 void ANIM_set_active_channel(bAnimContext *ac, void *data, eAnimCont_Types datatype, eAnimFilter_Flags filter, void *channel_data, eAnim_ChannelType channel_type);
 
 
-/* Delete the F-Curve from the given AnimData block (if possible), as appropriate according to animation context */
+/* Delete the F-Curve from the given AnimData block (if possible),
+ * as appropriate according to animation context */
 void ANIM_fcurve_delete_from_animdata(bAnimContext *ac, struct AnimData *adt, struct FCurve *fcu);
 
 /* ************************************************ */
index 3306fa0..bce04f4 100644 (file)
@@ -58,26 +58,32 @@ struct UndoType;
 
 typedef struct EditBone {
        struct EditBone *next, *prev;
-       struct IDProperty *prop; /* User-Defined Properties on this Bone */
-       struct EditBone *parent; /* Editbones have a one-way link  (i.e. children refer
-                                 * to parents.  This is converted to a two-way link for
-                                 * normal bones when leaving editmode. */
-       char    name[64];       /* MAXBONENAME */
-       float   roll;           /* Roll along axis.  We'll ultimately use the axis/angle method
-                                * for determining the transformation matrix of the bone.  The axis
-                                * is tail-head while roll provides the angle. Refer to Graphics
-                                * Gems 1 p. 466 (section IX.6) if it's not already in here somewhere*/
-
-       float head[3];          /* Orientation and length is implicit during editing */
+       /** User-Defined Properties on this Bone */
+       struct IDProperty *prop;
+       /** Editbones have a one-way link  (i.e. children refer
+        * to parents.  This is converted to a two-way link for
+        * normal bones when leaving editmode. */
+       struct EditBone *parent;
+       /** (64 == MAXBONENAME) */
+       char    name[64];
+       /** Roll along axis.  We'll ultimately use the axis/angle method
+        * for determining the transformation matrix of the bone.  The axis
+        * is tail-head while roll provides the angle. Refer to Graphics
+        * Gems 1 p. 466 (section IX.6) if it's not already in here somewhere*/
+       float   roll;
+
+       /** Orientation and length is implicit during editing */
+       float head[3];
        float tail[3];
-       /* All joints are considered to have zero rotation with respect to
+       /** All joints are considered to have zero rotation with respect to
         * their parents. Therefore any rotations specified during the
         * animation are automatically relative to the bones' rest positions*/
        int flag;
        int layer;
 
        float dist, weight;
-       float xwidth, length, zwidth;  /* put them in order! transform uses this as scale */
+       /** put them in order! transform uses this as scale */
+       float xwidth, length, zwidth;
        float rad_head, rad_tail;
 
        /* Bendy-Bone parameters */
@@ -87,22 +93,28 @@ typedef struct EditBone {
        float ease1, ease2;
        float scaleIn, scaleOut;
 
-       float oldlength;        /* for envelope scaling */
+       /** for envelope scaling */
+       float oldlength;
 
        short segments;
 
-       char bbone_prev_type;   /* Type of next/prev bone handles */
+       /** Type of next/prev bone handles */
+       char bbone_prev_type;
        char bbone_next_type;
-       struct EditBone *bbone_prev;    /* Next/prev bones to use as handle references when calculating bbones (optional) */
+       /** Next/prev bones to use as handle references when calculating bbones (optional) */
+       struct EditBone *bbone_prev;
        struct EditBone *bbone_next;
 
        /* Used for display */
-       float disp_mat[4][4];  /*  in Armature space, rest pos matrix */
-       float disp_tail_mat[4][4];  /*  in Armature space, rest pos matrix */
-       /* 32 == MAX_BBONE_SUBDIV */
-       float disp_bbone_mat[32][4][4]; /*  in Armature space, rest pos matrix */
-
-       struct EditBone *bbone_child;   /* connected child temporary during drawing */
+       /** in Armature space, rest pos matrix */
+       float disp_mat[4][4];
+       /** in Armature space, rest pos matrix */
+       float disp_tail_mat[4][4];
+       /** in Armature space, rest pos matrix (32 == MAX_BBONE_SUBDIV) */
+       float disp_bbone_mat[32][4][4];
+
+       /** connected child temporary during drawing */
+       struct EditBone *bbone_child;
 
        /* Used to store temporary data */
        union {
index 64739d9..aa54005 100644 (file)
@@ -134,8 +134,10 @@ typedef enum FSMenuCategory {
 typedef enum FSMenuInsert {
        FS_INSERT_SORTED = (1 << 0),
        FS_INSERT_SAVE   = (1 << 1),
-       FS_INSERT_FIRST  = (1 << 2),  /* moves the item to the front of the list when its not already there */
-       FS_INSERT_LAST   = (1 << 3),  /* just append to preseve delivered order */
+       /** moves the item to the front of the list when its not already there */
+       FS_INSERT_FIRST  = (1 << 2),
+       /** just append to preseve delivered order */
+       FS_INSERT_LAST   = (1 << 3),
 } FSMenuInsert;
 
 struct FSMenu;
index 87c6c0b..9fce191 100644 (file)
@@ -67,14 +67,20 @@ typedef struct ActKeyColumn {
        struct ActKeyColumn *next, *prev;
 
        /* sorting-tree linkage */
-       struct ActKeyColumn *left, *right;  /* 'children' of this node, less than and greater than it (respectively) */
-       struct ActKeyColumn *parent;        /* parent of this node in the tree */
-       char tree_col;                      /* DLRB_BLACK or DLRB_RED */
+       /** 'children' of this node, less than and greater than it (respectively) */
+       struct ActKeyColumn *left, *right;
+       /** parent of this node in the tree */
+       struct ActKeyColumn *parent;
+       /** DLRB_BLACK or DLRB_RED */
+       char tree_col;
 
        /* keyframe info */
-       char key_type;                      /* eBezTripe_KeyframeType */
-       char handle_type;                   /* eKeyframeHandleDrawOpts */
-       char extreme_type;                  /* eKeyframeExtremeDrawOpts */
+       /** eBezTripe_KeyframeType */
+       char key_type;
+       /** eKeyframeHandleDrawOpts */
+       char handle_type;
+       /** eKeyframeExtremeDrawOpts */
+       char extreme_type;
        short sel;
        float cfra;
 
@@ -134,7 +140,8 @@ typedef enum eKeyframeExtremeDrawOpts {
 } eKeyframeExtremeDrawOpts;
 
 /* draw simple diamond-shape keyframe */
-/* caller should set up vertex format, bind GPU_SHADER_KEYFRAME_DIAMOND, immBegin(GPU_PRIM_POINTS, n), then call this n times */
+/* caller should set up vertex format, bind GPU_SHADER_KEYFRAME_DIAMOND,
+ * immBegin(GPU_PRIM_POINTS, n), then call this n times */
 void draw_keyframe_shape(float x, float y, float size, bool sel, short key_type, short mode, float alpha,
                          unsigned int pos_id, unsigned int size_id, unsigned int color_id, unsigned int outline_color_id,
                          unsigned int linemask_id, short ipo_type, short extreme_type);
index e7ea5cf..8a398c9 100644 (file)
@@ -155,20 +155,30 @@ typedef enum eKeyframeIterFlags {
 
 typedef struct KeyframeEditData {
        /* generic properties/data access */
-       ListBase list;              /* temp list for storing custom list of data to check */
-       struct Scene *scene;        /* pointer to current scene - many tools need access to cfra/etc.  */
-       void *data;                 /* pointer to custom data - usually 'Object' but also 'rectf', but could be other types too */
-       float f1, f2;               /* storage of times/values as 'decimals' */
-       int i1, i2;                 /* storage of times/values/flags as 'whole' numbers */
+       /** temp list for storing custom list of data to check */
+       ListBase list;
+       /** pointer to current scene - many tools need access to cfra/etc.  */
+       struct Scene *scene;
+       /** pointer to custom data - usually 'Object' but also 'rectf', but could be other types too */
+       void *data;
+       /** storage of times/values as 'decimals' */
+       float f1, f2;
+       /** storage of times/values/flags as 'whole' numbers */
+       int i1, i2;
 
        /* current iteration data */
-       struct FCurve *fcu;         /* F-Curve that is being iterated over */
-       int curIndex;               /* index of current keyframe being iterated over */
-       float channel_y;            /* y-position of midpoint of the channel (for the dopesheet) */
+       /** F-Curve that is being iterated over */
+       struct FCurve *fcu;
+       /** index of current keyframe being iterated over */
+       int curIndex;
+       /** y-position of midpoint of the channel (for the dopesheet) */
+       float channel_y;
 
        /* flags */
-       eKeyframeVertOk curflags;        /* current flags for the keyframe we're reached in the iteration process */
-       eKeyframeIterFlags iterflags;    /* settings for iteration process */
+       /** current flags for the keyframe we're reached in the iteration process */
+       eKeyframeVertOk curflags;
+       /** settings for iteration process */
+       eKeyframeIterFlags iterflags;
 } KeyframeEditData;
 
 /* ------- Function Pointer Typedefs ---------------- */
@@ -213,7 +223,8 @@ typedef enum eKeyMergeMode {
 /* ---------------- Looping API --------------------- */
 
 /* functions for looping over keyframes */
-/* function for working with F-Curve data only (i.e. when filters have been chosen to explicitly use this) */
+/* function for working with F-Curve data only
+ * (i.e. when filters have been chosen to explicitly use this) */
 short ANIM_fcurve_keyframes_loop(KeyframeEditData *ked, struct FCurve *fcu, KeyframeEditFunc key_ok, KeyframeEditFunc key_cb, FcuEditFunc fcu_cb);
 /* function for working with any type (i.e. one of the known types) of animation channel
  * - filterflag is bDopeSheet->flag (DOPESHEET_FILTERFLAG)
@@ -267,7 +278,8 @@ short bezt_to_cfraelem(KeyframeEditData *ked, struct BezTriple *bezt);
 void bezt_remap_times(KeyframeEditData *ked, struct BezTriple *bezt);
 
 /* ------ 1.5-D Region Testing Uitls (Lasso/Circle Select) ------- */
-/* XXX: These are temporary, until we can unify GP/Mask Keyframe handling and standard FCurve Keyframe handling */
+/* XXX: These are temporary,
+ * until we can unify GP/Mask Keyframe handling and standard FCurve Keyframe handling */
 
 bool keyframe_region_lasso_test(
         const KeyframeEdit_LassoData *data_lasso,
index aba381f..b114063 100644 (file)
@@ -195,7 +195,8 @@ typedef enum eModifyKey_Returns {
        MODIFYKEY_MISSING_TYPEINFO = -2,
 } eModifyKey_Returns;
 
-/* poll the current KeyingSet, updating it's set of paths (if "builtin"/"relative") for context changes */
+/* poll the current KeyingSet, updating it's set of paths
+ * (if "builtin"/"relative") for context changes */
 short ANIM_validate_keyingset(struct bContext *C, ListBase *dsources, struct KeyingSet *ks);
 
 /* use the specified KeyingSet to add/remove various Keyframes on the specified frame */
@@ -203,7 +204,8 @@ int ANIM_apply_keyingset(struct bContext *C, ListBase *dsources, struct bAction
 
 /* -------- */
 
-/* Get the first builtin KeyingSet with the given name, which occurs after the given one (or start of list if none given) */
+/* Get the first builtin KeyingSet with the given name, which occurs after the given one
+ * (or start of list if none given) */
 struct KeyingSet *ANIM_builtin_keyingset_get_named(struct KeyingSet *prevKS, const char name[]);
 
 /* Find KeyingSet type info given a name */
@@ -240,18 +242,27 @@ bool ANIM_keyingset_context_ok_poll(struct bContext *C, struct KeyingSet *ks);
 
 /* Flags for use by driver creation calls */
 typedef enum eCreateDriverFlags {
-       CREATEDRIVER_WITH_DEFAULT_DVAR  = (1 << 0),   /* create drivers with a default variable for nicer UI */
-       CREATEDRIVER_WITH_FMODIFIER     = (1 << 1),   /* create drivers with Generator FModifier (for backwards compat) */
+       /** create drivers with a default variable for nicer UI */
+       CREATEDRIVER_WITH_DEFAULT_DVAR  = (1 << 0),
+       /** create drivers with Generator FModifier (for backwards compat) */
+       CREATEDRIVER_WITH_FMODIFIER     = (1 << 1),
 } eCreateDriverFlags;
 
 /* Heuristic to use for connecting target properties to driven ones */
 typedef enum eCreateDriver_MappingTypes {
-       CREATEDRIVER_MAPPING_1_N        = 0,           /* 1 to Many - Use the specified index, and drive all elements with it */
-       CREATEDRIVER_MAPPING_1_1        = 1,           /* 1 to 1 - Only for the specified index on each side */
-       CREATEDRIVER_MAPPING_N_N        = 2,           /* Many to Many - Match up the indices one by one (only for drivers on vectors/arrays) */
-
-       CREATEDRIVER_MAPPING_NONE       = 3,           /* None (Single Prop)    - Do not create driver with any targets; these will get added later instead */
-       CREATEDRIVER_MAPPING_NONE_ALL   = 4,           /* None (All Properties) - Do not create driver with any targets; these will get added later instead */
+       /** 1 to Many - Use the specified index, and drive all elements with it */
+       CREATEDRIVER_MAPPING_1_N        = 0,
+       /** 1 to 1 - Only for the specified index on each side */
+       CREATEDRIVER_MAPPING_1_1        = 1,
+       /** Many to Many - Match up the indices one by one (only for drivers on vectors/arrays) */
+       CREATEDRIVER_MAPPING_N_N        = 2,
+
+       /** None (Single Prop):
+        * Do not create driver with any targets; these will get added later instead */
+       CREATEDRIVER_MAPPING_NONE       = 3,
+       /** None (All Properties):
+        * Do not create driver with any targets; these will get added later instead */
+       CREATEDRIVER_MAPPING_NONE_ALL   = 4,
 } eCreateDriver_MappingTypes;
 
 /* RNA Enum of eCreateDriver_MappingTypes, for use by the appropriate operators */
index f674a0d..0ff1ee4 100644 (file)
 struct wmEvent;
 
 typedef struct NumInput {
-       short  idx_max;                      /* idx_max < NUM_MAX_ELEMENTS */
+       /** idx_max < NUM_MAX_ELEMENTS */
+       short  idx_max;
        int    unit_sys;
-       int    unit_type[NUM_MAX_ELEMENTS];  /* Each value can have a different type */
+       /** Each value can have a different type */
+       int    unit_type[NUM_MAX_ELEMENTS];
        bool   unit_use_radians;
 
-       short  flag;                         /* Flags affecting all values' behavior */
-       short  val_flag[NUM_MAX_ELEMENTS];   /* Per-value flags */
-       float  val[NUM_MAX_ELEMENTS];        /* Direct value of the input */
-       float  val_org[NUM_MAX_ELEMENTS];    /* Original value of the input, for reset */
-       float  val_inc[NUM_MAX_ELEMENTS];    /* Increment steps */
-
-       short  idx;                          /* Active element/value */
-       char   str[NUM_STR_REP_LEN];         /* String as typed by user for edited value (we assume ASCII world!) */
-       /* Current position of cursor in edited value str (first byte of "current" letter, so 0 for an empty str) */
+       /** Flags affecting all values' behavior */
+       short  flag;
+       /** Per-value flags */
+       short  val_flag[NUM_MAX_ELEMENTS];
+       /** Direct value of the input */
+       float  val[NUM_MAX_ELEMENTS];
+       /** Original value of the input, for reset */
+       float  val_org[NUM_MAX_ELEMENTS];
+       /** Increment steps */
+       float  val_inc[NUM_MAX_ELEMENTS];
+
+       /** Active element/value */
+       short  idx;
+       /** String as typed by user for edited value (we assume ASCII world!) */
+       char   str[NUM_STR_REP_LEN];
+       /** Current position of cursor in edited value str
+        * (first byte of "current" letter, so 0 for an empty str) */
        int    str_cur;
 } NumInput;
 
index 288ee8a..5a02bc2 100644 (file)
@@ -67,8 +67,10 @@ struct uiLayout;
 #include "BLI_compiler_attrs.h"
 
 /* object_edit.c */
-struct Object *ED_object_context(struct bContext *C);               /* context.object */
-struct Object *ED_object_active_context(struct bContext *C); /* context.object or context.active_object */
+/* context.object */
+struct Object *ED_object_context(struct bContext *C);
+/* context.object or context.active_object */
+struct Object *ED_object_active_context(struct bContext *C);
 void ED_collection_hide_menu_draw(const struct bContext *C, struct uiLayout *layout);
 
 /* object_utils.c */
@@ -175,7 +177,8 @@ float ED_object_new_primitive_matrix(
         const float loc[3], const float rot[3], float primmat[4][4]);
 
 
-/* Avoid allowing too much insane values even by typing (typos can hang/crash Blender otherwise). */
+/* Avoid allowing too much insane values even by typing
+ * (typos can hang/crash Blender otherwise). */
 #define OBJECT_ADD_SIZE_MAXF 1.0e12f
 
 void ED_object_add_unit_props_size(struct wmOperatorType *ot);
index 8ef7fee..e53f303 100644 (file)
@@ -77,10 +77,18 @@ typedef struct ScreenFrameRateInfo {
 
 /* Enum for Action Zone Edges. Which edge of area is action zone. */
 typedef enum {
-       AE_RIGHT_TO_TOPLEFT,    /* Region located on the left, _right_ edge is action zone. Region minimized to the top left */
-       AE_LEFT_TO_TOPRIGHT,    /* Region located on the right, _left_ edge is action zone. Region minimized to the top right */
-       AE_TOP_TO_BOTTOMRIGHT,  /* Region located at the bottom, _top_ edge is action zone. Region minimized to the bottom right */
-       AE_BOTTOM_TO_TOPLEFT    /* Region located at the top, _bottom_ edge is action zone. Region minimized to the top left */
+       /** Region located on the left, _right_ edge is action zone.
+        * Region minimized to the top left */
+       AE_RIGHT_TO_TOPLEFT,
+       /** Region located on the right, _left_ edge is action zone.
+        * Region minimized to the top right */
+       AE_LEFT_TO_TOPRIGHT,
+       /** Region located at the bottom, _top_ edge is action zone.
+        * Region minimized to the bottom right */
+       AE_TOP_TO_BOTTOMRIGHT,
+       /** Region located at the top, _bottom_ edge is action zone.
+        * Region minimized to the top left */
+       AE_BOTTOM_TO_TOPLEFT
 } AZEdge;
 
 typedef enum {
index 363bb31..e15514f 100644 (file)
@@ -61,9 +61,12 @@ enum {
 
 /* mode flags: */
 enum {
-       TM_ALL_JOINTS      = (1 << 0),  /* all joints (for bones only) */
-       TM_SKIP_HANDLES    = (1 << 1),  /* skip handles when control point is selected (for curves only) */
-       TM_CALC_NORMALS    = (1 << 2),  /* fill in normals when available */
+       /** all joints (for bones only) */
+       TM_ALL_JOINTS      = (1 << 0),
+       /** skip handles when control point is selected (for curves only) */
+       TM_SKIP_HANDLES    = (1 << 1),
+       /** fill in normals when available */
+       TM_CALC_NORMALS    = (1 << 2),
 };
 
 enum {
index 748fb18..b3c3a6a 100644 (file)
@@ -155,11 +155,16 @@ void  ED_view3d_depth_tag_update(struct RegionView3D *rv3d);
 /* return values for ED_view3d_project_...() */
 typedef enum {
        V3D_PROJ_RET_OK   = 0,
-       V3D_PROJ_RET_CLIP_NEAR = 1,  /* can't avoid this when in perspective mode, (can't avoid) */
-       V3D_PROJ_RET_CLIP_ZERO = 2,  /* so close to zero we can't apply a perspective matrix usefully */
-       V3D_PROJ_RET_CLIP_BB   = 3,  /* bounding box clip - RV3D_CLIPPING */
-       V3D_PROJ_RET_CLIP_WIN  = 4,  /* outside window bounds */
-       V3D_PROJ_RET_OVERFLOW  = 5   /* outside range (mainly for short), (can't avoid) */
+       /** can't avoid this when in perspective mode, (can't avoid) */
+       V3D_PROJ_RET_CLIP_NEAR = 1,
+       /** so close to zero we can't apply a perspective matrix usefully */
+       V3D_PROJ_RET_CLIP_ZERO = 2,
+       /** bounding box clip - RV3D_CLIPPING */
+       V3D_PROJ_RET_CLIP_BB   = 3,
+       /** outside window bounds */
+       V3D_PROJ_RET_CLIP_WIN  = 4,
+       /** outside range (mainly for short), (can't avoid) */
+       V3D_PROJ_RET_OVERFLOW  = 5
 } eV3DProjStatus;
 
 /* some clipping tests are optional */
index 821f683..e0969ff 100644 (file)
@@ -539,7 +539,9 @@ typedef void (*uiBlockCancelFunc)(struct bContext *C, void *arg1);
 void UI_popup_block_invoke(struct bContext *C, uiBlockCreateFunc func, void *arg);
 void UI_popup_block_invoke_ex(struct bContext *C, uiBlockCreateFunc func, void *arg, const char *opname, int opcontext);
 void UI_popup_block_ex(struct bContext *C, uiBlockCreateFunc func, uiBlockHandleFunc popup_func, uiBlockCancelFunc cancel_func, void *arg, struct wmOperator *op);
-/* void uiPupBlockOperator(struct bContext *C, uiBlockCreateFunc func, struct wmOperator *op, int opcontext); */ /* UNUSED */
+#if 0  /* UNUSED */
+void uiPupBlockOperator(struct bContext *C, uiBlockCreateFunc func, struct wmOperator *op, int opcontext);
+#endif
 
 void UI_popup_block_close(struct bContext *C, struct wmWindow *win, uiBlock *block);
 
@@ -1254,13 +1256,19 @@ void uiItemsFullEnumO_items(
         const EnumPropertyItem *item_array, int totitem);
 
 void uiItemL(uiLayout *layout, const char *name, int icon); /* label */
-void uiItemLDrag(uiLayout *layout, struct PointerRNA *ptr, const char *name, int icon); /* label icon for dragging */
-void uiItemM(uiLayout *layout, const char *menuname, const char *name, int icon); /* menu */
-void uiItemMContents(uiLayout *layout, const char *menuname); /* menu contents */
-void uiItemV(uiLayout *layout, const char *name, int icon, int argval); /* value */
-void uiItemS(uiLayout *layout); /* separator */
+/* label icon for dragging */
+void uiItemLDrag(uiLayout *layout, struct PointerRNA *ptr, const char *name, int icon);
+/* menu */
+void uiItemM(uiLayout *layout, const char *menuname, const char *name, int icon);
+/* menu contents */
+void uiItemMContents(uiLayout *layout, const char *menuname);
+/* value */
+void uiItemV(uiLayout *layout, const char *name, int icon, int argval);
+/* separator */
+void uiItemS(uiLayout *layout);
 void uiItemS_ex(uiLayout *layout, float factor);
-void uiItemSpacer(uiLayout *layout); /* Special separator. */
+/* Special separator. */
+void uiItemSpacer(uiLayout *layout);
 
 void uiItemPopoverPanel_ptr(
         uiLayout *layout, struct bContext *C,
index 7d82e8d..d60e2d0 100644 (file)
@@ -164,7 +164,8 @@ void ui_block_to_window_rctf(const ARegion *ar, uiBlock *block, rctf *rct_dst, c
        ui_block_to_window_fl(ar, block, &rct_dst->xmax, &rct_dst->ymax);
 }
 
-void ui_window_to_block_fl(const ARegion *ar, uiBlock *block, float *x, float *y)   /* for mouse cursor */
+/* for mouse cursor */
+void ui_window_to_block_fl(const ARegion *ar, uiBlock *block, float *x, float *y)
 {
        float a, b, c, d, e, f, px, py;
        int sx, sy, getsizex, getsizey;
@@ -505,7 +506,8 @@ static void ui_block_bounds_calc_popup(
        ui_block_bounds_calc(block);
 
        /* If given, adjust input coordinates such that they would generate real final popup position.
-        * Needed to handle correctly floating panels once they have been dragged around, see T52999. */
+        * Needed to handle correctly floating panels once they have been dragged around,
+        * see T52999. */
        if (r_xy) {
                r_xy[0] = xy[0] + block->rect.xmin - raw_x;
                r_xy[1] = xy[1] + block->rect.ymin - raw_y;
@@ -1116,7 +1118,8 @@ static bool ui_but_event_property_operator_string(
        /* Don't use the button again. */
        but = NULL;
 
-       /* this version is only for finding hotkeys for properties (which get set via context using operators) */
+       /* this version is only for finding hotkeys for properties
+        * (which get set via context using operators) */
        if (prop) {
                /* to avoid massive slowdowns on property panels, for now, we only check the
                 * hotkeys for Editor / Scene settings...
@@ -1327,7 +1330,8 @@ static void ui_menu_block_set_keymaps(const bContext *C, uiBlock *block)
                                        continue;
                                }
                                else if (((block->flag & UI_BLOCK_POPOVER) == 0) && UI_but_is_tool(but)) {
-                                       /* For non-popovers, shown in shortcut only (has special shortcut handling code). */
+                                       /* For non-popovers, shown in shortcut only
+                                        * (has special shortcut handling code). */
                                        continue;
                                }
                        }
@@ -2090,7 +2094,8 @@ void ui_but_value_set(uiBut *but, double value)
                                case PROP_ENUM:
                                        if (RNA_property_flag(prop) & PROP_ENUM_FLAG) {
                                                int ivalue = (int)value;
-                                               ivalue ^= RNA_property_enum_get(&but->rnapoin, prop); /* toggle for enum/flag buttons */
+                                               /* toggle for enum/flag buttons */
+                                               ivalue ^= RNA_property_enum_get(&but->rnapoin, prop);
                                                RNA_property_enum_set(&but->rnapoin, prop, ivalue);
                                        }
                                        else {
@@ -2294,7 +2299,8 @@ static float ui_get_but_step_unit(uiBut *but, float step_default)
 {
        int unit_type = RNA_SUBTYPE_UNIT_VALUE(UI_but_unit_type_get(but));
        const double step_orig = step_default * UI_PRECISION_FLOAT_SCALE;
-       /* Scaling up 'step_origg ' here is a bit arbitrary, its just giving better scales from user POV */
+       /* Scaling up 'step_origg ' here is a bit arbitrary,
+        * its just giving better scales from user POV */
        const double scale_step = ui_get_but_scale_unit(but, step_orig * 10);
        const double step = bUnit_ClosestScalar(scale_step, but->block->unit->system, unit_type);
 
@@ -2514,7 +2520,8 @@ bool ui_but_string_set_eval_num(bContext *C, uiBut *but, const char *str, double
                bool is_unit_but = (ui_but_is_float(but) && ui_but_is_unit(but));
                /* only enable verbose if we won't run again with units */
                if (BPY_execute_string_as_number(C, NULL, str, is_unit_but == false, r_value)) {
-                       /* if the value parsed ok without unit conversion this button may still need a unit multiplier */
+                       /* if the value parsed ok without unit conversion
+                        * this button may still need a unit multiplier */
                        if (is_unit_but) {
                                char str_new[128];
 
@@ -3569,7 +3576,8 @@ static void ui_def_but_rna__menu(bContext *UNUSED(C), uiLayout *layout, void *bu
                                        uiItemL(column, item->name, item->icon);
                                }
                                else {
-                                       /* Do not use uiItemL here, as our root layout is a menu one, it will add a fake blank icon! */
+                                       /* Do not use uiItemL here, as our root layout is a menu one,
+                                        * it will add a fake blank icon! */
                                        uiDefBut(block, UI_BTYPE_LABEL, 0, item->name, 0, 0, UI_UNIT_X * 5, UI_UNIT_Y, NULL, 0.0, 0.0, 0, 0, "");
                                }
                        }
@@ -4588,7 +4596,8 @@ void UI_but_func_search_set(
         uiButSearchFunc search_func, void *arg,
         uiButHandleFunc bfunc, void *active)
 {
-       /* needed since callers don't have access to internal functions (as an alternative we could expose it) */
+       /* needed since callers don't have access to internal functions
+        * (as an alternative we could expose it) */
        if (search_create_func == NULL) {
                search_create_func = ui_searchbox_create_generic;
        }
@@ -4637,7 +4646,8 @@ static void operator_enum_search_cb(const struct bContext *C, void *but, const c
                RNA_property_enum_items_gettexted((bContext *)C, ptr, prop, &item_array, NULL, &do_free);
 
                for (item = item_array; item->identifier; item++) {
-                       /* note: need to give the index rather than the identifier because the enum can be freed */
+                       /* note: need to give the index rather than the
+                        * identifier because the enum can be freed */
                        if (BLI_strcasestr(item->name, str)) {
                                if (false == UI_search_item_add(items, item->name, POINTER_FROM_INT(item->value), item->icon))
                                        break;
index d102c7c..488ee0f 100644 (file)
 #ifdef USE_UIBUT_SPATIAL_ALIGN
 
 /**
- * This struct stores a (simplified) 2D representation of all buttons of a same align group, with their
- * immediate neighbors (if found), and needed value to compute 'stitching' of aligned buttons.
+ * This struct stores a (simplified) 2D representation of all buttons of a same align group,
+ * with their immediate neighbors (if found),
+ * and needed value to compute 'stitching' of aligned buttons.
  *
- * \note This simplistic struct cannot fully represent complex layouts where buttons share some 'align space' with
- *       several others (see schema below), we'd need linked list and more complex code to handle that.
- *       However, looks like we can do without that for now, which is rather lucky!
+ * \note This simplistic struct cannot fully represent complex layouts where buttons share some
+ *       'align space' with several others (see schema below), we'd need linked list and more
+ *       complex code to handle that. However, looks like we can do without that for now,
+ *       which is rather lucky!
  *
  *       <pre>
- *       +-----------+-----------+
- *       |   BUT 1   |   BUT 2   |      BUT 3 has two 'top' neighbors...
- *       |-----------------------|  =>  In practice, we only store one of BUT 1 or 2 (which ones is not
- *       |         BUT 3         |      really deterministic), and assume the other stores a ref to BUT 3.
- *       +-----------------------+
+ *       +--------+-------+
+ *       | BUT 1  | BUT 2 |      BUT 3 has two 'top' neighbors...
+ *       |----------------|  =>  In practice, we only store one of BUT 1 or 2 (which ones is not
+ *       |      BUT 3     |      really deterministic), and assume the other stores a ref to BUT 3.
+ *       +----------------+
  *       </pre>
  *
- *       This will probably not work in all possible cases, but not sure we want to support such exotic cases anyway.
+ *       This will probably not work in all possible cases,
+ *       but not sure we want to support such exotic cases anyway.
  */
 typedef struct ButAlign {
        uiBut *but;
@@ -71,13 +74,15 @@ typedef struct ButAlign {
        /* Distances to the neighbors. */
        float dists[4];
 
-       /* Flags, used to mark whether we should 'stitch' the corners of this button with its neighbors' ones. */
+       /* Flags, used to mark whether we should 'stitch'
+        * the corners of this button with its neighbors' ones. */
        char flags[4];
 } ButAlign;
 
 /* Side-related enums and flags. */
 enum {
-       /* Sides (used as indices, order is **crucial**, this allows us to factorize code in a loop over the four sides). */
+       /* Sides (used as indices, order is **crucial**,
+        * this allows us to factorize code in a loop over the four sides). */
        LEFT         = 0,
        TOP          = 1,
        RIGHT        = 2,
@@ -162,14 +167,17 @@ static void block_align_proximity_compute(ButAlign *butal, ButAlign *butal_other
        }
 
        for (side = 0; side < RIGHT; side++) {
-               /* We are only interested in buttons which share a same line (LEFT/RIGHT sides) or column (TOP/DOWN sides). */
+               /* We are only interested in buttons which share a same line
+                * (LEFT/RIGHT sides) or column (TOP/DOWN sides). */
                if (buts_share[IS_COLUMN(side)]) {
                        side_opp = OPPOSITE(side);
 
-                       /* We check both opposite sides at once, because with very small buttons, delta could be below max_delta for
-                        * the wrong side (that is, in horizontal case, the total width of two buttons can be below max_delta).
-                        * We rely on exact zero value here as an 'already processed' flag, so ensure we never actually
-                        * set a zero value at this stage. FLT_MIN is zero-enough for UI position computing. ;) */
+                       /* We check both opposite sides at once, because with very small buttons,
+                        * delta could be below max_delta for the wrong side
+                        * (that is, in horizontal case, the total width of two buttons can be below max_delta).
+                        * We rely on exact zero value here as an 'already processed' flag,
+                        * so ensure we never actually set a zero value at this stage.
+                        * FLT_MIN is zero-enough for UI position computing. ;) */
                        delta = max_ff(fabsf(*butal->borders[side] - *butal_other->borders[side_opp]), FLT_MIN);
                        delta_side_opp = max_ff(fabsf(*butal->borders[side_opp] - *butal_other->borders[side]), FLT_MIN);
                        if (delta_side_opp < delta) {
@@ -178,18 +186,21 @@ static void block_align_proximity_compute(ButAlign *butal, ButAlign *butal_other
                        }
 
                        if (delta < max_delta) {
-                               /* We are only interested in neighbors that are at least as close as already found ones. */
+                               /* We are only interested in neighbors that are
+                                * at least as close as already found ones. */
                                if (delta <= butal->dists[side]) {
                                        {
                                                /* We found an as close or closer neighbor.
                                                 * If both buttons are alignable, we set them as each other neighbors.
-                                                * Else, we have an unalignable one, we need to reset the others matching neighbor to NULL
-                                                * if its 'proximity distance' is really lower with current one.
+                                                * Else, we have an unalignable one, we need to reset the others matching
+                                                * neighbor to NULL if its 'proximity distance'
+                                                * is really lower with current one.
                                                 *
-                                                * NOTE: We cannot only execute that piece of code in case we found a **closer** neighbor,
-                                                *       due to the limited way we represent neighbors (buttons only know **one** neighbor
-                                                *       on each side, when they can actually have several ones), it would prevent
-                                                *       some buttons to be properly 'neighborly-initialized'. */
+                                                * NOTE: We cannot only execute that piece of code in case we found a
+                                                *       **closer** neighbor, due to the limited way we represent neighbors
+                                                *       (buttons only know **one** neighbor on each side, when they can
+                                                *       actually have several ones), it would prevent some buttons to be
+                                                *       properly 'neighborly-initialized'. */
                                                if (butal_can_align && butal_other_can_align) {
                                                        butal->neighbors[side] = butal_other;
                                                        butal_other->neighbors[side_opp] = butal;
@@ -217,10 +228,12 @@ static void block_align_proximity_compute(ButAlign *butal, ButAlign *butal_other
                                                        butal_other->neighbors[side_opp] = butal;
                                                }
 
-                                               /* We have a pair of neighbors, we have to check whether we can stitch their matching corners.
+                                               /* We have a pair of neighbors, we have to check whether we
+                                                *   can stitch their matching corners.
                                                 *   E.g. if butal_other is on the left of butal (that is, side == LEFT),
-                                                *        if both TOP (side_s1) coordinates of buttons are close enough, we can stitch
-                                                *        their upper matching corners, and same for DOWN (side_s2) side. */
+                                                *        if both TOP (side_s1) coordinates of buttons are close enough,
+                                                *        we can stitch their upper matching corners,
+                                                *        and same for DOWN (side_s2) side. */
                                                delta = fabsf(*butal->borders[side_s1] - *butal_other->borders[side_s1]);
                                                if (delta < max_delta) {
                                                        butal->flags[side_s1] |= stitch;
@@ -233,7 +246,8 @@ static void block_align_proximity_compute(ButAlign *butal, ButAlign *butal_other
                                                }
                                        }
                                }
-                               /* We assume two buttons can only share one side at most - for until we have sperical UI... */
+                               /* We assume two buttons can only share one side at most - for until
+                                * we have sperical UI... */
                                return;
                        }
                }
@@ -329,9 +343,10 @@ static int ui_block_align_butal_cmp(const void *a, const void *b)
                return (*butal->borders[LEFT] > *butal_other->borders[LEFT]) ? 1 : -1;
        }
 
-       /* XXX We cannot actually assert here, since in some very compressed space cases, stupid UI code produces
-        *     widgets which have the same TOP and LEFT positions...
-        *     We do not care really, because this happens when UI is way too small to be usable anyway. */
+       /* XXX We cannot actually assert here, since in some very compressed space cases,
+        *     stupid UI code produces widgets which have the same TOP and LEFT positions...
+        *     We do not care really,
+        *     because this happens when UI is way too small to be usable anyway. */
        /* BLI_assert(0); */
        return 0;
 }
@@ -423,27 +438,29 @@ void ui_block_align_calc(uiBlock *block, const ARegion *region)
        }
 
        /* This will give us ButAlign items regrouped by align group, vertical and horizontal location.
-        * Note that, given how buttons are defined in UI code, butal_array shall already be "nearly sorted"... */
+        * Note that, given how buttons are defined in UI code,
+        * butal_array shall already be "nearly sorted"... */
        qsort(butal_array, (size_t)num_buttons, sizeof(*butal_array), ui_block_align_butal_cmp);
 
-       /* Third loop: for each pair of buttons in the same align group, we compute their potential proximity.
-        * Note that each pair is checked only once, and that we break early in case we know all remaining pairs will
-        * always be too far away. */
+       /* Third loop: for each pair of buttons in the same align group,
+        * we compute their potential proximity. Note that each pair is checked only once, and that we
+        * break early in case we know all remaining pairs will always be too far away. */
        for (i = 0, butal = butal_array; i < num_buttons; i++, butal++) {
                const short alignnr = butal->but->alignnr;
 
                for (j = i + 1, butal_other = &butal_array[i + 1]; j < num_buttons; j++, butal_other++) {
                        const float max_delta = MAX_DELTA;
 
-                       /* Since they are sorted, buttons after current butal can only be of same or higher group, and once
-                        * they are not of same group, we know we can break this sub-loop and start checking with next butal. */
+                       /* Since they are sorted, buttons after current butal can only be of same or higher
+                        * group, and once they are not of same group, we know we can break this sub-loop and
+                        * start checking with next butal. */
                        if (butal_other->but->alignnr != alignnr) {
                                break;
                        }
 
-                       /* Since they are sorted vertically first, buttons after current butal can only be at same or lower height,
-                        * and once they are lower than a given threshold, we know we can break this sub-loop and
-                        * start checking with next butal. */
+                       /* Since they are sorted vertically first, buttons after current butal can only be at
+                        * same or lower height, and once they are lower than a given threshold, we know we can
+                        * break this sub-loop and start checking with next butal. */
                        if ((*butal->borders[DOWN] - *butal_other->borders[TOP]) > max_delta) {
                                break;
                        }
index 4cd3acc..1ac5ce2 100644 (file)
@@ -216,7 +216,8 @@ bool ui_but_anim_expression_create(uiBut *but, const char *str)
        }
 
        /* make sure we have animdata for this */
-       /* FIXME: until materials can be handled by depsgraph, don't allow drivers to be created for them */
+       /* FIXME: until materials can be handled by depsgraph,
+        * don't allow drivers to be created for them */
        id = (ID *)but->rnapoin.id.data;
        if ((id == NULL) || (GS(id->name) == ID_MA) || (GS(id->name) == ID_TE)) {
                if (G.debug & G_DEBUG)
index 5ac4558..f71715f 100644 (file)
@@ -54,7 +54,8 @@
 #include "WM_api.h"
 #include "WM_types.h"
 
-/* This hack is needed because we don't have a good way to re-reference keymap items once added: T42944 */
+/* This hack is needed because we don't have a good way to
+ * re-reference keymap items once added: T42944 */
 #define USE_KEYMAP_ADD_HACK
 
 /* -------------------------------------------------------------------- */
@@ -135,7 +136,8 @@ static uiBlock *menu_add_shortcut(bContext *C, ARegion *ar, void *arg)
        IDProperty *prop = (but->opptr) ? but->opptr->data : NULL;
        int kmi_id;
 
-       /* XXX this guess_opname can potentially return a different keymap than being found on adding later... */
+       /* XXX this guess_opname can potentially return a different keymap
+        * than being found on adding later... */
        km = WM_keymap_guess_opname(C, but->optype->idname);
        kmi = WM_keymap_add_item(km, but->optype->idname, AKEY, KM_PRESS, 0, 0);
        kmi_id = kmi->id;
@@ -407,7 +409,8 @@ bool ui_popup_context_menu_for_button(bContext *C, uiBut *but)
                bool is_idprop = RNA_property_is_idprop(prop);
                bool is_set = RNA_property_is_set(ptr, prop);
 
-               /* second slower test, saved people finding keyframe items in menus when its not possible */
+               /* second slower test,
+                * saved people finding keyframe items in menus when its not possible */
                if (is_anim)
                        is_anim = RNA_property_path_from_ID_check(&but->rnapoin, but->rnaprop);
 
@@ -418,7 +421,8 @@ bool ui_popup_context_menu_for_button(bContext *C, uiBut *but)
                const int override_status = RNA_property_static_override_status(ptr, prop, -1);
                const bool is_overridable = (override_status & RNA_OVERRIDE_STATUS_OVERRIDABLE) != 0;
 
-               /* Set the (button_pointer, button_prop) and pointer data for Python access to the hovered ui element. */
+               /* Set the (button_pointer, button_prop)
+                * and pointer data for Python access to the hovered ui element. */
                uiLayoutSetContextFromBut(layout, but);
 
                /* Keyframes */
@@ -675,7 +679,8 @@ bool ui_popup_context_menu_for_button(bContext *C, uiBut *but)
                }
        }
 
-       /* Pointer properties and string properties with prop_search support jumping to target object/bone. */
+       /* Pointer properties and string properties with
+        * prop_search support jumping to target object/bone. */
        if (but->rnapoin.data && but->rnaprop) {
                const PropertyType type = RNA_property_type(but->rnaprop);
 
index ac1578d..dab8dd3 100644 (file)
@@ -547,7 +547,8 @@ void UI_draw_text_underline(int pos_x, int pos_y, int len, int height, const flo
 
 /* ************** SPECIAL BUTTON DRAWING FUNCTIONS ************* */
 
-/* based on UI_draw_roundbox_gl_mode, check on making a version which allows us to skip some sides */
+/* based on UI_draw_roundbox_gl_mode,
+ * check on making a version which allows us to skip some sides */
 void ui_draw_but_TAB_outline(const rcti *rect, float rad, uchar highlight[3], uchar highlight_fade[3])
 {
        GPUVertFormat *format = immVertexFormat();
@@ -892,7 +893,8 @@ void ui_draw_but_WAVEFORM(ARegion *UNUSED(ar), uiBut *but, const uiWidgetColors
        int scissor[4];
        float colors[3][3];
        float colorsycc[3][3] = {{1, 0, 1}, {1, 1, 0}, {0, 1, 1}};
-       float colors_alpha[3][3], colorsycc_alpha[3][3]; /* colors  pre multiplied by alpha for speed up */
+       /* colors  pre multiplied by alpha for speed up */
+       float colors_alpha[3][3], colorsycc_alpha[3][3];
        float min, max;
 
        if (scopes == NULL) return;
index 781032e..33d6952 100644 (file)
@@ -114,7 +114,8 @@ static int datadropper_init(bContext *C, wmOperator *op)
        type = RNA_property_pointer_type(&ddr->ptr, ddr->prop);
        ddr->idcode = RNA_type_to_ID_code(type);
        BLI_assert(ddr->idcode != 0);
-       /* Note we can translate here (instead of on draw time), because this struct has very short lifetime. */
+       /* Note we can translate here (instead of on draw time),
+        * because this struct has very short lifetime. */
        ddr->idcode_name = TIP_(BKE_idcode_to_name(ddr->idcode));
 
        PointerRNA ptr = RNA_property_pointer_get(&ddr->ptr, ddr->prop);
index e037692..86cc517 100644 (file)
@@ -229,10 +229,14 @@ typedef struct uiButMultiState {
 
 typedef struct uiHandleButtonMulti {
        enum {
-               BUTTON_MULTI_INIT_UNSET = 0,    /* gesture direction unknown, wait until mouse has moved enough... */
-               BUTTON_MULTI_INIT_SETUP,        /* vertical gesture detected, flag buttons interactively (UI_BUT_DRAG_MULTI) */
-               BUTTON_MULTI_INIT_ENABLE,       /* flag buttons finished, apply horizontal motion to active and flagged */
-               BUTTON_MULTI_INIT_DISABLE,      /* vertical gesture _not_ detected, take no further action */
+               /** gesture direction unknown, wait until mouse has moved enough... */
+               BUTTON_MULTI_INIT_UNSET = 0,
+               /** vertical gesture detected, flag buttons interactively (UI_BUT_DRAG_MULTI) */
+               BUTTON_MULTI_INIT_SETUP,
+               /** flag buttons finished, apply horizontal motion to active and flagged */
+               BUTTON_MULTI_INIT_ENABLE,
+               /** vertical gesture _not_ detected, take no further action */
+               BUTTON_MULTI_INIT_DISABLE,
        } init;
 
        bool has_mbuts;  /* any buttons flagged UI_BUT_DRAG_MULTI */
@@ -241,11 +245,12 @@ typedef struct uiHandleButtonMulti {
 
        bool is_proportional;
 
-       /* In some cases we directly apply the changes to multiple buttons, so we don't want to do it twice. */
+       /* In some cases we directly apply the changes to multiple buttons,
+        * so we don't want to do it twice. */
        bool skip;
 
-       /* before activating, we need to check gesture direction
-        * accumulate signed cursor movement here so we can tell if this is a vertical motion or not. */
+       /* before activating, we need to check gesture direction accumulate signed cursor movement
+        * here so we can tell if this is a vertical motion or not. */
        float drag_dir[2];
 
        /* values copied direct from event->x,y
@@ -323,8 +328,10 @@ typedef struct uiHandleButtonData {
        CBData *dragcbd;
 
 #ifdef USE_CONT_MOUSE_CORRECT
-       /* when ungrabbing buttons which are #ui_but_is_cursor_warp(), we may want to position them
-        * FLT_MAX signifies do-nothing, use #ui_block_to_window_fl() to get this into a usable space  */
+       /* when ungrabbing buttons which are #ui_but_is_cursor_warp(),
+        * we may want to position them.
+        * FLT_MAX signifies do-nothing, use #ui_block_to_window_fl()
+        * to get this into a usable space. */
        float ungrab_mval[2];
 #endif
 
@@ -1335,7 +1342,8 @@ static void ui_drag_toggle_set(bContext *C, uiDragToggleHandle *drag_info, const
                                        BLI_rctf_cent_y(&but->rect),
                                };
 
-                               /* check if this is a different button, chances are high the button wont move about :) */
+                               /* check if this is a different button,
+                                * chances are high the button wont move about :) */
                                if (len_manhattan_v2v2(drag_info->but_cent_start, but_cent_new) > 1.0f) {
                                        if (fabsf(drag_info->but_cent_start[0] - but_cent_new[0]) <
                                            fabsf(drag_info->but_cent_start[1] - but_cent_new[1]))
@@ -1596,18 +1604,22 @@ static void ui_selectcontext_apply(
                        RNA_property_int_range(&but->rnapoin, prop, &min.i, &max.i);
                }
                else if (rna_type == PROP_ENUM) {
-                       delta.i = RNA_property_enum_get(&but->rnapoin, prop);  /* not a delta infact */
+                       /* not a delta infact */
+                       delta.i = RNA_property_enum_get(&but->rnapoin, prop);
                }
                else if (rna_type == PROP_BOOLEAN) {
                        if (is_array) {
-                               delta.b = RNA_property_boolean_get_index(&but->rnapoin, prop, index);  /* not a delta infact */
+                               /* not a delta infact */
+                               delta.b = RNA_property_boolean_get_index(&but->rnapoin, prop, index);
                        }
                        else {
-                               delta.b = RNA_property_boolean_get(&but->rnapoin, prop);  /* not a delta infact */
+                               /* not a delta infact */
+                               delta.b = RNA_property_boolean_get(&but->rnapoin, prop);
                        }
                }
                else if (rna_type == PROP_POINTER) {
-                       delta.p = RNA_property_pointer_get(&but->rnapoin, prop);  /* not a delta infact */
+                       /* not a delta infact */
+                       delta.p = RNA_property_pointer_get(&but->rnapoin, prop);
                }
 
 #ifdef USE_ALLSELECT_LAYER_HACK
@@ -3244,7 +3256,8 @@ static void ui_do_but_textedit(
                {
                        bool had_selection = but->selsta != but->selend;
 
-                       /* exit on LMB only on RELEASE for searchbox, to mimic other popups, and allow multiple menu levels */
+                       /* exit on LMB only on RELEASE for searchbox, to mimic other popups,
+                        * and allow multiple menu levels */
                        if (data->searchbox)
                                inbox = ui_searchbox_inside(data->searchbox, event->x, event->y);
 
@@ -3282,7 +3295,8 @@ static void ui_do_but_textedit(
                                changed = true;
                        }
                        else if (inbox) {
-                               /* if we allow activation on key press, it gives problems launching operators [#35713] */
+                               /* if we allow activation on key press,
+                                * it gives problems launching operators T35713. */
                                if (event->val == KM_RELEASE) {
                                        button_activate_state(C, but, BUTTON_STATE_EXIT);
                                        retval = WM_UI_HANDLER_BREAK;
@@ -3741,7 +3755,8 @@ static bool ui_do_but_ANY_drag_toggle(
                }
        }
        else if (data->state == BUTTON_STATE_WAIT_DRAG) {
-               /* note: the 'BUTTON_STATE_WAIT_DRAG' part of 'ui_do_but_EXIT' could be refactored into its own function */
+               /* note: the 'BUTTON_STATE_WAIT_DRAG' part of 'ui_do_but_EXIT' could be refactored into
+                * its own function */
                data->applied = false;
                *r_retval = ui_do_but_EXIT(C, but, data, event);
                return true;
@@ -4312,7 +4327,8 @@ static bool ui_numedit_but_NUM(
                float non_linear_pixel_map;
                float non_linear_scale;
 
-               /* Use a non-linear mapping of the mouse drag especially for large floats (normal behavior) */
+               /* Use a non-linear mapping of the mouse drag especially for large floats
+                * (normal behavior) */
                deler = 500;
                if (is_float) {
                        /* not needed for smaller float buttons */
@@ -4454,8 +4470,10 @@ static int ui_do_but_NUM(
                }
 
                /* XXX hardcoded keymap check.... */
-               if (type == MOUSEPAN && event->alt)
-                       retval = WM_UI_HANDLER_BREAK; /* allow accumulating values, otherwise scrolling gets preference */
+               if (type == MOUSEPAN && event->alt) {
+                       /* allow accumulating values, otherwise scrolling gets preference */
+                       retval = WM_UI_HANDLER_BREAK;
+               }
                else if (type == WHEELDOWNMOUSE && event->ctrl) {
                        mx = but->rect.xmin;
                        but->drawflag &= ~UI_BUT_ACTIVE_RIGHT;
@@ -4756,8 +4774,10 @@ static int ui_do_but_SLI(
                }
 
                /* XXX hardcoded keymap check.... */
-               if (type == MOUSEPAN && event->alt)
-                       retval = WM_UI_HANDLER_BREAK; /* allow accumulating values, otherwise scrolling gets preference */
+               if (type == MOUSEPAN && event->alt) {
+                       /* allow accumulating values, otherwise scrolling gets preference */
+                       retval = WM_UI_HANDLER_BREAK;
+               }
                else if (type == WHEELDOWNMOUSE && event->ctrl) {
                        mx = but->rect.xmin;
                        click = 2;
@@ -4772,7 +4792,8 @@ static int ui_do_but_SLI(
                                retval = WM_UI_HANDLER_BREAK;
                        }
 #ifndef USE_ALLSELECT
-                       /* alt-click on sides to get "arrows" like in UI_BTYPE_NUM buttons, and match wheel usage above */
+                       /* alt-click on sides to get "arrows" like in UI_BTYPE_NUM buttons,
+                        * and match wheel usage above */
                        else if (event->type == LEFTMOUSE && event->alt) {
                                int halfpos = BLI_rctf_cent_x(&but->rect);
                                click = 2;
@@ -4877,7 +4898,8 @@ static int ui_do_but_SLI(
 
 #if 0
                        if (but->type == SLI) {
-                               f = (float)(mx - but->rect.xmin) / (BLI_rctf_size_x(&but->rect)); /* same as below */
+                               /* same as below */
+                               f = (float)(mx - but->rect.xmin) / (BLI_rctf_size_x(&but->rect));
                        }
                        else
 #endif
@@ -6108,7 +6130,8 @@ static bool ui_numedit_but_CURVE(
        int a;
        bool changed = false;
 
-       /* evtx evty and drag coords are absolute mousecoords, prevents errors when editing when layout changes */
+       /* evtx evty and drag coords are absolute mousecoords,
+        * prevents errors when editing when layout changes */
        mx = evtx;
        my = evty;
        ui_window_to_block(data->region, block, &mx, &my);
@@ -6277,7 +6300,8 @@ static int ui_do_but_CURVE(
 
                                                changed = true;
 
-                                               /* reset cmp back to the curve points again, rather than drawing segments */
+                                               /* reset cmp back to the curve points again,
+                                                * rather than drawing segments */
                                                cmp = cuma->curve;
 
                                                /* find newly added point and make it 'sel' */
@@ -7197,7 +7221,8 @@ void UI_but_tooltip_refresh(bContext *C, uiBut *but)
        }
 }
 
-/* removes tooltip timer from active but (meaning tooltip is disabled until it's reenabled again) */
+/* removes tooltip timer from active but
+ * (meaning tooltip is disabled until it's reenabled again) */
 void UI_but_tooltip_timer_remove(bContext *C, uiBut *but)
 {
        uiHandleButtonData *data;
@@ -8005,7 +8030,8 @@ static int ui_handle_button_event(bContext *C, const wmEvent *event, uiBut *but)
                                uiBut *but_other = ui_but_find_mouse_over(ar, event);
                                bool exit = false;
 
-                               /* always deactivate button for pie menus, else moving to blank space will leave activated */
+                               /* always deactivate button for pie menus,
+                                * else moving to blank space will leave activated */
                                if ((!ui_block_is_menu(block) || ui_block_is_pie_menu(block)) &&
                                    !ui_but_contains_point_px(ar, but, event->x, event->y))
                                {
@@ -8041,7 +8067,8 @@ static int ui_handle_button_event(bContext *C, const wmEvent *event, uiBut *but)
 
                                break;
                        }
-                       /* XXX hardcoded keymap check... but anyway, while view changes, tooltips should be removed */
+                       /* XXX hardcoded keymap check... but anyway,
+                        * while view changes, tooltips should be removed */
                        case WHEELUPMOUSE:
                        case WHEELDOWNMOUSE:
                        case MIDDLEMOUSE:
@@ -8267,7 +8294,8 @@ static int ui_handle_list_event(bContext *C, const wmEvent *event, ARegion *ar,
                        }
 
                        if (dyn_data->items_filter_neworder || dyn_data->items_filter_flags) {
-                               /* If we have a display order different from collection order, we have some work! */
+                               /* If we have a display order different from
+                                * collection order, we have some work! */
                                int *org_order = MEM_mallocN(dyn_data->items_shown * sizeof(int), __func__);
                                const int *new_order = dyn_data->items_filter_neworder;
                                int i, org_idx = -1, len = dyn_data->items_len;
@@ -9053,7 +9081,8 @@ static int ui_handle_menu_event(
                                                }
                                        }
                                        else if (ELEM(event->val, KM_RELEASE, KM_CLICK)) {
-                                               /* For buttons that use a hold function, exit when mouse-up outside the menu. */
+                                               /* For buttons that use a hold function,
+                                                * exit when mouse-up outside the menu. */
                                                if (block->flag & UI_BLOCK_POPUP_HOLD) {
                                                        /* Note, we could check the cursor is over the parent button. */
                                                        menu->menuretval = UI_RETURN_CANCEL;
@@ -9306,7 +9335,8 @@ static int ui_pie_handler(bContext *C, const wmEvent *event, uiPopupBlockHandle
        bool is_click_style;
        float dist;
 
-       /* we block all events, this is modal interaction, except for drop events which is described below */
+       /* we block all events, this is modal interaction,
+        * except for drop events which is described below */
        int retval = WM_UI_HANDLER_BREAK;
 
        if (event->type == EVT_DROP) {
@@ -9455,7 +9485,8 @@ static int ui_pie_handler(bContext *C, const wmEvent *event, uiPopupBlockHandle
                                                        block->pie_data.flags |= UI_PIE_DRAG_STYLE;
                                                }
 
-                                               /* here instead, we use the offset location to account for the initial direction timeout */
+                                               /* here instead, we use the offset location to account for the initial
+                                                * direction timeout */
                                                if ((U.pie_menu_confirm > 0) &&
                                                    (dist >= U.dpi_fac * (U.pie_menu_threshold + U.pie_menu_confirm)))
                                                {
@@ -9573,7 +9604,8 @@ static int ui_handle_menus_recursive(
                uiBlock *block = menu->region->uiblocks.first;
                const bool is_menu = ui_block_is_menu(block);
                bool inside = false;
-               /* root pie menus accept the key that spawned them as double click to improve responsiveness */
+               /* root pie menus accept the key that spawned
+                * them as double click to improve responsiveness */
                bool do_recursion = (!(block->flag & UI_BLOCK_RADIAL) || event->type != block->pie_data.event);
 
                if (do_recursion) {
@@ -9773,7 +9805,8 @@ static int ui_handler_region_menu(bContext *C, const wmEvent *event, void *UNUSE
                data = but->active;
 
                if ((data->state == BUTTON_STATE_MENU_OPEN) &&
-                   (is_inside_menu == false) && /* make sure mouse isn't inside another menu (see T43247) */
+                   /* make sure mouse isn't inside another menu (see T43247) */
+                   (is_inside_menu == false) &&
                    (ELEM(but->type, UI_BTYPE_PULLDOWN, UI_BTYPE_POPOVER)) &&
                    (but_other = ui_but_find_mouse_over(ar, event)) &&
                    (but != but_other) &&
@@ -9837,7 +9870,8 @@ static int ui_popup_handler(bContext *C, const wmEvent *event, void *userdata)
 {
        uiPopupBlockHandle *menu = userdata;
        struct ARegion *menu_region;
-       /* we block all events, this is modal interaction, except for drop events which is described below */
+       /* we block all events, this is modal interaction,
+        * except for drop events which is described below */
        int retval = WM_UI_HANDLER_BREAK;
        bool reset_pie = false;
 
index eb35f69..f3bb61c 100644 (file)
@@ -709,7 +709,10 @@ static void init_internal_icons(void)
 
                if (icondir) {
                        BLI_join_dirfile(iconfilestr, sizeof(iconfilestr), icondir, btheme->tui.iconfile);
-                       bbuf = IMB_loadiffname(iconfilestr, IB_rect, NULL); /* if the image is missing bbuf will just be NULL */
+
+                       /* if the image is missing bbuf will just be NULL */
+                       bbuf = IMB_loadiffname(iconfilestr, IB_rect, NULL);
+
                        if (bbuf && (bbuf->x < ICON_IMAGE_W || bbuf->y < ICON_IMAGE_H)) {
                                printf("\n***WARNING***\nIcons file %s too small.\nUsing built-in Icons instead\n", iconfilestr);
                                IMB_freeImBuf(bbuf);
@@ -1520,7 +1523,8 @@ static void icon_draw_size(
        }
        else if (di->type == ICON_TYPE_GEOM) {
 #ifdef USE_UI_TOOLBAR_HACK
-               /* TODO(campbell): scale icons up for toolbar, we need a way to detect larger buttons and do this automatic. */
+               /* TODO(campbell): scale icons up for toolbar,
+                * we need a way to detect larger buttons and do this automatic. */
                {
                        float scale = (float)ICON_DEFAULT_HEIGHT_TOOLBAR / (float)ICON_DEFAULT_HEIGHT;
                        y = (y + (h / 2)) - ((h * scale) / 2);
@@ -1615,7 +1619,8 @@ static void icon_draw_size(
 static void ui_id_preview_image_render_size(
         const bContext *C, Scene *scene, ID *id, PreviewImage *pi, int size, const bool use_job)
 {
-       if (((pi->flag[size] & PRV_CHANGED) || !pi->rect[size])) { /* changed only ever set by dynamic icons */
+       /* changed only ever set by dynamic icons */
+       if (((pi->flag[size] & PRV_CHANGED) || !pi->rect[size])) {
                /* create the rect if necessary */
                icon_set_image(C, scene, id, pi, size, use_job);
 
@@ -1628,10 +1633,14 @@ void UI_id_icon_render(const bContext *C, Scene *scene, ID *id, const bool big,
        PreviewImage *pi = BKE_previewimg_id_ensure(id);
 
        if (pi) {
-               if (big)
-                       ui_id_preview_image_render_size(C, scene, id, pi, ICON_SIZE_PREVIEW, use_job);  /* bigger preview size */
-               else
-                       ui_id_preview_image_render_size(C, scene, id, pi, ICON_SIZE_ICON, use_job);     /* icon size */
+               if (big) {
+                       /* bigger preview size */
+                       ui_id_preview_image_render_size(C, scene, id, pi, ICON_SIZE_PREVIEW, use_job);
+               }
+               else {
+                       /* icon size */
+                       ui_id_preview_image_render_size(C, scene, id, pi, ICON_SIZE_ICON, use_job);
+               }
        }
 }
 
index 0a69ff6..20671ae 100644 (file)
@@ -106,7 +106,8 @@ typedef enum {
 } uiWidgetTypeEnum;
 
 #define UI_MENU_WIDTH_MIN       (UI_UNIT_Y * 9)
-#define UI_MENU_SUBMENU_PADDING (6 * UI_DPI_FAC) /* some extra padding added to menus containing submenu icons */
+/* some extra padding added to menus containing submenu icons */
+#define UI_MENU_SUBMENU_PADDING (6 * UI_DPI_FAC)
 
 /* menu scrolling */
 #define UI_MENU_SCROLL_ARROW   12
@@ -181,13 +182,20 @@ extern const short ui_radial_dir_to_angle[8];
 
 /* PieMenuData->flags */
 enum {
-       UI_PIE_DEGREES_RANGE_LARGE  = (1 << 0),  /* pie menu item collision is detected at 90 degrees */
-       UI_PIE_INITIAL_DIRECTION    = (1 << 1),  /* use initial center of pie menu to calculate direction */
-       UI_PIE_DRAG_STYLE           = (1 << 2),  /* pie menu is drag style */
-       UI_PIE_INVALID_DIR          = (1 << 3),  /* mouse not far enough from center position  */
-       UI_PIE_CLICK_STYLE          = (1 << 4),  /* pie menu changed to click style, click to confirm  */
-       UI_PIE_ANIMATION_FINISHED   = (1 << 5),  /* pie animation finished, do not calculate any more motion  */
-       UI_PIE_GESTURE_END_WAIT     = (1 << 6),  /* pie gesture selection has been done, now wait for mouse motion to end */
+       /** pie menu item collision is detected at 90 degrees */
+       UI_PIE_DEGREES_RANGE_LARGE  = (1 << 0),
+       /** use initial center of pie menu to calculate direction */
+       UI_PIE_INITIAL_DIRECTION    = (1 << 1),
+       /** pie menu is drag style */
+       UI_PIE_DRAG_STYLE           = (1 << 2),
+       /** mouse not far enough from center position  */
+       UI_PIE_INVALID_DIR          = (1 << 3),
+       /** pie menu changed to click style, click to confirm  */
+       UI_PIE_CLICK_STYLE          = (1 << 4),
+       /** pie animation finished, do not calculate any more motion  */
+       UI_PIE_ANIMATION_FINISHED   = (1 << 5),
+       /** pie gesture selection has been done, now wait for mouse motion to end */
+       UI_PIE_GESTURE_END_WAIT     = (1 << 6),
 };
 
 #define PIE_CLICK_THRESHOLD_SQ 50.0f
@@ -256,7 +264,7 @@ struct uiBut {
        void *rename_arg1;
        void *rename_orig;
 
-       /* Run an action when holding the button down. */
+       /** Run an action when holding the button down. */
        uiButHandleHoldFunc hold_func;
        void *hold_argN;
 
@@ -264,14 +272,18 @@ struct uiBut {
        uiButToolTipFunc tip_func;
        void *tip_argN;
 
-       /* info on why button is disabled, displayed in tooltip */
+       /** info on why button is disabled, displayed in tooltip */
        const char *disabled_info;
 
        BIFIconID icon;
-       char dt; /* drawtype: UI_EMBOSS, UI_EMBOSS_NONE ... etc, copied from the block */
-       signed char pie_dir; /* direction in a pie menu, used for collision detection (RadialDirection) */
-       bool changed; /* could be made into a single flag */
-       uchar unit_type; /* so buttons can support unit systems which are not RNA */
+       /** drawtype: UI_EMBOSS, UI_EMBOSS_NONE ... etc, copied from the block */
+       char dt;
+       /** direction in a pie menu, used for collision detection (RadialDirection) */
+       signed char pie_dir;
+       /** could be made into a single flag */
+       bool changed;
+       /** so buttons can support unit systems which are not RNA */
+       uchar unit_type;
        short modifier_key;
        short iconadd;
 
@@ -336,7 +348,7 @@ typedef struct ColorPickerData {
 } ColorPickerData;
 
 struct PieMenuData {
-       /* store title and icon to allow access when pie levels are created */
+       /** store title and icon to allow access when pie levels are created */
        const char *title;
        int icon;
 
@@ -346,13 +358,14 @@ struct PieMenuData {
        float last_pos[2];
        double duration_gesture;
        int flags;
-       int event; /* initial event used to fire the pie menu, store here so we can query for release */
+       /** initial event used to fire the pie menu, store here so we can query for release */
+       int event;
        float alphafac;
 };
 
 /* uiBlock.content_hints */
 enum eBlockContentHints {
-       /* In a menu block, if there is a single sub-menu button, we add some
+       /** In a menu block, if there is a single sub-menu button, we add some
         * padding to the right to put nicely aligned triangle icons there. */
        UI_BLOCK_CONTAINS_SUBMENU_BUT = (1 << 0),
 };
@@ -403,14 +416,16 @@ struct uiBlock {
 
        int flag;
        short alignnr;
-       /* Hints about the buttons of this block. Used to avoid iterating over
+       /** Hints about the buttons of this block. Used to avoid iterating over
         * buttons to find out if some criteria is met by any. Instead, check this
         * criteria when adding the button and set a flag here if it's met. */
        short content_hints; /* eBlockContentHints */
 
        char direction;
-       char theme_style; /* UI_BLOCK_THEME_STYLE_* */
-       char dt; /* drawtype: UI_EMBOSS, UI_EMBOSS_NONE ... etc, copied to buttons */
+       /** UI_BLOCK_THEME_STYLE_* */
+       char theme_style;
+       /** drawtype: UI_EMBOSS, UI_EMBOSS_NONE ... etc, copied to buttons */
+       char dt;
        bool auto_open;
        char _pad[5];
        double auto_open_last;
@@ -418,32 +433,46 @@ struct uiBlock {
        const char *lockstr;
 
        char lock;
-       char active;                /* to keep blocks while drawing and free them afterwards */
-       char tooltipdisabled;       /* to avoid tooltip after click */
-       char endblock;              /* UI_block_end done? */
-
-       eBlockBoundsCalc bounds_type;  /* for doing delayed */
+       /** to keep blocks while drawing and free them afterwards */
+       char active;
+       /** to avoid tooltip after click */
+       char tooltipdisabled;
+       /** UI_block_end done? */
+       char endblock;
+
+       /** for doing delayed */
+       eBlockBoundsCalc bounds_type;
        int mx, my;
-       int bounds, minbounds;      /* for doing delayed */
+       /** for doing delayed */
+       int bounds, minbounds;
 
-       rctf safety;                /* pulldowns, to detect outside, can differ per case how it is created */
-       ListBase saferct;           /* uiSafetyRct list */
+       /** pulldowns, to detect outside, can differ per case how it is created */
+       rctf safety;
+       /** uiSafetyRct list */
+       ListBase saferct;
 
        uiPopupBlockHandle *handle; /* handle */
 
-       struct wmOperator *ui_operator; /* use so presets can find the operator, */
-                                       /* across menus and from nested popups which fail for operator context. */
+       /** use so presets can find the operator,
+        * across menus and from nested popups which fail for operator context. */
+       struct wmOperator *ui_operator;
 
-       void *evil_C;               /* XXX hack for dynamic operator enums */
+       /** XXX hack for dynamic operator enums */
+       void *evil_C;
 
-       struct UnitSettings *unit;  /* unit system, used a lot for numeric buttons so include here rather then fetching through the scene every time. */
-       ColorPickerData color_pickers; /* XXX, only accessed by color picker templates */
+       /** unit system, used a lot for numeric buttons so include here
+        * rather then fetching through the scene every time. */
+       struct UnitSettings *unit;
+       /** \note only accessed by color picker templates. */
+       ColorPickerData color_pickers;
 
        bool is_color_gamma_picker; /* Block for color picker with gamma baked in. */
 
-       char display_device[64]; /* display device name used to display this block,
-                                 * used by color widgets to transform colors from/to scene linear
-                                 */
+       /** display device name used to display this block,
+        * used by color widgets to transform colors from/to scene linear
+        */
+       char display_device[64];
+
        struct PieMenuData pie_data;
 };
 
index 09033a5..6cd2ba0 100644 (file)
@@ -158,7 +158,7 @@ struct uiLayout {
        bContextStore *context;
        ListBase items;
 
-       /* Sub layout to add child items, if not the layout itself. */
+       /** Sub layout to add child items, if not the layout itself. */
        uiLayout *child_items_layout;
 
        int x, y, w, h;
@@ -169,10 +169,12 @@ struct uiLayout {
        bool enabled;
        bool redalert;
        bool keepaspect;
-       bool variable_size;  /* For layouts inside gridflow, they and their items shall never have a fixed maximal size. */
+       /** For layouts inside gridflow, they and their items shall never have a fixed maximal size. */
+       bool variable_size;
        char alignment;
        char emboss;
-       float units[2];  /* for fixed width or height to avoid UI size changes */
+       /** for fixed width or height to avoid UI size changes */
+       float units[2];
 };
 
 typedef struct uiLayoutItemFlow {
@@ -188,9 +190,12 @@ typedef struct uiLayoutItemGridFlow {
        bool row_major;     /* Fill first row first, instead of filling first column first. */
        bool even_columns;  /* Same width for all columns. */
        bool even_rows;     /* Same height for all rows. */
-       /* If positive, absolute fixed number of columns.
-        * If 0, fully automatic (based on available width).
-        * If negative, automatic but only generates number of columns/rows multiple of given (absolute) value. */
+       /**
+        * - If positive, absolute fixed number of columns.
+        * - If 0, fully automatic (based on available width).
+        * - If negative, automatic but only generates number of columns/rows
+        *   multiple of given (absolute) value.
+        */
        int columns_len;
 
        /* Pure internal runtime storage. */
@@ -274,8 +279,9 @@ static int ui_layout_vary_direction(uiLayout *layout)
 
 static bool ui_layout_variable_size(uiLayout *layout)
 {
-       /* Note that this code is probably a bit flacky, we'd probably want to know whether it's variable in X and/or Y,
-        * etc. But for now it mimics previous one, with addition of variable flag set for children of gridflow layouts. */
+       /* Note that this code is probably a bit flacky, we'd probably want to know whether it's
+        * variable in X and/or Y, etc. But for now it mimics previous one,
+        * with addition of variable flag set for children of gridflow layouts. */
        return ui_layout_vary_direction(layout) == UI_ITEM_VARY_X || layout->variable_size;
 }
 
@@ -697,7 +703,8 @@ static void ui_item_enum_expand_exec(
 
                        /* Separate items, potentially with a label. */
                        if (next_item->identifier) {
-                               /* Item without identifier but with name: Add group label for the following items. */
+                               /* Item without identifier but with name:
+                                * Add group label for the following items. */
                                if (item->name) {
                                        if (!is_first) {
                                                uiItemS(block->curlayout);
@@ -735,7 +742,8 @@ static void ui_item_enum_expand_exec(
                if (uiLayoutGetLocalDir(layout) != UI_LAYOUT_HORIZONTAL)
                        but->drawflag |= UI_BUT_TEXT_LEFT;
 
-               /* Allow quick, inaccurate swipe motions to switch tabs (no need to keep cursor over them). */
+               /* Allow quick, inaccurate swipe motions to switch tabs
+                * (no need to keep cursor over them). */
                if (but_type == UI_BTYPE_TAB) {
                        but->flag |= UI_BUT_DRAG_LOCK;
                }
@@ -806,7 +814,8 @@ static uiBut *ui_item_with_label(
                }
                else {
                        if (ui_layout_variable_size(layout)) {
-                               /* w_hint is width for label in this case. Use a default width for property button(s) */
+                               /* w_hint is width for label in this case.
+                                * Use a default width for property button(s) */
                                prop_but_width = UI_UNIT_X * 5;
                                w_label = w_hint;
                        }
@@ -1253,7 +1262,8 @@ void uiItemsFullEnumO_items(
                                        but = block->buttons.last;
                                }
                                else {
-                                       /* Do not use uiItemL here, as our root layout is a menu one, it will add a fake blank icon! */
+                                       /* Do not use uiItemL here, as our root layout is a menu one,
+                                        * it will add a fake blank icon! */
                                        but = uiDefBut(
                                                block, UI_BTYPE_LABEL, 0, item->name, 0, 0, UI_UNIT_X * 5, UI_UNIT_Y, NULL,
                                                0.0, 0.0, 0, 0, "");
@@ -1262,7 +1272,8 @@ void uiItemsFullEnumO_items(
                        }
                        else {
                                if (radial) {
-                                       /* invisible dummy button to ensure all items are always at the same position */
+                                       /* invisible dummy button to ensure all items are
+                                        * always at the same position */
                                        uiItemS(target);
                                }
                                else {
@@ -1751,7 +1762,8 @@ void uiItemFullR(uiLayout *layout, PointerRNA *ptr, PropertyRNA *prop, int index
                        if ((type == PROP_ENUM) && (flag & UI_ITEM_R_EXPAND)) {
                                /* Expanded enums each have their own name. */
 
-                               /* Often expanded enum's are better arranged into a row, so check the existing layout. */
+                               /* Often expanded enum's are better arranged into a row,
+                                * so check the existing layout. */
                                if (uiLayoutGetLocalDir(layout) == UI_LAYOUT_HORIZONTAL) {
                                        layout = uiLayoutRow(layout_split, true);
                                }
@@ -2106,7 +2118,8 @@ void ui_but_add_search(uiBut *but, PointerRNA *ptr, PropertyRNA *prop, PointerRN
                but->free_search_arg = true;
        }
        else if (but->type == UI_BTYPE_SEARCH_MENU) {
-               /* In case we fail to find proper searchprop, so other code might have already set but->type to search menu... */
+               /* In case we fail to find proper searchprop,
+                * so other code might have already set but->type to search menu... */
                but->flag |= UI_BUT_DISABLED;
        }
 }
@@ -2262,7 +2275,8 @@ static uiBut *ui_item_menu(
        }
 
        if (ELEM(layout->root->type, UI_LAYOUT_PANEL, UI_LAYOUT_TOOLBAR) ||
-           (force_menu && layout->root->type != UI_LAYOUT_MENU))  /* We never want a dropdown in menu! */
+           /* We never want a dropdown in menu! */
+           (force_menu && layout->root->type != UI_LAYOUT_MENU))
        {
                UI_but_type_set_menu_from_pulldown(but);
        }
@@ -3182,7 +3196,8 @@ typedef struct UILayoutGridFlowInput {
        const bool even_rows : 1;  /* All rows will have same height. */
        const int space_x;  /* Space between columns. */
        const int space_y;  /* Space between rows. */
-       /* Real data about current position and size of this layout item (either estimated, or final values). */
+       /* Real data about current position and size of this layout item
+        * (either estimated, or final values). */
        const int litem_w;  /* Layout item width. */
        const int litem_x;  /* Layout item X position. */
        const int litem_y;  /* Layout item Y position. */
@@ -3430,7 +3445,8 @@ static void ui_litem_estimate_grid_flow(uiLayout *litem)
                        }
                }
 
-               /* Set evenly-spaced axes size (quick optimization in case we have even columns and rows). */
+               /* Set evenly-spaced axes size
+                * (quick optimization in case we have even columns and rows). */
                if (gflow->even_columns && gflow->even_rows) {
                        litem->w = (int)(gflow->tot_columns * avg_w) + space_x * (gflow->tot_columns - 1);
                        litem->h = (int)(gflow->tot_rows * max_h) + space_y * (gflow->tot_rows - 1);
index 484debf..31ae3bc 100644 (file)
@@ -444,7 +444,8 @@ static int override_type_set_button_exec(bContext *C, wmOperator *op)
                        operation = IDOVERRIDESTATIC_OP_REPLACE;
                        break;
                case UIOverride_Type_Difference:
-                       operation = IDOVERRIDESTATIC_OP_ADD;  /* override code will automatically switch to subtract if needed. */
+                       /* override code will automatically switch to subtract if needed. */
+                       operation = IDOVERRIDESTATIC_OP_ADD;
                        break;
                case UIOverride_Type_Factor:
                        operation = IDOVERRIDESTATIC_OP_MULTIPLY;
@@ -549,13 +550,15 @@ static int override_remove_button_exec(bContext *C, wmOperator *op)
 
        if (!all && index != -1) {
                bool is_strict_find;
-               /* Remove override operation for given item, add singular operations for the other items as needed. */
+               /* Remove override operation for given item,
+                * add singular operations for the other items as needed. */
                IDOverrideStaticPropertyOperation *opop = BKE_override_static_property_operation_find(
                        oprop, NULL, NULL, index, index, false, &is_strict_find);
                BLI_assert(opop != NULL);
                if (!is_strict_find) {
                        /* No specific override operation, we have to get generic one,
-                        * and create item-specific override operations for all but given index, before removing generic one. */
+                        * and create item-specific override operations for all but given index,
+                        * before removing generic one. */
                        for (int idx = RNA_property_array_length(&ptr, prop); idx--; ) {
                                if (idx != index) {
                                        BKE_override_static_property_operation_get(oprop, opop->operation, NULL, NULL, idx, idx, true, NULL, NULL);
@@ -735,7 +738,8 @@ bool UI_context_copy_to_selected_list(
                }
                else if (GS(id->name) == ID_SCE) {
                        /* Sequencer's ID is scene :/ */
-                       /* Try to recursively find an RNA_Sequence ancestor, to handle situations like T41062... */
+                       /* Try to recursively find an RNA_Sequence ancestor,
+                        * to handle situations like T41062... */
                        if ((*r_path = RNA_path_resolve_from_type_to_property(ptr, prop, &RNA_Sequence)) != NULL) {
                                *r_lb = CTX_data_collection_get(C, "selected_editable_sequences");
                        }
index c53aade..1720eec 100644 (file)
@@ -28,7 +28,8 @@
  */
 
 
-/* a full doc with API notes can be found in bf-blender/trunk/blender/doc/guides/interface_API.txt */
+/* a full doc with API notes can be found in
+ * bf-blender/trunk/blender/doc/guides/interface_API.txt */
 
 #include <math.h>
 #include <stdlib.h>
@@ -1872,8 +1873,10 @@ void UI_panel_category_draw_all(ARegion *ar, const char *category_id_active)
        const int px_x_sign = is_left ? px : -px;
        const int category_tabs_width = round_fl_to_int(UI_PANEL_CATEGORY_MARGIN_WIDTH * zoom);
        const float dpi_fac = UI_DPI_FAC;
-       const int tab_v_pad_text = round_fl_to_int((2 + ((px * 3) * dpi_fac)) * zoom);  /* padding of tabs around text */
-       const int tab_v_pad = round_fl_to_int((4 + (2 * px * dpi_fac)) * zoom);  /* padding between tabs */
+       /* padding of tabs around text */
+       const int tab_v_pad_text = round_fl_to_int((2 + ((px * 3) * dpi_fac)) * zoom);
+       /* padding between tabs */
+       const int tab_v_pad = round_fl_to_int((4 + (2 * px * dpi_fac)) * zoom);
        const float tab_curve_radius = ((px * 3) * dpi_fac) * zoom;
        /* We flip the tab drawing, so always use these flags. */
        const int roundboxtype = UI_CNR_TOP_LEFT | UI_CNR_BOTTOM_LEFT;
@@ -2153,7 +2156,8 @@ static int ui_handle_panel_category_cycling(const wmEvent *event, ARegion *ar, c
                                        const bool backwards = event->shift;
                                        pc_dyn = backwards ? pc_dyn->prev : pc_dyn->next;
                                        if (!pc_dyn) {
-                                               /* proper cyclic behavior, back to first/last category (only used for ctrl+tab) */
+                                               /* proper cyclic behavior,
+                                                * back to first/last category (only used for ctrl+tab) */
                                                pc_dyn = backwards ? ar->panels_category.last : ar->panels_category.first;
                                        }
                                }
@@ -2225,10 +2229,13 @@ int ui_handler_panel_region(bContext *C, const wmEvent *event, ARegion *ar, cons
                /* checks for mouse position inside */
                pa = block->panel;
 
-               if (!pa || pa->paneltab != NULL)
+               if (!pa || pa->paneltab != NULL) {
                        continue;
-               if (pa->type && pa->type->flag & PNL_NO_HEADER)  /* XXX - accessed freed panels when scripts reload, need to fix. */
+               }
+               /* XXX - accessed freed panels when scripts reload, need to fix. */
+               if (pa->type && pa->type->flag & PNL_NO_HEADER) {
                        continue;
+               }
 
                mouse_state = ui_panel_mouse_state_get(block, pa, mx, my);
 
index 819fbc8..b084d66 100644 (file)
@@ -175,7 +175,8 @@ static void ui_update_color_picker_buts_rgb(
        }
        ui_rgb_to_color_picker_compat_v(tmp, hsv);
 
-       /* this updates button strings, is hackish... but button pointers are on stack of caller function */
+       /* this updates button strings,
+        * is hackish... but button pointers are on stack of caller function */
        for (bt = block->buttons.first; bt; bt = bt->next) {
                if (bt->custom_data != cpicker)
                        continue;
@@ -193,7 +194,8 @@ static void ui_update_color_picker_buts_rgb(
                        double intpart;
                        char col[16];
 
-                       /* Hex code is assumed to be in sRGB space (coming from other applications, web, etc) */
+                       /* Hex code is assumed to be in sRGB space
+                        * (coming from other applications, web, etc) */
                        copy_v3_v3(rgb_hex, rgb);
                        if (from_but && !ui_but_is_color_gamma(from_but)) {
                                IMB_colormanagement_scene_linear_to_srgb_v3(rgb_hex);
index 44effc8..eae2b01 100644 (file)
@@ -126,7 +126,8 @@ uiPieMenu *UI_pie_menu_begin(struct bContext *C, const char *title, int icon, co
        pie->block_radial->puphash = ui_popup_menu_hash(title);
        pie->block_radial->flag |= UI_BLOCK_RADIAL;
 
-       /* if pie is spawned by a left click, release or click event, it is always assumed to be click style */
+       /* if pie is spawned by a left click, release or click event,
+        * it is always assumed to be click style */
        if (event->type == LEFTMOUSE || ELEM(event->val, KM_RELEASE, KM_CLICK)) {
                pie->block_radial->pie_data.flags |= UI_PIE_CLICK_STYLE;
                pie->block_radial->pie_data.event = EVENT_NONE;
index e136614..da5bd30 100644 (file)
@@ -393,7 +393,8 @@ uiPopupMenu *UI_popup_menu_begin_ex(bContext *C, const char *title, const char *
        pup->layout = UI_block_layout(pup->block, UI_LAYOUT_VERTICAL, UI_LAYOUT_MENU, 0, 0, 200, 0, MENU_PADDING, style);
 
        /* note, this intentionally differs from the menu & submenu default because many operators
-        * use popups like this to select one of their options - where having invoke doesn't make sense */
+        * use popups like this to select one of their options -
+        * where having invoke doesn't make sense */
        uiLayoutSetOperatorContext(pup->layout, WM_OP_EXEC_REGION_WIN);
 
        /* create in advance so we can let buttons point to retval already */
@@ -640,7 +641,8 @@ void UI_popup_block_close(bContext *C, wmWindow *win, uiBlock *block)
                        UI_popup_handlers_remove(&win->modalhandlers, block->handle);
                        ui_popup_block_free(C, block->handle);
 
-                       /* In the case we have nested popups, closing one may need to redraw another, see: T48874 */
+                       /* In the case we have nested popups,
+                        * closing one may need to redraw another, see: T48874 */
                        for (ARegion *ar = screen->regionbase.first; ar; ar = ar->next) {
                                ED_region_tag_refresh_ui(ar);
                        }
index cb4939a..e5607f6 100644 (file)
@@ -165,7 +165,8 @@ static uiBlock *ui_block_func_POPOVER(bContext *C, uiPopupBlockHandle *handle, v
                if (!handle->refresh) {
                        float center[2] = {BLI_rctf_cent_x(&pup->but->rect), BLI_rctf_cent_y(&pup->but->rect)};
                        ui_block_to_window_fl(handle->ctx_region, pup->but->block, &center[0], &center[1]);
-                       /* These variables aren't used for popovers, we could add new variables if there is a conflict. */
+                       /* These variables aren't used for popovers,
+                        * we could add new variables if there is a conflict. */
                        handle->prev_mx = block->mx = (int)center[0];
                        handle->prev_my = block->my = (int)center[1];
                }
index d6e1fcd..9091bb1 100644 (file)
@@ -90,10 +90,14 @@ typedef struct uiSearchboxData {
        rcti bbox;
        uiFontStyle fstyle;
        uiSearchItems items;
-       int active;     /* index in items array */
-       bool noback;    /* when menu opened with enough space for this */
-       bool preview;   /* draw thumbnail previews, rather than list */
-       bool use_sep;   /* use the UI_SEP_CHAR char for splitting shortcuts (good for operators, bad for data) */
+       /** index in items array */
+       int active;
+       /** when menu opened with enough space for this */
+       bool noback;
+       /** draw thumbnail previews, rather than list */
+       bool preview;
+       /** use the UI_SEP_CHAR char for splitting shortcuts (good for operators, bad for data) */
+       bool use_sep;
        int prv_rows, prv_cols;
 } uiSearchboxData;
 
@@ -262,7 +266,8 @@ bool ui_searchbox_apply(uiBut *but, ARegion *ar)
                return true;
        }
        else if (but->flag & UI_BUT_VALUE_CLEAR) {
-               /* It is valid for _VALUE_CLEAR flavor to have no active element (it's a valid way to unlink). */
+               /* It is valid for _VALUE_CLEAR flavor to have no active element
+                * (it's a valid way to unlink). */
                but->editstr[0] = '\0';
 
                return true;
index 94bed87..b4df8d2 100644 (file)
@@ -520,7 +520,8 @@ void uiStyleInit(void)
        }
 #endif
 
-       /* XXX, this should be moved into a style, but for now best only load the monospaced font once. */
+       /* XXX, this should be moved into a style,
+        * but for now best only load the monospaced font once. */
        BLI_assert(blf_mono_font == -1);
        if (U.font_path_ui_mono[0]) {
                blf_mono_font = BLF_load_unique(U.font_path_ui_mono);
index 7cf68a1..ed3e2e8 100644 (file)
@@ -129,8 +129,12 @@ static void template_add_button_search_menu(
                ARegion *region = CTX_wm_region(C);
                ScrArea *area = CTX_wm_area(C);
                /* XXX ugly top-bar exception */
-               const bool use_big_size = (region->regiontype != RGN_TYPE_HEADER) && (area->spacetype != SPACE_TOPBAR); /* silly check, could be more generic */
-               /* Ugly exception for screens here, drawing their preview in icon size looks ugly/useless */
+               const bool use_big_size = (
+                       /* silly check, could be more generic */
+                       (region->regiontype != RGN_TYPE_HEADER) &&
+                       (area->spacetype != SPACE_TOPBAR));
+               /* Ugly exception for screens here,
+                * drawing their preview in icon size looks ugly/useless */
                const bool use_preview_icon = use_big_size || (id && (GS(id->name) != ID_SCR));
                const short width = UI_UNIT_X * (use_big_size ? 6 : 1.6f);
                const short height = UI_UNIT_Y * (use_big_size ? 6 : 1);
@@ -1048,7 +1052,9 @@ void uiTemplateAnyID(
        }
 
        /* Start drawing UI Elements using standard defines */
-       split = uiLayoutSplit(layout, 0.33f, false);  /* NOTE: split amount here needs to be synced with normal labels */
+
+       /* NOTE: split amount here needs to be synced with normal labels */
+       split = uiLayoutSplit(layout, 0.33f, false);
 
        /* FIRST PART ................................................ */
        row = uiLayoutRow(split, false);
@@ -1066,14 +1072,20 @@ void uiTemplateAnyID(
        row = uiLayoutRow(split, true);
 
        /* ID-Type Selector - just have a menu of icons */
-       sub = uiLayoutRow(row, true);                     /* HACK: special group just for the enum, otherwise we */
-       uiLayoutSetAlignment(sub, UI_LAYOUT_ALIGN_LEFT);  /*       we get ugly layout with text included too...  */
+
+       /* HACK: special group just for the enum,
+        * otherwise we get ugly layout with text included too... */
+       sub = uiLayoutRow(row, true);
+       uiLayoutSetAlignment(sub, UI_LAYOUT_ALIGN_LEFT);
 
        uiItemFullR(sub, ptr, propType, 0, 0, UI_ITEM_R_ICON_ONLY, "", ICON_NONE);
 
        /* ID-Block Selector - just use pointer widget... */
-       sub = uiLayoutRow(row, true);                       /* HACK: special group to counteract the effects of the previous */
-       uiLayoutSetAlignment(sub, UI_LAYOUT_ALIGN_EXPAND);  /*       enum, which now pushes everything too far right         */
+
+       /* HACK: special group to counteract the effects of the previous enum,
+        * which now pushes everything too far right. */
+       sub = uiLayoutRow(row, true);
+       uiLayoutSetAlignment(sub, UI_LAYOUT_ALIGN_EXPAND);
 
        uiItemFullR(sub, ptr, propID, 0, 0, 0, "", ICON_NONE);
 }
@@ -1312,7 +1324,8 @@ void uiTemplatePathBuilder(
        /* Path (existing string) Widget */
        uiItemR(row, ptr, propname, 0, text, ICON_RNA);
 
-       /* TODO: attach something to this to make allow searching of nested properties to 'build' the path */
+       /* TODO: attach something to this to make allow
+        * searching of nested properties to 'build' the path */
 }
 
 /************************ Modifier Template *************************/
@@ -1350,7 +1363,8 @@ static int modifier_can_delete(ModifierData *md)
        return 1;
 }
 
-/* Check whether Modifier is a simulation or not, this is used for switching to the physics/particles context tab */
+/* Check whether Modifier is a simulation or not,
+ * this is used for switching to the physics/particles context tab */
 static int modifier_is_simulation(ModifierData *md)
 {
        /* Physic Tab */
@@ -1477,7 +1491,8 @@ static uiLayout *draw_modifier(
                UI_block_align_end(block);
 
                UI_block_emboss_set(block, UI_EMBOSS_NONE);
-               /* When Modifier is a simulation, show button to switch to context rather than the delete button. */
+               /* When Modifier is a simulation,
+                * show button to switch to context rather than the delete button. */
                if (modifier_can_delete(md) &&
                    !modifier_is_simulation(md))
                {
@@ -1542,7 +1557,8 @@ static uiLayout *draw_modifier(
                        }
                }
 
-               /* result is the layout block inside the box, that we return so that modifier settings can be drawn */
+               /* result is the layout block inside the box,
+                * that we return so that modifier settings can be drawn */
                result = uiLayoutColumn(box, false);
                block = uiLayoutAbsoluteBlock(box);
        }
@@ -1679,7 +1695,8 @@ static uiLayout *gpencil_draw_modifier(
                uiItemO(row, CTX_IFACE_(BLT_I18NCONTEXT_OPERATOR_DEFAULT, "Copy"), ICON_NONE,
                        "OBJECT_OT_gpencil_modifier_copy");
 
-               /* result is the layout block inside the box, that we return so that modifier settings can be drawn */
+               /* result is the layout block inside the box,
+                * that we return so that modifier settings can be drawn */
                result = uiLayoutColumn(box, false);
                block = uiLayoutAbsoluteBlock(box);
        }
@@ -1797,7 +1814,8 @@ static uiLayout *gpencil_draw_shaderfx(
                /* only here obdata, the rest of effect is ob level */
                UI_block_lock_set(block, BKE_object_obdata_is_libdata(ob), ERROR_LIBDATA_MESSAGE);
 
-               /* result is the layout block inside the box, that we return so that effect settings can be drawn */
+               /* result is the layout block inside the box,
+                * that we return so that effect settings can be drawn */
                result = uiLayoutColumn(box, false);
                block = uiLayoutAbsoluteBlock(box);
        }
@@ -2022,7 +2040,8 @@ static uiLayout *draw_constraint(uiLayout *layout, Object *ob, bConstraint *con)
        if (proxy_protected) {
                UI_block_emboss_set(block, UI_EMBOSS_NONE);
 
-               /* draw a ghost icon (for proxy) and also a lock beside it, to show that constraint is "proxy locked" */
+               /* draw a ghost icon (for proxy) and also a lock beside it,
+                * to show that constraint is "proxy locked" */
                uiDefIconBut(block, UI_BTYPE_BUT, B_CONSTRAINT_TEST, ICON_GHOST_ENABLED, xco + 12.2f * UI_UNIT_X, yco, 0.95f * UI_UNIT_X, 0.95f * UI_UNIT_Y,
                             NULL, 0.0, 0.0, 0.0, 0.0, TIP_("Proxy Protected"));
                uiDefIconBut(block, UI_BTYPE_BUT, B_CONSTRAINT_TEST, ICON_LOCKED, xco + 13.1f * UI_UNIT_X, yco, 0.95f * UI_UNIT_X, 0.95f * UI_UNIT_Y,
@@ -3057,7 +3076,8 @@ static void curvemap_buttons_reset(bContext *C, void *cb_v, void *cumap_v)
        rna_update_cb(C, cb_v, NULL);
 }
 
-/* still unsure how this call evolves... we use labeltype for defining what curve-channels to show */
+/* still unsure how this call evolves...
+ * we use labeltype for defining what curve-channels to show */
 static void curvemap_buttons_layout(
         uiLayout *layout, PointerRNA *ptr, char labeltype, bool levels,
         bool brush, bool neg_slope, bool tone, RNAUpdateCb *cb)
@@ -3756,7 +3776,8 @@ static void uilist_prepare(
                rows = min_ii(dyn_data->height, maxrows);
        }
 
-       /* If list length changes or list is tagged to check this, and active is out of view, scroll to it .*/
+       /* If list length changes or list is tagged to check this,
+        * and active is out of view, scroll to it .*/
        if (ui_list->list_last_len != len || ui_list->flag & UILST_SCROLL_TO_ACTIVE_ITEM) {
                if (activei_row < ui_list->list_scroll) {
                        ui_list->list_scroll = activei_row;
@@ -4397,7 +4418,8 @@ eAutoPropButsReturn uiTemplateOperatorPropertyButs(
 
        /* menu */
        if (op->type->flag & OPTYPE_PRESET) {
-               /* XXX, no simple way to get WM_MT_operator_presets.bl_label from python! Label remains the same always! */
+               /* XXX, no simple way to get WM_MT_operator_presets.bl_label
+                * from python! Label remains the same always! */
                PointerRNA op_ptr;
                uiLayout *row;
 
@@ -4868,7 +4890,8 @@ void uiTemplateKeymapItemProperties(uiLayout *layout, PointerRNA *ptr)
                        if (but->rnaprop) {
                                UI_but_func_set(but, keymap_item_modified, ptr->data, NULL);
 
-                               /* Otherwise the keymap will be re-generated which we're trying to edit, see: T47685 */
+                               /* Otherwise the keymap will be re-generated which we're trying to edit,
+                                * see: T47685 */
                                UI_but_flag_enable(but, UI_BUT_UPDATE_DELAY);
                        }
                }
index a7787d1..5d26250 100644 (file)
@@ -278,7 +278,8 @@ void ui_rna_collection_search_cb(const struct bContext *C, void *arg, const char
                        iconid = ui_id_icon_get(C, itemptr.data, false);
                }
                else {
-                       name = RNA_struct_name_get_alloc(&itemptr, NULL, 0, NULL); /* could use the string length here */
+                       /* could use the string length here */
+                       name = RNA_struct_name_get_alloc(&itemptr, NULL, 0, NULL);
                }
 
                if (name) {
index fcb995d..a531306 100644 (file)
@@ -368,7 +368,8 @@ GPUBatch *ui_batch_roundbox_widget_get(int tria)
                set_roundbox_vertex_data(&vflag_step, last_data);
                set_roundbox_vertex(&vflag_step, 0, 0, 0, false, false, EMBOSS);
                /* Emboss */
-               bool rev = false; /* go back and forth : avoid degenerate triangle (but beware of backface cull) */
+               /* go back and forth : avoid degenerate triangle (but beware of backface cull) */
+               bool rev = false;
                for (int j = 0; j < WIDGET_AA_JITTER; j++, rev = !rev) {
                        for (int c = (rev) ? 1 : 0; (rev) ? c >= 0 : c < 2; (rev) ? c-- : c++) {
                                int sta = (rev) ? WIDGET_CURVE_RESOLU - 1 : 0;
@@ -685,7 +686,8 @@ static void round_box__edges(uiWidgetBase *wt, int roundboxalign, const rcti *re
        float maxxi = maxx - U.pixelsize;
        float minyi = miny + U.pixelsize;
        float maxyi = maxy - U.pixelsize;
-       float facxi = (maxxi != minxi) ? 1.0f / (maxxi - minxi) : 0.0f; /* for uv, can divide by zero */
+       /* for uv, can divide by zero */
+       float facxi = (maxxi != minxi) ? 1.0f / (maxxi - minxi) : 0.0f;
        float facyi = (maxyi != minyi) ? 1.0f / (maxyi - minyi) : 0.0f;
        int a, tot = 0, minsize;
        const int hnum = (
@@ -1448,7 +1450,8 @@ float UI_text_clip_middle_ex(
        float strwidth;
 
        /* Add some epsilon to OK width, avoids 'ellipsing' text that nearly fits!
-        * Better to have a small piece of the last char cut out, than two remaining chars replaced by an ellipsis... */
+        * Better to have a small piece of the last char cut out,
+        * than two remaining chars replaced by an ellipsis... */
        okwidth += 1.0f + UI_DPI_FAC;
 
        BLI_assert(str[0]);
@@ -1463,7 +1466,8 @@ float UI_text_clip_middle_ex(
        strwidth = BLF_width(fstyle->uifont_id, str, max_len);
 
        if ((okwidth > 0.0f) && (strwidth > okwidth)) {
-               /* utf8 two-dots leader '..' (shorter than ellipsis '...'), some compilers complain with real litteral string. */
+               /* utf8 two-dots leader '..' (shorter than ellipsis '...'),
+                * some compilers complain with real litteral string. */
                const char sep[] = {0xe2, 0x80, 0xA5, 0x0};
                const int sep_len = sizeof(sep) - 1;
                const float sep_strwidth = BLF_width(fstyle->uifont_id, sep, sep_len + 1);
@@ -1527,7 +1531,8 @@ float UI_text_clip_middle_ex(
                        else {
                                memmove(str + l_end + sep_len, str + r_offset, r_len);
                                memcpy(str + l_end, sep, sep_len);
-                               final_lpart_len = (size_t)(l_end + sep_len + r_len - 1);  /* -1 to remove trailing '\0'! */
+                               /* -1 to remove trailing '\0'! */
+                               final_lpart_len = (size_t)(l_end + sep_len + r_len - 1);
 
                                while (BLF_width(fstyle->uifont_id, str, max_len) > okwidth) {
                                        /* This will happen because a lot of string width processing is done in integer pixels,
@@ -1690,7 +1695,8 @@ static void ui_text_clip_right_label(const uiFontStyle *fstyle, uiBut *but, cons
                        const char *prev_utf8 = BLI_str_find_prev_char_utf8(but->drawstr, cp2);
                        int bytes = cp2 - prev_utf8;
 
-                       /* shift the text after and including cp2 back by 1 char, +1 to include null terminator */
+                       /* shift the text after and including cp2 back by 1 char,
+                        * +1 to include null terminator */
                        memmove(cp2 - bytes, cp2, drawstr_len + 1);
                        cp2 -= bytes;
 
@@ -2323,7 +2329,8 @@ static void widget_state(uiWidgetType *wt, int state, int drawflag)
 static void widget_state_numslider(uiWidgetType *wt, int state, int drawflag)
 {
        uiWidgetStateColors *wcol_state = wt->wcol_state;
-       float blend = wcol_state->blend - 0.2f; /* XXX special tweak to make sure that bar will still be visible */
+       /* XXX special tweak to make sure that bar will still be visible */
+       float blend = wcol_state->blend - 0.2f;
 
        /* call this for option button */
        widget_state(wt, state, drawflag);
@@ -2756,7 +2763,9 @@ void ui_draw_gradient(const rcti *rect, const float hsv[3], const int type, cons
        immBindBuiltinProgram(GPU_SHADER_2D_SMOOTH_COLOR);
 
        immBegin(GPU_PRIM_TRIS, steps * 3 * 6);
-       for (dx = 0.0f; dx < 0.999f; dx += color_step) { /* 0.999 = prevent float inaccuracy for steps */
+
+       /* 0.999 = prevent float inaccuracy for steps */
+       for (dx = 0.0f; dx < 0.999f; dx += color_step) {
                const float dx_next = dx + color_step;
 
                /* previous color */
@@ -3366,7 +3375,8 @@ static void widget_numslider(uiBut *but, uiWidgetColors *wcol, rcti *rect, int s
        wtb.draw_inner = false;
        widgetbase_draw(&wtb, wcol);
 
-       /* Add space at either side of the button so text aligns with numbuttons (which have arrow icons). */
+       /* Add space at either side of the button so text aligns with numbuttons
+        * (which have arrow icons). */
        if (!(state & UI_STATE_TEXT_INPUT)) {
                rect->xmax -= toffs;
                rect->xmin += toffs;
@@ -3763,7 +3773,8 @@ static void widget_tab(uiWidgetColors *wcol, rcti *rect, int state, int roundbox
        const float rad = wcol->roundness * U.widget_unit;
        const bool is_active = (state & UI_SELECT);
 
-/* Draw shaded outline - Disabled for now, seems incorrect and also looks nicer without it imho ;) */
+/* Draw shaded outline - Disabled for now,
+ * seems incorrect and also looks nicer without it imho ;) */
 //#define USE_TAB_SHADED_HIGHLIGHT
 
        uiWidgetBase wtb;
@@ -4252,7 +4263,8 @@ void ui_draw_but(const bContext *C, ARegion *ar, uiStyle *style, uiBut *but, rct
                                break;
 
                        case UI_BTYPE_HSVCUBE:
-                               if (ELEM(but->a1, UI_GRAD_V_ALT, UI_GRAD_L_ALT)) {  /* vertical V slider, uses new widget draw now */
+                               if (ELEM(but->a1, UI_GRAD_V_ALT, UI_GRAD_L_ALT)) {
+                                       /* vertical V slider, uses new widget draw now */
                                        ui_draw_but_HSV_v(but, rect);
                                }
                                else {  /* other HSV pickers... */
@@ -4745,7 +4757,8 @@ void ui_draw_menu_item(const uiFontStyle *fstyle, rcti *rect, const char *name,
                aspect = ICON_DEFAULT_HEIGHT / height;
 
                GPU_blend(true);
-               UI_icon_draw_aspect(xs, ys, iconid, aspect, 1.0f, wt->wcol.text); /* XXX scale weak get from fstyle? */
+               /* XXX scale weak get from fstyle? */
+               UI_icon_draw_aspect(xs, ys, iconid, aspect, 1.0f, wt->wcol.text);
                GPU_blend(false);
        }
 }
index 3aadfeb..3f8f09d 100644 (file)
@@ -236,7 +236,8 @@ const uchar *UI_ThemeGetColorPtr(bTheme *btheme, int spacetype, int colorid)
                                case TH_HEADER:
                                        cp = ts->header; break;
                                case TH_HEADERDESEL:
-                                       /* we calculate a dynamic builtin header deselect color, also for pulldowns... */
+                                       /* we calculate a dynamic builtin header deselect color,
+                                        * also for pulldowns... */
                                        cp = ts->header;
                                        headerdesel[0] = cp[0] > 10 ? cp[0] - 10 : 0;
                                        headerdesel[1] = cp[1] > 10 ? cp[1] - 10 : 0;
index 6352dac..fb8e7e8 100644 (file)
@@ -234,7 +234,8 @@ void UI_view2d_region_reinit(View2D *v2d, short type, int winx, int winy)
                 */
                case V2D_COMMONVIEW_STANDARD:
                {
-                       /* for now, aspect ratio should be maintained, and zoom is clamped within sane default limits */
+                       /* for now, aspect ratio should be maintained,
+                        * and zoom is clamped within sane default limits */
                        v2d->keepzoom = (V2D_KEEPASPECT | V2D_LIMITZOOM);
                        v2d->minzoom = 0.01f;
                        v2d->maxzoom = 1000.0f;
@@ -289,7 +290,8 @@ void UI_view2d_region_reinit(View2D *v2d, short type, int winx, int winy)
                        /* scroller settings are currently not set here... that is left for regions... */
                        break;
                }
-               /* 'header' regions - zoom, aspect ratio, alignment, and panning restrictions are set here */
+               /* 'header' regions - zoom, aspect ratio,
+                * alignment, and panning restrictions are set here */
                case V2D_COMMONVIEW_HEADER:
                {
                        /* zoom + aspect ratio are locked */
@@ -322,7 +324,8 @@ void UI_view2d_region_reinit(View2D *v2d, short type, int winx, int winy)
                case V2D_COMMONVIEW_PANELS_UI:
                {
 
-                       /* for now, aspect ratio should be maintained, and zoom is clamped within sane default limits */
+                       /* for now, aspect ratio should be maintained,
+                        * and zoom is clamped within sane default limits */
                        v2d->keepzoom = (V2D_KEEPASPECT | V2D_LIMITZOOM | V2D_KEEPZOOM);
                        v2d->minzoom = 0.5f;
                        v2d->maxzoom = 2.0f;
@@ -357,7 +360,8 @@ void UI_view2d_region_reinit(View2D *v2d, short type, int winx, int winy)
                }
                /* other view types are completely defined using their own settings already */
                default:
-                       /* we don't do anything here, as settings should be fine, but just make sure that rect */
+                       /* we don't do anything here,
+                        * as settings should be fine, but just make sure that rect */
                        break;
        }
 
@@ -377,7 +381,8 @@ void UI_view2d_region_reinit(View2D *v2d, short type, int winx, int winy)
        }
 
        /* set 'tot' rect before setting cur? */
-       /* XXX confusing stuff here still - I made this function not check scroller hide - that happens in totrect_set */
+       /* XXX confusing stuff here still -
+        * I made this function not check scroller hide - that happens in totrect_set */
        if (tot_changed)
                UI_view2d_totRect_set_resize(v2d, winx, winy, !do_init);
        else
@@ -486,7 +491,8 @@ static void ui_view2d_curRect_validate_resize(View2D *v2d, bool resize, bool mas
                }
        }
        else {
-               /* make sure sizes don't exceed that of the min/max sizes (even though we're not doing zoom clamping) */
+               /* make sure sizes don't exceed that of the min/max sizes
+                * (even though we're not doing zoom clamping) */
                CLAMP(width, v2d->min[0], v2d->max[0]);
                CLAMP(height, v2d->min[1], v2d->max[1]);
        }
@@ -576,7 +582,8 @@ static void ui_view2d_curRect_validate_resize(View2D *v2d, bool resize, bool mas
                v2d->oldwiny = (short)winy;
        }
 
-       /* Step 2: apply new sizes to cur rect, but need to take into account alignment settings here... */
+       /* Step 2: apply new sizes to cur rect,
+        * but need to take into account alignment settings here... */
        if ((width != curwidth) || (height != curheight)) {
                float temp, dh;
 
@@ -675,7 +682,8 @@ static void ui_view2d_curRect_validate_resize(View2D *v2d, bool resize, bool mas
                         * (XXX - in the past, max was favored... if there are bugs, swap!)
                         */
                        if ((cur->xmin < tot->xmin) && (cur->xmax > tot->xmax)) {
-                               /* outside boundaries on both sides, so take middle-point of tot, and place in balanced way */
+                               /* outside boundaries on both sides,
+                                * so take middle-point of tot, and place in balanced way */
                                temp = BLI_rctf_cent_x(tot);
                                diff = curwidth * 0.5f;
 
@@ -725,7 +733,8 @@ static void ui_view2d_curRect_validate_resize(View2D *v2d, bool resize, bool mas
                         * We favour moving the 'minimum' across, as that's origin for most things
                         */
                        if ((cur->ymin < tot->ymin) && (cur->ymax > tot->ymax)) {
-                               /* outside boundaries on both sides, so take middle-point of tot, and place in balanced way */
+                               /* outside boundaries on both sides,
+                                * so take middle-point of tot, and place in balanced way */
                                temp = BLI_rctf_cent_y(tot);
                                diff = curheight * 0.5f;
 
@@ -803,7 +812,8 @@ void UI_view2d_curRect_validate(View2D *v2d)
 
 /* ------------------ */
 
-/* Called by menus to activate it, or by view2d operators to make sure 'related' views stay in synchrony */
+/* Called by menus to activate it, or by view2d operators
+ * to make sure 'related' views stay in synchrony */
 void UI_view2d_sync(bScreen *screen, ScrArea *area, View2D *v2dcur, int flag)
 {
        ScrArea *sa;
@@ -992,7 +1002,8 @@ void UI_view2d_totRect_set(View2D *v2d, int width, int height)
 
        UI_view2d_totRect_set_resize(v2d, width, height, 0);
 
-       /* solve bad recursion... if scroller state changed, mask is different, so you get different rects */
+       /* solve bad recursion... if scroller state changed,
+        * mask is different, so you get different rects */
        if (scroll != view2d_scroll_mapped(v2d->scroll)) {
                UI_view2d_totRect_set_resize(v2d, width, height, 0);
        }
@@ -1072,7 +1083,8 @@ static void view2d_map_cur_using_mask(View2D *v2d, rctf *curmasked)
                float sizex = BLI_rcti_size_x(&v2d->mask);
                float sizey = BLI_rcti_size_y(&v2d->mask);
 
-               /* prevent tiny or narrow regions to get invalid coordinates - mask can get negative even... */
+               /* prevent tiny or narrow regions to get
+                * invalid coordinates - mask can get negative even... */
                if (sizex > 0.0f && sizey > 0.0f) {
                        float dx = BLI_rctf_size_x(&v2d->cur) / (sizex + 1);
                        float dy = BLI_rctf_size_y(&v2d->cur) / (sizey + 1);
@@ -1109,7 +1121,8 @@ void UI_view2d_view_ortho(View2D *v2d)
        if (sizey > 0)
                yofs = eps * BLI_rctf_size_y(&v2d->cur) / sizey;
 
-       /* apply mask-based adjustments to cur rect (due to scrollers), to eliminate scaling artifacts */
+       /* apply mask-based adjustments to cur rect (due to scrollers),
+        * to eliminate scaling artifacts */
        view2d_map_cur_using_mask(v2d, &curmasked);
 
        BLI_rctf_translate(&curmasked, -xofs, -yofs);
@@ -1145,7 +1158,8 @@ void UI_view2d_view_orthoSpecial(ARegion *ar, View2D *v2d, const bool xaxis)
        xofs = 0.0f; // (v2d->flag & V2D_PIXELOFS_X) ? GLA_PIXEL_OFS : 0.0f;
        yofs = 0.0f; // (v2d->flag & V2D_PIXELOFS_Y) ? GLA_PIXEL_OFS : 0.0f;
 
-       /* apply mask-based adjustments to cur rect (due to scrollers), to eliminate scaling artifacts */
+       /* apply mask-based adjustments to cur rect (due to scrollers),
+        * to eliminate scaling artifacts */
        view2d_map_cur_using_mask(v2d, &curmasked);
 
        /* only set matrix with 'cur' coordinates on relevant axes */
@@ -1202,7 +1216,9 @@ static void step_to_grid(float *step, int *power, int unit)
                /* for frames, we want 1.0 frame intervals only */
                if (unit == V2D_UNIT_FRAMES) {
                        rem = 1.0f;
-                       *step = 2.0f; /* use 2 since there are grid lines drawn in between, this way to get 1 line per frame */
+                       /* use 2 since there are grid lines drawn in between,
+                        * this way to get 1 line per frame */
+                       *step = 2.0f;
                }
 
                /* prevents printing 1.0 2.0 3.0 etc */
@@ -1806,7 +1822,8 @@ static void scroll_printstr(Scene *scene, float x, float y, float val, int power
                BLI_timecode_string_from_time_seconds(timecode_str, sizeof(timecode_str), power, val);
        }
 
-       /* get length of string, and adjust printing location to fit it into the horizontal scrollbar */
+       /* get length of string,
+        * and adjust printing location&nb