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 13:26:16 +0000 (00:26 +1100)
Prevents clang-format wrapping text before comments.

169 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/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_edit.c
source/blender/editors/armature/armature_relations.c
source/blender/editors/armature/armature_utils.c
source/blender/editors/armature/pose_lib.c
source/blender/editors/armature/pose_slide.c
source/blender/editors/armature/pose_transform.c
source/blender/editors/curve/editcurve.c
source/blender/editors/curve/editcurve_add.c
source/blender/editors/gpencil/gpencil_brush.c
source/blender/editors/gpencil/gpencil_convert.c
source/blender/editors/gpencil/gpencil_edit.c
source/blender/editors/gpencil/gpencil_interpolate.c
source/blender/editors/gpencil/gpencil_paint.c
source/blender/editors/gpencil/gpencil_select.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_search.c
source/blender/editors/interface/interface_style.c
source/blender/editors/interface/interface_templates.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/mask/mask_draw.c
source/blender/editors/mask/mask_ops.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_constraint.c
source/blender/editors/object/object_data_transfer.c
source/blender/editors/object/object_edit.c
source/blender/editors/object/object_relations.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/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_clip/clip_buttons.c
source/blender/editors/space_clip/clip_dopesheet_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_outliner/outliner_draw.c
source/blender/editors/space_outliner/outliner_edit.c
source/blender/editors/space_outliner/outliner_tools.c
source/blender/editors/space_outliner/outliner_tree.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/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_edit.c
source/blender/editors/space_view3d/view3d_fly.c
source/blender/editors/space_view3d/view3d_project.c
source/blender/editors/space_view3d/view3d_select.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_input.c
source/blender/editors/transform/transform_ops.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_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 8c306ab..de268b4 100644 (file)
@@ -3522,7 +3522,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?
@@ -4348,7 +4349,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;
@@ -4536,7 +4538,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 8900e19..e5758c1 100644 (file)
@@ -877,7 +877,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 */
@@ -1556,7 +1557,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);
 }
 
 /* ----------------------------------------------------------- */
@@ -2997,17 +2999,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 a21bc49..7fa049a 100644 (file)
@@ -213,7 +213,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 e7430b5..4dcf7f9 100644 (file)
@@ -879,7 +879,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;
 
@@ -972,7 +973,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,8 +1268,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)
@@ -1289,9 +1294,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) */
@@ -1314,7 +1319,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);
                        }
@@ -1727,7 +1733,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)
@@ -3252,7 +3259,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 ebe085c..e6025fe 100644 (file)
@@ -327,7 +327,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 3136b3e..9e1235d 100644 (file)
@@ -349,9 +349,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 54e0839..ef8b56c 100644 (file)
@@ -873,9 +873,10 @@ void summary_to_keylist(bAnimContext *ac, DLRBT_Tree *keys, DLRBT_Tree *blocks)
                /* 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 c8b0ed3..303afd6 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 */
@@ -672,25 +673,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;
@@ -700,7 +711,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 */
@@ -720,7 +734,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 9ea43c1..9a98af2 100644 (file)
@@ -443,11 +443,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 e4aeef7..b164db6 100644 (file)
@@ -1024,7 +1024,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 1ba64a6..1dec207 100644 (file)
@@ -85,7 +85,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 e4ce1d5..dec67ca 100644 (file)
@@ -141,7 +141,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);
@@ -538,7 +539,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;
                                }
                        }
index 46e1b86..4936136 100644 (file)
@@ -627,7 +627,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;
@@ -660,8 +662,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 1aa852c..9c2bb70 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 */
@@ -728,7 +731,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);
@@ -942,7 +946,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) */
        }
 }
 
@@ -1765,7 +1770,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 2980bf4..bee2d22 100644 (file)
@@ -95,10 +95,13 @@ typedef struct tPoseSlideOp {
        ListBase pfLinks;   /* links between posechannels and f-curves  */
        DLRBT_Tree keys;    /* binary tree for quicker searching for keyframes (when applicable) */
 
-       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;
 
        float prevFrameF;   /* prevFrame, but in local action time (for F-Curve lookups to work) */
        float nextFrameF;   /* nextFrame, but in local action time (for F-Curve lookups to work) */
@@ -106,10 +109,13 @@ typedef struct tPoseSlideOp {
        short mode;         /* sliding mode (ePoseSlide_Modes) */
        short flag;         /* unused for now, but can later get used for storing runtime settings.... */
 
-       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 */
 } tPoseSlideOp;
@@ -310,7 +316,8 @@ static void pose_slide_apply_val(tPoseSlideOp *pso, FCurve *fcu, float *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;
@@ -323,7 +330,8 @@ static void pose_slide_apply_val(tPoseSlideOp *pso, FCurve *fcu, float *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;
@@ -522,7 +530,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 f21d990..27c2ff6 100644 (file)
@@ -104,7 +104,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;
        }
 
@@ -731,7 +731,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 342eef0..f376181 100644 (file)
@@ -156,8 +156,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);
@@ -176,8 +177,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);
@@ -1259,8 +1261,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);
                }
        }
@@ -2137,7 +2140,8 @@ static void adduplicateflagNurb(Object *obedit, 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 {
@@ -2621,7 +2625,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) {
@@ -2691,7 +2696,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) {
index a6100d9..f3b9143 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 a5c6fd8..9435184 100644 (file)
@@ -607,7 +607,8 @@ static bool gp_brush_twist_apply(tGP_BrushEditData *gso, bGPDstroke *gps, int i,
                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 9f94cf3..bea294a 100644 (file)
@@ -1165,7 +1165,8 @@ static void gp_layer_to_curve(bContext *C, ReportList *reports, bGPdata *gpd, bG
                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 312521d..cd228af 100644 (file)
@@ -2156,13 +2156,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 e2d7080..9dc8c1f 100644 (file)
@@ -1073,7 +1073,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 27cde27..12ba034 100644 (file)
@@ -2199,7 +2199,8 @@ static void gpencil_draw_apply_event(wmOperator *op, const wmEvent *event)
        gpencil_draw_apply(op, p);
 
        /* 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);
 }
 
 /* ------------------------------- */
@@ -2771,7 +2772,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_SKIP_SAVE);
 }
index 7bb0c00..784f77d 100644 (file)
@@ -772,7 +772,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 b805b2e..526f42a 100644 (file)
@@ -114,37 +114,51 @@ 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;
 
        void   *owner;          /* for per-element F-Curves (e.g. NLA Control Curves), the element that this represents (e.g. NlaStrip) */
 } 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,
@@ -231,42 +245,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;
 
@@ -409,30 +427,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;
 
 
@@ -522,7 +552,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 9ad716c..5c9618a 100644 (file)
@@ -54,26 +54,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 */
@@ -83,7 +89,8 @@ typedef struct EditBone {
        float ease1, ease2;
        float scaleIn, scaleOut;
 
-       float oldlength;        /* for envelope scaling */
+       /** for envelope scaling */
+       float oldlength;
 
        short segments;
 
index e2c8ecf..82037bd 100644 (file)
@@ -123,8 +123,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 a290571..5f4b8ce 100644 (file)
@@ -55,9 +55,12 @@ 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 */
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 b3496b1..b4d1b5d 100644 (file)
@@ -189,7 +189,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 */
@@ -197,7 +198,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 */
@@ -234,18 +236,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 00558a3..88cb702 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 */
+       /** 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];
 
-       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) */
+       /** 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 d40c4f8..3f96d66 100644 (file)
@@ -158,7 +158,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(struct wmOperatorType *ot);
index 2b1ce46..0e78fca 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;
 
 /* for editing areas/regions */
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 c21a6cc..9dce4a7 100644 (file)
@@ -123,11 +123,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 48c9170..429f967 100644 (file)
@@ -439,7 +439,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);
 
index 94b1720..97520f0 100644 (file)
@@ -150,7 +150,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;
@@ -413,7 +414,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;
@@ -1950,7 +1952,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 {
@@ -2155,7 +2158,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);
 
@@ -2385,7 +2389,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];
 
@@ -3415,7 +3420,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, "");
                                }
                        }
@@ -4441,7 +4447,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;
        }
@@ -4490,7 +4497,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), 0))
                                        break;
index 36b0b93..8e0cfa7 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,
@@ -146,14 +151,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) {
@@ -162,18 +170,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;
@@ -201,10 +212,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;
@@ -217,7 +230,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;
                        }
                }
@@ -313,9 +327,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;
 }
@@ -370,27 +385,29 @@ void ui_block_align_calc(uiBlock *block)
        }
 
        /* 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 10a2379..5204127 100644 (file)
@@ -183,7 +183,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 54e755f..ed6d590 100644 (file)
@@ -52,7 +52,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
 
 /* -------------------------------------------------------------------- */
@@ -130,7 +131,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;
@@ -277,7 +279,8 @@ bool ui_popup_context_menu_for_button(bContext *C, uiBut *but)
                /*bool is_idprop = RNA_property_is_idprop(prop);*/ /* XXX does not work as expected, not strictly needed */
                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);
 
index 13acc63..7e3bc7e 100644 (file)
@@ -630,7 +630,8 @@ void ui_draw_but_WAVEFORM(ARegion *ar, uiBut *but, uiWidgetColors *UNUSED(wcol),
        GLint 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 43ccd65..21b34ad 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 ac60140..bae4a3f 100644 (file)
@@ -232,10 +232,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 */
@@ -244,11 +248,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
@@ -325,8 +330,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
 
@@ -1319,7 +1326,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]))
@@ -1580,18 +1588,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
@@ -3302,7 +3314,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);
 
@@ -3340,7 +3353,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;
@@ -3788,7 +3802,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;
@@ -4304,7 +4319,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 */
@@ -4397,8 +4413,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;
                        click = 1;
@@ -4698,8 +4716,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;
@@ -4714,7 +4734,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;
@@ -4819,7 +4840,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
@@ -6075,7 +6097,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);
@@ -6243,7 +6266,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' */
@@ -7173,7 +7197,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;
@@ -7916,7 +7941,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))
                                {
@@ -7952,7 +7978,8 @@ static int ui_handle_button_event(bContext *C, const wmEvent *event, uiBut *but)
                                retval = WM_UI_HANDLER_CONTINUE;
                                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:
@@ -8182,7 +8209,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;
@@ -8970,7 +8998,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;
@@ -9214,7 +9243,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) {
@@ -9361,7 +9391,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.pie_menu_threshold + U.pie_menu_confirm))
                                                {
@@ -9479,7 +9510,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) {
@@ -9743,7 +9775,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 7ef95ca..871f72f 100644 (file)
@@ -473,7 +473,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);
@@ -1142,7 +1145,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);
 
@@ -1155,10 +1159,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 671bd07..f2d2394 100644 (file)
@@ -173,13 +173,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
@@ -264,7 +271,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;
 
@@ -275,7 +282,7 @@ 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;
@@ -342,7 +349,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;
 
@@ -352,7 +359,8 @@ 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;
 };
 
@@ -404,8 +412,10 @@ struct uiBlock {
        short alignnr;
 
        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[6];
        double auto_open_last;
@@ -413,32 +423,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 color_profile;         /* color profile for correcting linear colors for display */
 
-       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 4324e05..48cecc9 100644 (file)
@@ -1086,7 +1086,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, "");
@@ -1095,7 +1096,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 {
@@ -1812,7 +1814,8 @@ void ui_but_add_search(uiBut *but, PointerRNA *ptr, PropertyRNA *prop, PointerRN
                UI_but_func_search_set(but, ui_searchbox_create_generic, rna_search_cb, but, NULL, NULL);
        }
        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;
        }
 }
@@ -1953,7 +1956,8 @@ static uiBut *ui_item_menu(
                UI_block_emboss_set(block, UI_EMBOSS);
        }
        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);
        }
index 56f95cd..18b6e33 100644 (file)
@@ -461,7 +461,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 8332ab7..b146134 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>
@@ -1619,8 +1620,10 @@ void UI_panel_category_draw_all(ARegion *ar, const char *category_id_active)
        const int px = max_ii(1, round_fl_to_int(U.pixelsize));
        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;
        const int roundboxtype = UI_CNR_TOP_LEFT | UI_CNR_BOTTOM_LEFT;
        bool is_alpha;
@@ -1873,7 +1876,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;
                                        }
                                }
@@ -1940,10 +1944,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 f107995..42d1971 100644 (file)
@@ -155,7 +155,8 @@ static void ui_update_color_picker_buts_rgb(
        if (block->color_profile)
                display = ui_block_cm_display_get(block);
 
-       /* 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;
index ef2d273..0e5634d 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 fac96fc..7a0775d 100644 (file)
@@ -388,7 +388,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 */
index 8d4389c..c7b41eb 100644 (file)
@@ -89,10 +89,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;
 
@@ -261,7 +265,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 5991d99..519c798 100644 (file)
@@ -510,7 +510,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 f9fcf95..636f0e1 100644 (file)
@@ -813,7 +813,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);
@@ -831,14 +833,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);
 }
@@ -874,7 +882,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 *************************/
@@ -912,7 +921,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 */
@@ -1040,7 +1050,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) ||
                     STREQ(scene->r.engine, RE_engine_id_BLENDER_GAME)))
@@ -2205,7 +2216,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, RNAUpdateCb *cb)
@@ -2969,7 +2981,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;
@@ -3595,7 +3608,8 @@ void 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;
 
@@ -4024,7 +4038,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 5aa56dd..0db0484 100644 (file)
@@ -355,7 +355,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 = (
@@ -1032,7 +1033,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]);
@@ -1261,7 +1263,8 @@ static void ui_text_clip_right_label(uiFontStyle *fstyle, uiBut *but, const rcti
                        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;
 
@@ -2082,7 +2085,8 @@ static void widget_state(uiWidgetType *wt, int state)
 static void widget_state_numslider(uiWidgetType *wt, int state)
 {
        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);
@@ -3871,7 +3875,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... */
index 9681d77..c4cd6d2 100644 (file)
@@ -226,7 +226,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 eebe0f6..d1ba88e 100644 (file)
@@ -220,7 +220,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;
@@ -275,7 +276,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 */
@@ -308,7 +310,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;
@@ -343,7 +346,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;
        }
 
@@ -358,7 +362,8 @@ void UI_view2d_region_reinit(View2D *v2d, short type, int winx, int winy)
        view2d_masks(v2d, 0);
 
        /* 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
@@ -467,7 +472,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]);
        }
@@ -557,7 +563,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;
 
@@ -656,7 +663,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;
 
@@ -706,7 +714,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;
 
@@ -784,7 +793,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;
@@ -973,7 +983,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);
        }
@@ -1053,7 +1064,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);
@@ -1090,7 +1102,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);
@@ -1129,7 +1142,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 */
@@ -1189,7 +1203,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 */
@@ -1675,7 +1691,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 to fit it into the horizontal scrollbar */
        len = strlen(timecode_str);
        if (dir == 'h') {
                /* seconds/timecode display has slightly longer strings... */
@@ -1798,7 +1815,8 @@ void UI_view2d_scrollers_draw(const bContext *C, View2D *v2d, View2DScrollers *v
                                                        break;
 
                                                case V2D_UNIT_DEGREES:      /* Graph Editor for rotation Drivers */
-                                                       /* HACK: although we're drawing horizontal, we make this draw as 'vertical', just to get degree signs */
+                                                       /* HACK: although we're drawing horizontal,
+                                                        * we make this draw as 'vertical', just to get degree signs */
                                                        scroll_printstr(scene, fac, h, val, grid->powerx, V2D_UNIT_DEGREES, 'v');
                                                        break;
                                        }
@@ -1867,7 +1885,8 @@ void UI_view2d_scrollers_draw(const bContext *C, View2D *v2d, View2DScrollers *v
                        UI_ThemeColor(TH_TEXT);
                        val = grid->starty;
 
-                       /* if vertical clamping (to whole numbers) is used (i.e. in Sequencer), apply correction */
+                       /* if vertical clamping (to whole numbers) is used (i.e. in Sequencer),
+                        * apply correction */
                        if (vs->yclamp == V2D_GRID_CLAMP)
                                fac += 0.5f * dfac;
 
index e08f953..5e8ad98 100644 (file)
@@ -76,19 +76,28 @@ static bool view2d_poll(bContext *C)
 
 /* temp customdata for operator */
 typedef struct v2dViewPanData {
-       bScreen *sc;            /* screen where view pan was initiated */
-       ScrArea *sa;            /* area where view pan was initiated */
-       ARegion *ar;            /* region where view pan was initiated */
-       View2D *v2d;            /* view2d we're operating in */
+       /** screen where view pan was initiated */
+       bScreen *sc;
+       /** area where view pan was initiated */
+       ScrArea *sa;
+       /** region where view pan was initiated */
+       ARegion *ar;
+       /** view2d we're operating in */
+       View2D *v2d;
 
-       float facx, facy;       /* amount to move view relative to zoom */
+       /** amount to move view relative to zoom */
+       float facx, facy;
 
        /* options for version 1 */
-       int startx, starty;     /* mouse x/y values in window when operator was initiated */
-       int lastx, lasty;       /* previous x/y values of mouse in window */
-       int invoke_event;       /* event starting pan, for modal exit */
-
-       short in_scroller;      /* for MMB in scrollers (old feature in past, but now not that useful) */
+       /** mouse x/y values in window when operator was initiated */
+       int startx, starty;
+       /** previous x/y values of mouse in window */
+       int lastx, lasty;
+       /** event starting pan, for modal exit */
+       int invoke_event;
+
+       /** for MMB in scrollers (old feature in past, but now not that useful) */
+       short in_scroller;
 } v2dViewPanData;
 
 /* initialize panning customdata */
@@ -260,7 +269,8 @@ static int view_pan_invoke(bContext *C, wmOperator *op, const wmEvent *event)
        return OPERATOR_RUNNING_MODAL;
 }
 
-/* handle user input - calculations of mouse-movement need to be done here, not in the apply callback! */
+/* handle user input - calculations of mouse-movement
+ * need to be done here, not in the apply callback! */
 static int view_pan_modal(bContext *C, wmOperator *op, const wmEvent *event)
 {
        v2dViewPanData *vpd = op->customdata;
@@ -672,7 +682,8 @@ static void view_zoomstep_apply_ex(
                        v2d->cur.xmax -= dx;
 
                        if (use_mousepos && (U.uiflag & USER_ZOOM_TO_MOUSEPOS)) {
-                               /* get zoom fac the same way as in ui_view2d_curRect_validate_resize - better keep in sync! */
+                               /* get zoom fac the same way as in
+                                * ui_view2d_curRect_validate_resize - better keep in sync! */
                                const float zoomx = (float)(BLI_rcti_size_x(&v2d->mask) + 1) / BLI_rctf_size_x(&v2d->cur);
 
                                /* only move view to mouse if zoom fac is inside minzoom/maxzoom */
@@ -705,7 +716,8 @@ static void view_zoomstep_apply_ex(
                        v2d->cur.ymax -= dy;
 
                        if (use_mousepos && (U.uiflag & USER_ZOOM_TO_MOUSEPOS)) {
-                               /* get zoom fac the same way as in ui_view2d_curRect_validate_resize - better keep in sync! */
+                               /* get zoom fac the same way as in
+                                * ui_view2d_curRect_validate_resize - better keep in sync! */
                                const float zoomy = (float)(BLI_rcti_size_y(&v2d->mask) + 1) / BLI_rctf_size_y(&v2d->cur);
 
                                /* only move view to mouse if zoom fac is inside minzoom/maxzoom */
@@ -1032,7 +1044,8 @@ static int view_zoomdrag_invoke(bContext *C, wmOperator *op, const wmEvent *even
                        fac = 0.01f * (event->prevy - event->y);
                dy = fac * BLI_rctf_size_y(&v2d->cur) / 10.0f;
 
-               /* support trackpad zoom to always zoom entirely - the v2d code uses portrait or landscape exceptions */
+               /* support trackpad zoom to always zoom entirely - the v2d code uses portrait or
+                * landscape exceptions */
                if (v2d->keepzoom & V2D_KEEPASPECT) {
                        if (fabsf(dx) > fabsf(dy))
                                dy = dx;
@@ -1085,7 +1098,8 @@ static int view_zoomdrag_invoke(bContext *C, wmOperator *op, const wmEvent *even
        return OPERATOR_RUNNING_MODAL;
 }
 
-/* handle user input - calculations of mouse-movement need to be done here, not in the apply callback! */
+/* handle user input - calculations of mouse-movement need to be done here,
+ * not in the apply callback! */
 static int view_zoomdrag_modal(bContext *C, wmOperator *op, const wmEvent *event)
 {
        v2dViewZoomData *vzd = op->customdata;
@@ -1131,7 +1145,8 @@ static int view_zoomdrag_modal(bContext *C, wmOperator *op, const wmEvent *event
 
                }
 
-               /* support zoom to always zoom entirely - the v2d code uses portrait or landscape exceptions */
+               /* support zoom to always zoom entirely - the v2d code uses portrait or
+                * landscape exceptions */
                if (v2d->keepzoom & V2D_KEEPASPECT) {
                        if (fabsf(dx) > fabsf(dy))
                                dy = dx;
@@ -1260,7 +1275,8 @@ static int view_borderzoom_exec(bContext *C, wmOperator *op)
                 */
                float zoom, center, size;
 
-               /* TODO: is this zoom factor calculation valid? It seems to produce same results every time... */
+               /* TODO: is this zoom factor calculation valid?
+                * It seems to produce same results every time... */
                if ((v2d->keepzoom & V2D_LOCKZOOM_X) == 0) {
                        size = BLI_rctf_size_x(&cur_new);
                        zoom = size / BLI_rctf_size_x(&rect);
@@ -1477,7 +1493,8 @@ void UI_view2d_smooth_view(
                        if (v2d->smooth_timer)
                                WM_event_remove_timer(wm, win, v2d->smooth_timer);
                        /* TIMER1 is hardcoded in keymap */
-                       v2d->smooth_timer = WM_event_add_timer(wm, win, TIMER1, 1.0 / 100.0); /* max 30 frs/sec */
+                       /* max 30 frs/sec */
+                       v2d->smooth_timer = WM_event_add_timer(wm, win, TIMER1, 1.0 / 100.0);
 
                        ok = true;
                }
@@ -1573,20 +1590,31 @@ static void VIEW2D_OT_smoothview(wmOperatorType *ot)
 
 /* customdata for scroller-invoke data */
 typedef struct v2dScrollerMove {
-       View2D *v2d;            /* View2D data that this operation affects */
-       ARegion *ar;            /* region that the scroller is in */
-
-       short scroller;         /* scroller that mouse is in ('h' or 'v') */
-       short zone; /* -1 is min zoomer, 0 is bar, 1 is max zoomer */             // XXX find some way to provide visual feedback of this (active color?)
-
-       float fac;              /* view adjustment factor, based on size of region */
-       float fac_round;        /* for pixel rounding (avoid visible UI jitter) */
-       float delta;            /* amount moved by mouse on axis of interest */
-
-       float scrollbarwidth;   /* width of the scrollbar itself, used for page up/down clicks */
-       int scrollbar_orig;      /* initial location of scrollbar x/y, mouse relative */
+       /** View2D data that this operation affects */
+       View2D *v2d;
+       /** region that the scroller is in */
+       ARegion *ar;
 
-       int lastx, lasty;       /* previous mouse coordinates (in screen coordinates) for determining movement */
+       /** scroller that mouse is in ('h' or 'v') */
+       short scroller;
+       /* XXX find some way to provide visual feedback of this (active color?) */
+       /** -1 is min zoomer, 0 is bar, 1 is max zoomer */
+       short zone;
+
+       /** view adjustment factor, based on size of region */
+       float fac;
+       /** for pixel rounding (avoid visible UI jitter) */
+       float fac_round;
+       /** amount moved by mouse on axis of interest */
+       float delta;
+
+       /** width of the scrollbar itself, used for page up/down clicks */
+       float scrollbarwidth;
+       /** initial location of scrollbar x/y, mouse relative */
+       int scrollbar_orig;
+
+       /** previous mouse coordinates (in screen coordinates) for determining movement */
+       int lastx, lasty;
 } v2dScrollerMove;
 
 
@@ -1694,8 +1722,8 @@ static void scroller_activate_init(bContext *C, wmOperator *op, const wmEvent *e
         */
        scrollers = UI_view2d_scrollers_calc(C, v2d, V2D_ARG_DUMMY, V2D_ARG_DUMMY, V2D_ARG_DUMMY, V2D_ARG_DUMMY);
 
-       /* use a union of 'cur' & 'tot' incase the current view is far outside 'tot'.
-        * In this cases moving the scroll bars has far too little effect and the view can get stuck [#31476] */
+       /* use a union of 'cur' & 'tot' incase the current view is far outside 'tot'. In this cases
+        * moving the scroll bars has far too little effect and the view can get stuck T31476. */
        tot_cur_union = v2d->tot;
        BLI_rctf_union(&tot_cur_union, &v2d->cur);
 
@@ -1841,10 +1869,12 @@ static int scroller_activate_modal(bContext *C, wmOperator *op, const wmEvent *e
                        if (ELEM(vsm->zone, SCROLLHANDLE_BAR, SCROLLHANDLE_MAX)) {
                                /* if using bar (i.e. 'panning') or 'max' zoom widget */
                                switch (vsm->scroller) {
-                                       case 'h': /* horizontal scroller - so only horizontal movement ('cur' moves opposite to mouse) */
+                                       case 'h': /* horizontal scroller - so only horizontal movement
+                                                  * ('cur' moves opposite to mouse) */
                                                vsm->delta = (float)(event->x - vsm->lastx);
                                                break;
-                                       case 'v': /* vertical scroller - so only vertical movement ('cur' moves opposite to mouse) */
+                                       case 'v': /* vertical scroller - so only vertical movement
+                                                  * ('cur' moves opposite to mouse) */
                                                vsm->delta = (float)(event->y - vsm->lasty);
                                                break;
                                }
@@ -1852,10 +1882,12 @@ static int scroller_activate_modal(bContext *C, wmOperator *op, const wmEvent *e
                        else if (vsm->zone == SCROLLHANDLE_MIN) {
                                /* using 'min' zoom widget */
                                switch (vsm->scroller) {
-                                       case 'h': /* horizontal scroller - so only horizontal movement ('cur' moves with mouse) */
+                                       case 'h': /* horizontal scroller - so only horizontal movement
+                                                  * ('cur' moves with mouse) */
                                                vsm->delta = (float)(vsm->lastx - event->x);
                                                break;
-                                       case 'v': /* vertical scroller - so only vertical movement ('cur' moves with to mouse) */
+                                       case 'v': /* vertical scroller - so only vertical movement
+                                                  * ('cur' moves with to mouse) */
                                                vsm->delta = (float)(vsm->lasty - event->y);
                                                break;
                                }
@@ -1897,7 +1929,8 @@ static int scroller_activate_modal(bContext *C, wmOperator *op, const wmEvent *e
 }
 
 
-/* a click (or click drag in progress) should have occurred, so check if it happened in scrollbar */
+/* a click (or click drag in progress)
+ * should have occurred, so check if it happened in scrollbar */
 static int scroller_activate_invoke(bContext *C, wmOperator *op, const wmEvent *event)
 {
        ARegion *ar = CTX_wm_region(C);
@@ -1907,7 +1940,8 @@ static int scroller_activate_invoke(bContext *C, wmOperator *op, const wmEvent *
        /* check if mouse in scrollbars, if they're enabled */
        in_scroller = UI_view2d_mouse_in_scrollers(C, v2d, event->x, event->y);
 
-       /* if in a scroller, init customdata then set modal handler which will catch mousedown to start doing useful stuff */
+       /* if in a scroller, init customdata then set modal handler which will
+        * catch mousedown to start doing useful stuff */
        if (in_scroller) {
                v2dScrollerMove *vsm;
 
@@ -1918,10 +1952,12 @@ static int scroller_activate_invoke(bContext *C, wmOperator *op, const wmEvent *
                /* support for quick jump to location - gtk and qt do this on linux */
                if (event->type == MIDDLEMOUSE) {
                        switch (vsm->scroller) {
-                               case 'h': /* horizontal scroller - so only horizontal movement ('cur' moves opposite to mouse) */
+                               case 'h': /* horizontal scroller - so only horizontal movement
+                                          * ('cur' moves opposite to mouse) */
                                        vsm->delta = (float)(event->x - vsm->scrollbar_orig);
                                        break;
-                               case 'v': /* vertical scroller - so only vertical movement ('cur' moves opposite to mouse) */
+                               case 'v': /* vertical scroller - so only vertical movement
+                                          * ('cur' moves opposite to mouse) */
                                        vsm->delta = (float)(event->y - vsm->scrollbar_orig);
                                        break;
                        }
@@ -1978,7 +2014,8 @@ static int scroller_activate_invoke(bContext *C, wmOperator *op, const wmEvent *
                return OPERATOR_RUNNING_MODAL;
        }
        else {
-               /* not in scroller, so nothing happened... (pass through let's something else catch event) */
+               /* not in scroller, so nothing happened...
+                * (pass through let's something else catch event) */
                return OPERATOR_PASS_THROUGH;
        }
 }
index 407c19a..4d15dbb 100644 (file)
@@ -677,14 +677,17 @@ static float *mask_rasterize(Mask *mask, const int width, const int height)
 
 /* sets up the opengl context.
  * width, height are to match the values from ED_mask_get_size() */
-void ED_mask_draw_region(Mask *mask, ARegion *ar,
-                         const char draw_flag, const char draw_type, const char overlay_mode,
-                         const int width_i, const int height_i,  /* convert directly into aspect corrected vars */
-                         const float aspx, const float aspy,
-                         const bool do_scale_applied, const bool do_draw_cb,
-                         float stabmat[4][4], /* optional - only used by clip */
-                         const bContext *C    /* optional - only used when do_post_draw is set or called from clip editor */
-                         )
+void ED_mask_draw_region(
+        Mask *mask, ARegion *ar,
+        const char draw_flag, const char draw_type, const char overlay_mode,
+        /* convert directly into aspect corrected vars */
+        const int width_i, const int height_i,
+        const float aspx, const float aspy,
+        const bool do_scale_applied, const bool do_draw_cb,
+        /* optional - only used by clip */
+        float stabmat[4][4],
+        /* optional - only used when do_post_draw is set or called from clip editor */
+        const bContext *C)
 {
        struct View2D *v2d = &ar->v2d;
 
index cf67a97..38b39cb 100644 (file)
@@ -2215,7 +2215,8 @@ static int mask_duplicate_exec(bContext *C, wmOperator *UNUSED(op))
                                        MaskSplinePoint *new_point;
                                        int b;
 
-                                       /* BKE_mask_spline_add might allocate the points, need to free them in this case. */
+                                       /* BKE_mask_spline_add might allocate the points,
+                                        * need to free them in this case. */
                                        if (new_spline->points) {
                                                MEM_freeN(new_spline->points);
                                        }
index e49c013..a752b5d 100644 (file)
@@ -80,7 +80,8 @@ typedef struct {
        float initial_length[NUM_VALUE_KINDS];
        float scale[NUM_VALUE_KINDS];
        NumInput num_input[NUM_VALUE_KINDS];
-       float shift_value[NUM_VALUE_KINDS]; /* The current value when shift is pressed. Negative when shift not active. */
+       /** The current value when shift is pressed. Negative when shift not active. */
+       float shift_value[NUM_VALUE_KINDS];
        bool is_modal;
 
        /* modal only */
@@ -166,7 +167,8 @@ static bool edbm_bevel_init(bContext *C, wmOperator *op, const bool is_modal)
                if (i == OFFSET_VALUE) {
                        opdata->num_input[i].unit_sys = scene->unit.system;
                }
-               opdata->num_input[i].unit_type[0] = B_UNIT_NONE;  /* Not sure this is a factor or a unit? */
+               /* Not sure this is a factor or a unit? */
+               opdata->num_input[i].unit_type[0] = B_UNIT_NONE;
        }
 
        /* avoid the cost of allocating a bm copy */
index 54799e3..339c928 100644 (file)
@@ -213,7 +213,8 @@ static bool edbm_inset_calc(wmOperator *op)
        const float thickness          = RNA_float_get(op->ptr,   "thickness");
        const float depth              = RNA_float_get(op->ptr,   "depth");
        const bool use_outset          = RNA_boolean_get(op->ptr, "use_outset");
-       const bool use_select_inset    = RNA_boolean_get(op->ptr, "use_select_inset"); /* not passed onto the BMO */
+       /* not passed onto the BMO */
+       const bool use_select_inset    = RNA_boolean_get(op->ptr, "use_select_inset");
        const bool use_individual      = RNA_boolean_get(op->ptr, "use_individual");
        const bool use_interpolate     = RNA_boolean_get(op->ptr, "use_interpolate");
 
index 86b1a58..11c4f12 100644 (file)
@@ -610,8 +610,9 @@ static BMEdge *bm_face_split_edge_find(
                                        ok = false;
                                }
                                else if (found_other_face) {
-                                       /* double check that _all_ the faces used by v_pivot's edges are attached to this edge
-                                        * otherwise don't attempt the split since it will give non-deterministic results */
+                                       /* double check that _all_ the faces used by v_pivot's edges are attached
+                                        * to this edge otherwise don't attempt the split since it will give
+                                        * non-deterministic results */
                                        BMLoop *l_radial_iter = l_iter->radial_next;
                                        int other_face_shared = 0;
                                        if (l_radial_iter != l_iter) {
index 0b3c4db..2e4ee91 100644 (file)
@@ -2071,7 +2071,8 @@ static KnifeVert *knife_find_closest_vert(KnifeTool_OpData *kcd, float p[3], flo
 
                                knife_project_v2(kcd, kfv->cageco, kfv->sco);
 
-                               /* be strict about angle snapping, the vertex needs to be very close to the angle, or we ignore */
+                               /* be strict about angle snapping, the vertex needs to be very close to the angle,
+                                * or we ignore */
                                if (kcd->is_angle_snapping) {
                                        if (dist_squared_to_line_segment_v2(kfv->sco, kcd->prev.mval, kcd->curr.mval) > KNIFE_FLT_EPSBIG) {
                                                continue;
@@ -3116,7 +3117,8 @@ void EDBM_mesh_knife(bContext *C, LinkNode *polys, bool use_tag, bool cut_throug
                                keep_search = false;
                                BM_ITER_MESH (f, &iter, bm, BM_FACES_OF_MESH) {
                                        if (BM_elem_flag_test(f, BM_ELEM_TAG) == false && (F_ISECT_IS_UNKNOWN(f))) {
-                                               /* am I connected to a tagged face via an un-tagged edge (ie, not across a cut) */
+                                               /* am I connected to a tagged face via an un-tagged edge
+                                                * (ie, not across a cut) */
                                                BMLoop *l_first = BM_FACE_FIRST_LOOP(f);
                                                BMLoop *l_iter = l_first;
                                                bool found = false;
index 34951a2..8124ca3 100644 (file)
@@ -461,10 +461,13 @@ static void ringsel_finish(bContext *C, wmOperator *op)
                        /* XXX Is this piece of code ever used now? Simple loop select is now
                         *     in editmesh_select.c (around line 1000)... */
                        /* sets as active, useful for other tools */
-                       if (em->selectmode & SCE_SELECT_VERTEX)
-                               BM_select_history_store(em->bm, lcd->eed->v1);  /* low priority TODO, get vertrex close to mouse */
-                       if (em->selectmode & SCE_SELECT_EDGE)
+                       if (em->selectmode & SCE_SELECT_VERTEX) {
+                               /* low priority TODO, get vertrex close to mouse */
+                               BM_select_history_store(em->bm, lcd->eed->v1);
+                       }
+                       if (em->selectmode & SCE_SELECT_EDGE) {
                                BM_select_history_store(em->bm, lcd->eed);
+                       }
 
                        EDBM_selectmode_flush(lcd->em);
                        WM_event_add_notifier(C, NC_GEOM | ND_SELECT, lcd->ob->data);
@@ -848,7 +851,8 @@ void MESH_OT_loopcut(wmOperatorType *ot)
 
        /* properties */
        prop = RNA_def_int(ot->srna, "number_cuts", 1, 1, 1000000, "Number of Cuts", "", 1, 100);
-       /* avoid re-using last var because it can cause _very_ high poly meshes and annoy users (or worse crash) */
+       /* avoid re-using last var because it can cause
+        * _very_ high poly meshes and annoy users (or worse crash) */
        RNA_def_property_flag(prop, PROP_SKIP_SAVE);
 
        prop = RNA_def_float(ot->srna, "smoothness", 0.0f, -1e3f, 1e3f,
index eccc15b..1e55630 100644 (file)
@@ -66,7 +66,8 @@
  * \{ */
 
 struct PathSelectParams {
-       bool track_active;  /* ensure the active element is the last selected item (handy for picking) */
+       /** ensure the active element is the last selected item (handy for picking) */
+       bool track_active;
        bool use_topology_distance;
        bool use_face_step;
        bool use_fill;
index f23f0ae..9570578 100644 (file)
@@ -312,7 +312,8 @@ static EdgeLoopPair *edbm_ripsel_looptag_helper(BMesh *bm)
                uid = uid_end + bm->totedge;
 
                lp = BLI_array_append_ret(eloop_pairs);
-               BM_edge_loop_pair(e_last, &lp->l_a, &lp->l_b); /* no need to check, we know this will be true */
+               /* no need to check, we know this will be true */
+               BM_edge_loop_pair(e_last, &lp->l_a, &lp->l_b);
 
 
                BLI_assert(tot == uid_end - uid_start);
@@ -798,7 +799,8 @@ static int edbm_rip_invoke__vert(bContext *C, wmOperator *op, const wmEvent *eve
 
                                        if (do_fill) {
                                                /* Only needed when filling...
-                                                * Also, we never want to tag best edge, that one won't change during split. See T44618. */
+                                                * Also, we never want to tag best edge,
+                                                * that one won't change during split. See T44618. */
                                                if (larr[larr_len]->e == e_best) {
                                                        BM_elem_flag_enable(larr[larr_len]->prev->e, BM_ELEM_TAG);
                                                }
index 1edfcae..a1e9812 100644 (file)
@@ -197,7 +197,8 @@ void EDBM_automerge(Scene *scene, Object *obedit, bool update, const char hflag)
 /** \name Back-Buffer OpenGL Selection
  * \{ */
 
-unsigned int bm_solidoffs = 0, bm_wireoffs = 0, bm_vertoffs = 0;    /* set in drawobject.c ... for colorindices */
+/* set in drawobject.c ... for colorindices */
+unsigned int bm_solidoffs = 0, bm_wireoffs = 0, bm_vertoffs = 0;
 
 /* facilities for border select and circle select */
 static BLI_bitmap *selbuf = NULL;
@@ -2044,7 +2045,8 @@ void EDBM_selectmode_set(BMEditMesh *em)
 
        em->bm->selectmode = em->selectmode;
 
-       edbm_strip_selections(em); /* strip BMEditSelections from em->selected that are not relevant to new mode */
+       /* strip BMEditSelections from em->selected that are not relevant to new mode */
+       edbm_strip_selections(em);
 
        if (em->bm->totvertsel == 0 &&
            em->bm->totedgesel == 0 &&
index c0f73d1..7a7452f 100644 (file)
@@ -144,7 +144,8 @@ void MESH_OT_subdivide(wmOperatorType *ot)
 
        /* properties */
        prop = RNA_def_int(ot->srna, "number_cuts", 1, 1, 100, "Number of Cuts", "", 1, 10);
-       /* avoid re-using last var because it can cause _very_ high poly meshes and annoy users (or worse crash) */
+       /* avoid re-using last var because it can cause
+        * _very_ high poly meshes and annoy users (or worse crash) */
        RNA_def_property_flag(prop, PROP_SKIP_SAVE);
 
        RNA_def_float(ot->srna, "smoothness", 0.0f, 0.0f, 1e3f, "Smoothness", "Smoothness factor", 0.0f, 1.0f);
@@ -2946,8 +2947,10 @@ static float bm_edge_seg_isect(
                y12 = mouse_path[i][1];
 
                /* Perp. Distance from point to line */
-               if (m2 != MAXSLOPE) dist = (y12 - m2 * x12 - b2);  /* /sqrt(m2 * m2 + 1); Only looking for */
-               /* change in sign.  Skip extra math */
+               if (m2 != MAXSLOPE) {
+                       /* /sqrt(m2 * m2 + 1); Only looking for change in sign.  Skip extra math .*/
+                       dist = (y12 - m2 * x12 - b2);
+               }
                else dist = x22 - x12;
 
                if (i == 0) lastdist = dist;
@@ -3074,7 +3077,8 @@ static int edbm_knife_cut_exec(bContext *C, wmOperator *op)
 
        /* TODO, investigate using index lookup for screen_vert_coords() rather then a hash table */
 
-       /* the floating point coordinates of verts in screen space will be stored in a hash table according to the vertices pointer */
+       /* the floating point coordinates of verts in screen space will be
+        * stored in a hash table according to the vertices pointer */
        screen_vert_coords = sco = MEM_mallocN(bm->totvert * sizeof(float) * 2, __func__);
 
        BM_ITER_MESH_INDEX (bv, &iter, bm, BM_VERTS_OF_MESH, i) {
@@ -4933,7 +4937,8 @@ static void sort_bmelem_flag(
                float fact = reverse ? -1.0 : 1.0;
                int coidx = (action == SRT_VIEW_ZAXIS) ? 2 : 0;
 
-               mul_m4_m4m4(mat, rv3d->viewmat, ob->obmat);  /* Apply the view matrix to the object matrix. */
+               /* Apply the view matrix to the object matrix. */
+               mul_m4_m4m4(mat, rv3d->viewmat, ob->obmat);
 
                if (totelem[0]) {
                        pb = pblock[0] = MEM_callocN(sizeof(char) * totelem[0], "sort_bmelem vert pblock");
@@ -5074,9 +5079,11 @@ static void sort_bmelem_flag(
                                float srt = reverse ? (float)(MAXMAT - fa->mat_nr) : (float)fa->mat_nr;
                                pb[i] = false;
                                sb[affected[2]].org_idx = i;
-                               /* Multiplying with totface and adding i ensures us we keep current order for all faces of same mat. */
+                               /* Multiplying with totface and adding i ensures us
+                                * we keep current order for all faces of same mat. */
                                sb[affected[2]++].srt = srt * ((float)totelem[2]) + ((float)i);
-/*                             printf("e: %d; srt: %f; final: %f\n", i, srt, srt * ((float)totface) + ((float)i));*/
+                               // printf("e: %d; srt: %f; final: %f\n",
+                               //        i, srt, srt * ((float)totface) + ((float)i));
                        }
                        else {
                                pb[i] = true;
index 60aa410..15e3886 100644 (file)
@@ -801,7 +801,8 @@ UvElementMap *BM_uv_element_map_create(
                island_number = MEM_mallocN(sizeof(*island_number) * totfaces, "uv_island_number_face");
                copy_vn_i(island_number, totfaces, INVALID_ISLAND);
 
-               /* at this point, every UvElement in vert points to a UvElement sharing the same vertex. Now we should sort uv's in islands. */
+               /* at this point, every UvElement in vert points to a UvElement sharing the same vertex.
+                * Now we should sort uv's in islands. */
                for (i = 0; i < totuv; i++) {
                        if (element_map->buf[i].island == INVALID_ISLAND) {
                                element_map->buf[i].island = nislands;
@@ -820,7 +821,8 @@ UvElementMap *BM_uv_element_map_create(
                                                                initelement = element;
 
                                                        if (element->l->f == efa) {
-                                                               /* found the uv corresponding to our face and vertex. Now fill it to the buffer */
+                                                               /* found the uv corresponding to our face and vertex.
+                                                                * Now fill it to the buffer */
                                                                element->island = nislands;
                                                                map[element - element_map->buf] = islandbufsize;
                                                                islandbuf[islandbufsize].l = element->l;
@@ -1230,7 +1232,8 @@ void EDBM_mesh_hide(BMEditMesh *em, bool swap)
        EDBM_selectmode_flush(em);
 
        /* original hide flushing comment (OUTDATED):
-        * hide happens on least dominant select mode, and flushes up, not down! (helps preventing errors in subsurf) */
+        * hide happens on least dominant select mode, and flushes up, not down!
+        * (helps preventing errors in subsurf) */
        /* - vertex hidden, always means edge is hidden too
         * - edge hidden, always means face is hidden too
         * - face hidden, only set face hide
index 1615ec7..68bee0c 100644 (file)
@@ -313,7 +313,8 @@ void ED_mesh_mirrtopo_init(Mesh *me, DerivedMesh *dm, const int ob_mode, MirrTop
        if (em) {
                BMVert **vtable = em->bm->vtable;
                for (a = 1; a <= totvert; a++) {
-                       /* printf("I %d %ld %d\n", (a - last), MirrTopoPairs[a].hash, MirrTopoPairs[a].v_indexs); */
+                       // printf("I %d %ld %d\n",
+                       //        (a - last), MirrTopoPairs[a].hash, MirrTopoPairs[a].v_indexs);
                        if ((a == totvert) || (topo_pairs[a - 1].hash != topo_pairs[a].hash)) {
                                const int match_count = a - last;
                                if (match_count == 2) {
index 311f13b..c25b41e 100644 (file)
@@ -144,7 +144,8 @@ static void join_mesh_single(
                                        /* check if this mesh has such a shapekey */
                                        KeyBlock *okb = me->key ? BKE_keyblock_find_name(me->key, kb->name) : NULL;
                                        if (okb) {
-                                               /* copy this mesh's shapekey to the destination shapekey (need to transform first) */
+                                               /* copy this mesh's shapekey to the destination shapekey
+                                                * (need to transform first) */
                                                float (*ocos)[3] = okb->data;
                                                for (a = 0; a < me->totvert; a++, cos++, ocos++) {
                                                        copy_v3_v3(*cos, *ocos);
@@ -391,7 +392,8 @@ int join_mesh_exec(bContext *C, wmOperator *op)
 
 
                        if (me->totvert) {
-                               /* Add this object's materials to the base one's if they don't exist already (but only if limits not exceeded yet) */
+                               /* Add this object's materials to the base one's if they don't exist already
+                                * (but only if limits not exceeded yet) */
                                if (totcol < MAXMAT) {
                                        for (a = 1; a <= base->object->totcol; a++) {
                                                ma = give_current_material(base->object, a);
@@ -414,7 +416,8 @@ int join_mesh_exec(bContext *C, wmOperator *op)
                                        }
                                }
 
-                               /* if this mesh has shapekeys, check if destination mesh already has matching entries too */
+                               /* if this mesh has shapekeys,
+                                * check if destination mesh already has matching entries too */
                                if (me->key && key) {
                                        /* for remapping KeyBlock.relative */
                                        int      *index_map = MEM_mallocN(sizeof(int)        * me->key->totkey, __func__);
@@ -445,7 +448,8 @@ int join_mesh_exec(bContext *C, wmOperator *op)
 
                                        /* remap relative index values */
                                        for (kb = me->key->block.first, i = 0; kb; kb = kb->next, i++) {
-                                               if (LIKELY(kb->relative < me->key->totkey)) {  /* sanity check, should always be true */
+                                               /* sanity check, should always be true */
+                                               if (LIKELY(kb->relative < me->key->totkey)) {
                                                        kb_map[i]->relative = index_map[kb->relative];
                                                }
                                        }
@@ -479,8 +483,10 @@ int join_mesh_exec(bContext *C, wmOperator *op)
        /* inverse transform for all selected meshes in this object */
        invert_m4_m4(imat, ob->obmat);
 
-       /* Add back active mesh first. This allows to keep things similar as they were, as much as possible (i.e. data from
-        * active mesh will remain first ones in new result of the merge, in same order for CD layers, etc. See also T50084.
+       /* Add back active mesh first.
+        * This allows to keep things similar as they were, as much as possible
+        * (i.e. data from active mesh will remain first ones in new result of the merge,
+        * in same order for CD layers, etc). See also T50084.
         */
        join_mesh_single(
                    bmain, scene,
index 1bba15c..c15fa68 100644 (file)
@@ -1404,8 +1404,9 @@ static void make_object_duplilist_real(bContext *C, Scene *scene, Base *base,
                BLI_ghash_insert(dupli_gh, dob, ob_dst);
                if (parent_gh) {
                        void **val;
-                       /* Due to nature of hash/comparison of this ghash, a lot of duplis may be considered as 'the same',
-                        * this avoids trying to insert same key several time and raise asserts in debug builds... */
+                       /* Due to nature of hash/comparison of this ghash, a lot of duplis may be considered as
+                        * 'the same', this avoids trying to insert same key several time and
+                        * raise asserts in debug builds... */
                        if (!BLI_ghash_ensure_p(parent_gh, dob, &val)) {
                                *val = ob_dst;
                        }
index 70d3d85..5419eaf 100644 (file)
@@ -103,7 +103,8 @@ ListBase *get_active_constraints(Object *ob)
        return NULL;
 }
 
-/* Find the list that a given constraint belongs to, and/or also get the posechannel this is from (if applicable) */
+/* Find the list that a given constraint belongs to,
+ * and/or also get the posechannel this is from (if applicable) */
 ListBase *get_constraint_lb(Object *ob, bConstraint *con, bPoseChannel **r_pchan)
 {
        if (r_pchan)
@@ -1270,7 +1271,8 @@ static int constraint_delete_exec(bContext *C, wmOperator *UNUSED(op))
        if (BKE_constraint_remove_ex(lb, ob, con, true)) {
                /* there's no active constraint now, so make sure this is the case */
                BKE_constraints_active_set(&ob->constraints, NULL);
-               ED_object_constraint_update(bmain, ob); /* needed to set the flags on posebones correctly */
+               /* needed to set the flags on posebones correctly */
+               ED_object_constraint_update(bmain, ob);
 
                /* relatiols */
                DAG_relations_tag_update(CTX_data_main(C));
@@ -1599,7 +1601,8 @@ static bool get_new_constraint_target(bContext *C, int con_type, Object **tar_ob
                        return false;
 
                /* restricted target-type constraints -------------- */
-               /* NOTE: for these, we cannot try to add a target object if no valid ones are found, since that doesn't work */
+               /* NOTE: for these, we cannot try to add a target object if no valid ones are found,
+                * since that doesn't work */
                /* curve-based constraints - set the only_curve and only_ob flags */
                case CONSTRAINT_TYPE_CLAMPTO:
                case CONSTRAINT_TYPE_FOLLOWPATH:
index 9108134..64a0ae5 100644 (file)
@@ -69,7 +69,8 @@ static const EnumPropertyItem DT_layer_items[] = {
 #if 0  /* XXX For now, would like to finish/merge work from 2014 gsoc first. */
        {DT_TYPE_SHAPEKEY, "SHAPEKEYS", 0, "Shapekey(s)", "Transfer active or all shape keys"},
 #endif
-#if 0  /* XXX When SkinModifier is enabled, it seems to erase its own CD_MVERT_SKIN layer from final DM :( */
+#if 0  /* XXX When SkinModifier is enabled,
+        * it seems to erase its own CD_MVERT_SKIN layer from final DM :( */
        {DT_TYPE_SKIN, "SKIN", 0, "Skin Weight", "Transfer skin weights"},
 #endif
        {DT_TYPE_BWEIGHT_VERT, "BEVEL_WEIGHT_VERT", 0, "Bevel Weight", "Transfer bevel weights"},
index b4e17ce..173621b 100644 (file)
@@ -473,8 +473,10 @@ bool ED_object_editmode_exit_ex(Main *bmain, Scene *scene, Object *obedit, int f
                /* flag object caches as outdated */
                BKE_ptcache_ids_from_object(bmain, &pidlist, obedit, scene, 0);
                for (pid = pidlist.first; pid; pid = pid->next) {
-                       if (pid->type != PTCACHE_TYPE_PARTICLES) /* particles don't need reset on geometry change */
+                       /* particles don't need reset on geometry change */
+                       if (pid->type != PTCACHE_TYPE_PARTICLES) {
                                pid->cache->flag |= PTCACHE_OUTDATED;
+                       }
                }
                BLI_freelistN(&pidlist);
 
index d27edb5..29f8554 100644 (file)
@@ -499,7 +499,8 @@ void ED_object_parent_clear(Object *ob, const int type)
        switch (type) {
                case CLEAR_PARENT_ALL:
                {
-                       /* for deformers, remove corresponding modifiers to prevent a large number of modifiers building up */
+                       /* for deformers, remove corresponding modifiers to prevent
+                        * a large number of modifiers building up */
                        object_remove_parent_deform_modifiers(ob, ob->parent);
 
                        /* clear parenting relationship completely */
@@ -508,14 +509,16 @@ void ED_object_parent_clear(Object *ob, const int type)
                }
                case CLEAR_PARENT_KEEP_TRANSFORM:
                {
-                       /* remove parent, and apply the parented transform result as object's local transforms */
+                       /* remove parent, and apply the parented transform
+                        * result as object's local transforms */
                        ob->parent = NULL;
                        BKE_object_apply_mat4(ob, ob->obmat, true, false);
                        break;
                }
                case CLEAR_PARENT_INVERSE:
                {
-                       /* object stays parented, but the parent inverse (i.e. offset from parent to retain binding state)
+                       /* object stays parented, but the parent inverse
+                        * (i.e. offset from parent to retain binding state)
                         * is cleared. In other words: nothing to do here! */
                        break;
                }
@@ -1567,7 +1570,8 @@ static int make_links_data_exec(bContext *C, wmOperator *op)
                                                /* new approach, using functions from kernel */
                                                for (a = 0; a < ob_src->totcol; a++) {
                                                        Material *ma = give_current_material(ob_src, a + 1);
-                                                       assign_material(bmain, ob_dst, ma, a + 1, BKE_MAT_ASSIGN_USERPREF); /* also works with ma==NULL */
+                                                       /* also works with `ma == NULL` */
+                                                       assign_material(bmain, ob_dst, ma, a + 1, BKE_MAT_ASSIGN_USERPREF);
                                                }
                                                DAG_id_tag_update(&ob_dst->id, OB_RECALC_DATA);
                                                break;
@@ -1960,7 +1964,8 @@ static void single_mat_users(Main *bmain, Scene *scene, const int flag, const bo
                        for (a = 1; a <= ob->totcol; a++) {
                                ma = give_current_material(ob, a);
                                if (ma) {
-                                       /* do not test for LIB_TAG_NEW or use newid: this functions guaranteed delivers single_users! */
+                                       /* do not test for LIB_TAG_NEW or use newid:
+                                        * this functions guaranteed delivers single_users! */
 
                                        if (ma->id.us > 1) {
                                                man = BKE_material_copy(bmain, ma);
index e11782c..ce3ba15 100644 (file)
@@ -957,7 +957,9 @@ float ED_vgroup_vert_weight(Object *ob, bDeformGroup *dg, int vertnum)
 }
 
 void ED_vgroup_select_by_name(Object *ob, const char *name)
-{   /* note: ob->actdef==0 signals on painting to create a new one, if a bone in posemode is selected */
+{
+       /* note: ob->actdef==0 signals on painting to create a new one,
+     * if a bone in posemode is selected */
        ob->actdef = defgroup_name_index(ob, name) + 1;
 }
 
@@ -1960,20 +1962,22 @@ static int inv_cmp_mdef_vert_weights(const void *a1, const void *a2)
 {
        /* qsort sorts in ascending order.  We want descending order to save a memcopy
         * so this compare function is inverted from the standard greater than comparison qsort needs.
-        * A normal compare function is called with two pointer arguments and should return an integer less than, equal to,
-        * or greater than zero corresponding to whether its first argument is considered less than, equal to,
-        * or greater than its second argument.  This does the opposite. */
+        * A normal compare function is called with two pointer arguments and should return an integer
+        * less than, equal to, or greater than zero corresponding to whether its first argument is
+        * considered less than, equal to, or greater than its second argument.
+        * This does the opposite. */
        const struct MDeformWeight *dw1 = a1, *dw2 = a2;
 
        if      (dw1->weight < dw2->weight) return  1;
        else if (dw1->weight > dw2->weight) return -1;
-       else if (&dw1 < &dw2)               return  1; /* compare addresses so we have a stable sort algorithm */
+       else if (&dw1 < &dw2)               return  1; /* compare address for stable sort algorithm */
        else                                return -1;
 }
 
 /* Used for limiting the number of influencing bones per vertex when exporting
  * skinned meshes.  if all_deform_weights is True, limit all deform modifiers
- * to max_weights regardless of type, otherwise, only limit the number of influencing bones per vertex*/
+ * to max_weights regardless of type, otherwise,
+ * only limit the number of influencing bones per vertex. */
 static int vgroup_limit_total_subset(
         Object *ob,
         const bool *vgroup_validmap,
index 181c42a..edd3977 100644 (file)
@@ -3149,7 +3149,10 @@ static void brush_puff(PEData *data, int point_index)
                        /* find root coordinate and normal on emitter */
                        copy_v3_v3(co, key->co);
                        mul_m4_v3(mat, co);
-                       mul_v3_m4v3(kco, data->ob->imat, co); /* use 'kco' as the object space version of worldspace 'co', ob->imat is set before calling */
+
+                       /* use 'kco' as the object space version of worldspace 'co',
+                        * ob->imat is set before calling */
+                       mul_v3_m4v3(kco, data->ob->imat, co);
 
                        point_index = BLI_kdtree_find_nearest(edit->emitter_field, kco, NULL);
                        if (point_index == -1) return;
@@ -3233,7 +3236,10 @@ static void brush_puff(PEData *data, int point_index)
                                                float oco[3], onor[3];
                                                copy_v3_v3(oco, key->co);
                                                mul_m4_v3(mat, oco);
-                                               mul_v3_m4v3(kco, data->ob->imat, oco); /* use 'kco' as the object space version of worldspace 'co', ob->imat is set before calling */
+
+                                               /* use 'kco' as the object space version of worldspace 'co',
+                                                * ob->imat is set before calling */
+                                               mul_v3_m4v3(kco, data->ob->imat, oco);
 
                                                point_index = BLI_kdtree_find_nearest(edit->emitter_field, kco, NULL);
                                                if (point_index != -1) {
index e6b8ac5..1e66b05 100644 (file)
@@ -629,7 +629,8 @@ void PARTICLE_OT_disconnect_hair(wmOperatorType *ot)
        ot->exec = disconnect_hair_exec;
 
        /* flags */
-       ot->flag = OPTYPE_UNDO;  /* No REGISTER, redo does not work due to missing update, see T47750. */
+       /* No REGISTER, redo does not work due to missing update, see T47750. */
+       ot->flag = OPTYPE_UNDO;
 
        RNA_def_boolean(ot->srna, "all", 0, "All hair", "Disconnect all hair systems from the emitter mesh");
 }
@@ -885,7 +886,8 @@ void PARTICLE_OT_connect_hair(wmOperatorType *ot)
        ot->exec = connect_hair_exec;
 
        /* flags */
-       ot->flag = OPTYPE_UNDO;  /* No REGISTER, redo does not work due to missing update, see T47750. */
+       /* No REGISTER, redo does not work due to missing update, see T47750. */
+       ot->flag = OPTYPE_UNDO;
 
        RNA_def_boolean(ot->srna, "all", 0, "All hair", "Connect all hair systems to the emitter mesh");
 }
index d94776d..ca81b54 100644 (file)
@@ -146,7 +146,8 @@ static void image_buffer_rect_update(RenderJob *rj, RenderResult *rr, ImBuf *ibu
 
        /* if renrect argument, we only refresh scanlines */
        if (renrect) {
-               /* if (ymax == recty), rendering of layer is ready, we should not draw, other things happen... */
+               /* if (ymax == recty), rendering of layer is ready,
+                * we should not draw, other things happen... */
                if (rr->renlay == NULL || renrect->ymax >= rr->recty)
                        return;
 
@@ -649,8 +650,9 @@ static void render_endjob(void *rjv)
 {
        RenderJob *rj = rjv;
 
-       /* this render may be used again by the sequencer without the active 'Render' where the callbacks
-        * would be re-assigned. assign dummy callbacks to avoid referencing freed renderjobs bug [#24508] */
+       /* this render may be used again by the sequencer without the active
+        * 'Render' where the callbacks would be re-assigned. assign dummy callbacks
+        * to avoid referencing freed renderjobs bug T24508. */
        RE_InitRenderCB(rj->re);
 
        if (rj->main != G.main)
index 78cedf0..db84075 100644 (file)
@@ -670,7 +670,9 @@ static bool screen_opengl_render_init(bContext *C, wmOperator *op)
        oglrender->prevar = prevar;
 
        if (is_view_context) {
-               ED_view3d_context_user_region(C, &oglrender->v3d, &oglrender->ar); /* so quad view renders camera */
+               /* so quad view renders camera */
+               ED_view3d_context_user_region(C, &oglrender->v3d, &oglrender->ar);
+
                oglrender->rv3d = oglrender->ar->regiondata;
 
                /* MUST be cleared on exit */
index 8aa345d..a0c5a14 100644 (file)
@@ -1732,7 +1732,8 @@ void MATERIAL_OT_copy(wmOperatorType *ot)
        ot->exec = copy_material_exec;
 
        /* flags */
-       ot->flag = OPTYPE_REGISTER | OPTYPE_INTERNAL; /* no undo needed since no changes are made to the material */
+       /* no undo needed since no changes are made to the material */
+       ot->flag = OPTYPE_REGISTER | OPTYPE_INTERNAL;
 }
 
 static int paste_material_exec(bContext *C, wmOperator *UNUSED(op))
@@ -1886,7 +1887,8 @@ void TEXTURE_OT_slot_copy(wmOperatorType *ot)
        ot->poll = copy_mtex_poll;
 
        /* flags */
-       ot->flag = OPTYPE_REGISTER | OPTYPE_INTERNAL; /* no undo needed since no changes are made to the mtex */
+       /* no undo needed since no changes are made to the mtex */
+       ot->flag = OPTYPE_REGISTER | OPTYPE_INTERNAL;
 }
 
 static int paste_mtex_exec(bContext *C, wmOperator *UNUSED(op))
index 3ec2078..98ac33d 100644 (file)
@@ -279,7 +279,8 @@ static bool screenshot_poll(bContext *C)
 
 void SCREEN_OT_screenshot(wmOperatorType *ot)
 {
-       ot->name = "Save Screenshot"; /* weak: opname starting with 'save' makes filewindow give save-over */
+       /* weak: opname starting with 'save' makes filewindow give save-over */
+       ot->name = "Save Screenshot";
        ot->idname = "SCREEN_OT_screenshot";
        ot->description = "Capture a picture of the active area or whole Blender window";
 
index 90066a2..77a43de 100644 (file)
@@ -314,8 +314,10 @@ static int load_tex(Brush *br, ViewContext *vc, float zoom, bool col, bool prima
 
                pool = BKE_image_pool_new();
 
-               if (mtex->tex && mtex->tex->nodetree)
-                       ntreeTexBeginExecTree(mtex->tex->nodetree);  /* has internal flag to detect it only does it once */
+               if (mtex->tex && mtex->tex->nodetree) {
+                       /* has internal flag to detect it only does it once */
+                       ntreeTexBeginExecTree(mtex->tex->nodetree);
+               }
 
                LoadTexData data = {
                    .br = br, .vc = vc, .mtex = mtex, .buffer = buffer, .col = col,
@@ -787,7 +789,8 @@ static void paint_draw_alpha_overlay(
         UnifiedPaintSettings *ups, Brush *brush,
         ViewContext *vc, int x, int y, float zoom, ePaintMode mode)
 {
-       /* color means that primary brush texture is colured and secondary is used for alpha/mask control */
+       /* color means that primary brush texture is colured and
+        * secondary is used for alpha/mask control */
        bool col = ELEM(mode, PAINT_MODE_TEXTURE_3D, PAINT_MODE_TEXTURE_2D, PAINT_MODE_VERTEX) ? true : false;
        eOverlayControlFlags flags = BKE_paint_get_overlay_flags();
        /* save lots of GL state
@@ -1039,8 +1042,9 @@ static void paint_draw_cursor(bContext *C, int x, int y, void *UNUSED(unused))
        outline_col = brush->add_col;
        final_radius = (BKE_brush_size_get(scene, brush) * zoomx);
 
-       /* don't calculate rake angles while a stroke is active because the rake variables are global and
-        * we may get interference with the stroke itself. For line strokes, such interference is visible */
+       /* don't calculate rake angles while a stroke is active because the rake variables are global
+        * and we may get interference with the stroke itself.
+        * For line strokes, such interference is visible */
        if (!ups->stroke_active) {
                paint_calculate_rake_rotation(ups, brush, translation);
        }
index 59a5a87..c6e519d 100644 (file)
@@ -375,11 +375,14 @@ void paint_brush_init_tex(Brush *brush)
        /* init mtex nodes */
        if (brush) {
                MTex *mtex = &brush->mtex;
-               if (mtex->tex && mtex->tex->nodetree)
-                       ntreeTexBeginExecTree(mtex->tex->nodetree);  /* has internal flag to detect it only does it once */
+               if (mtex->tex && mtex->tex->nodetree) {
+                       /* has internal flag to detect it only does it once */
+                       ntreeTexBeginExecTree(mtex->tex->nodetree);
+               }
                mtex = &brush->mask_mtex;
-               if (mtex->tex && mtex->tex->nodetree)
+               if (mtex->tex && mtex->tex->nodetree) {
                        ntreeTexBeginExecTree(mtex->tex->nodetree);
+               }
        }
 }
 
@@ -1065,7 +1068,8 @@ static int texture_paint_toggle_exec(bContext *C, wmOperator *op)
 
                /* entering paint mode also sets image to editors */
                if (imapaint->mode == IMAGEPAINT_MODE_MATERIAL) {
-                       Material *ma = give_current_material(ob, ob->actcol); /* set the current material active paint slot on image editor */
+                       /* set the current material active paint slot on image editor */
+                       Material *ma = give_current_material(ob, ob->actcol);
 
                        if (ma && ma->texpaintslot)
                                ima = ma->texpaintslot[ma->paint_active_slot].ima;
index c5bd88d..c1b3491 100644 (file)
@@ -1520,7 +1520,8 @@ void paint_2d_bucket_fill(
                float image_init[2];
                int minx = ibuf->x, miny = ibuf->y, maxx = 0, maxy = 0;
                float pixel_color[4];
-               /* We are comparing to sum of three squared values (assumed in range [0,1]), so need to multiply... */
+               /* We are comparing to sum of three squared values
+                * (assumed in range [0,1]), so need to multiply... */
                float threshold_sq = br->fill_threshold * br->fill_threshold * 3;
 
                UI_view2d_region_to_view(s->v2d, mouse_init[0], mouse_init[1], &image_init[0], &image_init[1]);
index bfde868..bbbfb3e 100644 (file)
@@ -169,7 +169,8 @@ BLI_INLINE unsigned char f_to_char(const float val)
 #define PROJ_SRC_VIEW_FILL  4
 
 #define PROJ_VIEW_DATA_ID "view_data"
-#define PROJ_VIEW_DATA_SIZE (4 * 4 + 4 * 4 + 3) /* viewmat + winmat + clipsta + clipend + is_ortho */
+/* viewmat + winmat + clipsta + clipend + is_ortho */
+#define PROJ_VIEW_DATA_SIZE (4 * 4 + 4 * 4 + 3)
 
 #define PROJ_BUCKET_NULL        0
 #define PROJ_BUCKET_INIT        (1 << 0)
@@ -407,7 +408,8 @@ typedef struct ProjPixel {
         * new mask value is lower then mask_accum */
        unsigned short *mask_accum;
 
-       /* horrible hack, store tile valid flag pointer here to re-validate tiles used for anchored and drag-dot strokes */
+       /* horrible hack, store tile valid flag pointer here to re-validate tiles
+        * used for anchored and drag-dot strokes */
        bool *valid;
 
        PixelPointer origColor;
@@ -635,7 +637,8 @@ static void uvco_to_wrapped_pxco(const float uv[2], int ibuf_x, int ibuf_y, floa
        *y = *y * ibuf_y - 0.5f;
 }
 
-/* Set the top-most face color that the screen space coord 'pt' touches (or return 0 if none touch) */
+/* Set the top-most face color that the screen space coord 'pt' touches
+ * (or return 0 if none touch) */
 static bool project_paint_PickColor(
         const ProjPaintState *ps, const float pt[2],
         float *rgba_fp, unsigned char *rgba, const bool interp)
@@ -795,7 +798,8 @@ static int project_paint_occlude_ptv_clip(
 
 /* Check if a screenspace location is occluded by any other faces
  * check, pixelScreenCo must be in screenspace, its Z-Depth only needs to be used for comparison
- * and doesn't need to be correct in relation to X and Y coords (this is the case in perspective view) */
+ * and doesn't need to be correct in relation to X and Y coords
+ * (this is the case in perspective view) */
 static bool project_bucket_point_occluded(
         const ProjPaintState *ps, LinkNode *bucketFace,
         const int orig_face, const float pixelScreenCo[4])
@@ -1366,8 +1370,10 @@ static float project_paint_uvpixel_mask(
 
                        BKE_image_release_ibuf(other_tpage, ibuf_other, NULL);
 
-                       if (!ps->do_layer_stencil_inv) /* matching the gimps layer mask black/white rules, white==full opacity */
+                       if (!ps->do_layer_stencil_inv) {
+                               /* matching the gimps layer mask black/white rules, white==full opacity */
                                mask = (1.0f - mask);
+                       }
 
                        if (mask == 0.0f) {
                                return 0.0f;
@@ -1715,7 +1721,8 @@ static bool line_clip_rect2f(
                }
 
 
-               if (fabsf(l1[0] - l2[0]) < PROJ_PIXEL_TOLERANCE) { /* this is a single point  (or close to)*/
+               /* this is a single point  (or close to)*/
+               if (fabsf(l1[0] - l2[0]) < PROJ_PIXEL_TOLERANCE) {
                        if (BLI_rctf_isect_pt_v(rect, l1)) {
                                copy_v2_v2(l1_clip, l1);
                                copy_v2_v2(l2_clip, l2);
@@ -1743,7 +1750,8 @@ static bool line_clip_rect2f(
                        return 0;
                }
 
-               if (fabsf(l1[1] - l2[1]) < PROJ_PIXEL_TOLERANCE) { /* this is a single point  (or close to)*/
+               /* this is a single point  (or close to)*/
+               if (fabsf(l1[1] - l2[1]) < PROJ_PIXEL_TOLERANCE) {
                        if (BLI_rctf_isect_pt_v(rect, l1)) {
                                copy_v2_v2(l1_clip, l1);
                                copy_v2_v2(l2_clip, l2);
@@ -2120,7 +2128,8 @@ static void project_bucket_clip_face(
        float bucket_bounds_ss[4][2];
 
        /* detect pathological case where face the three vertices are almost collinear in screen space.
-        * mostly those will be culled but when flood filling or with smooth shading it's a possibility */
+        * mostly those will be culled but when flood filling or with
+        * smooth shading it's a possibility */
        if (min_fff(dist_squared_to_line_v2(v1coSS, v2coSS, v3coSS),
                    dist_squared_to_line_v2(v2coSS, v3coSS, v1coSS),
                    dist_squared_to_line_v2(v3coSS, v1coSS, v2coSS)) < PROJ_PIXEL_TOLERANCE)
@@ -2154,7 +2163,8 @@ static void project_bucket_clip_face(
                *tot = 3;
                return;
        }
-       /* handle pathological case here, no need for further intersections below since tringle area is almost zero */
+       /* handle pathological case here,
+        * no need for further intersections below since tringle area is almost zero */
        if (collinear) {
                int flag;
 
@@ -2202,7 +2212,8 @@ static void project_bucket_clip_face(
        }
 
        /* get the UV space bounding box */
-       /* use IsectPT2Df_limit here so we catch points are are touching the tri edge (or a small fraction over) */
+       /* use IsectPT2Df_limit here so we catch points are are touching the tri edge
+        * (or a small fraction over) */
        bucket_bounds_ss[0][0] = bucket_bounds->xmax;
        bucket_bounds_ss[0][1] = bucket_bounds->ymin;
        inside_face_flag |= (IsectPT2Df_limit(bucket_bounds_ss[0], v1coSS, v2coSS, v3coSS, 1 + PROJ_GEOM_TOLERANCE) ? ISECT_1 : 0);
@@ -2680,7 +2691,8 @@ static void project_paint_face_init(
 
 
 #if 0           /* TODO - investigate why this dosnt work sometimes! it should! */
-                               /* no intersection for this entire row, after some intersection above means we can quit now */
+                               /* no intersection for this entire row,
+                                * after some intersection above means we can quit now */
                                if (has_x_isect == 0 && has_isect) {
                                        break;
                                }
@@ -2799,15 +2811,17 @@ static void project_paint_face_init(
                                                                                float fac;
 
                                                                                /* We need to find the closest point along the face edge,
-                                                                                * getting the screen_px_from_*** wont work because our actual location
-                                                                                * is not relevant, since we are outside the face, Use VecLerpf to find
-                                                                                * our location on the side of the face's UV */
+                                                                                * getting the screen_px_from_*** wont work because our
+                                                                                * actual location is not relevant, since we are outside
+                                                                                * the face, Use VecLerpf to find our location on the side
+                                                                                * of the face's UV */
 #if 0
                                                                                if (is_ortho) screen_px_from_ortho(ps, uv, v1co, v2co, v3co, uv1co, uv2co, uv3co, pixelScreenCo);
                                                                                else          screen_px_from_persp(ps, uv, v1co, v2co, v3co, uv1co, uv2co, uv3co, pixelScreenCo);
 #endif
 
-                                                                               /* Since this is a seam we need to work out where on the line this pixel is */
+                                                                               /* Since this is a seam we need to work out where on
+                                                                                * the line this pixel is */
                                                                                //fac = line_point_factor_v2(uv, uv_seam_quad[0], uv_seam_quad[1]);
                                                                                fac = resolve_quad_u_v2(uv, UNPACK4(seam_subsection));
                                                                                interp_v3_v3v3(pixelScreenCo, edge_verts_inset_clip[0], edge_verts_inset_clip[1], fac);
@@ -2827,20 +2841,23 @@ static void project_paint_face_init(
                                                                                        if (ps->do_mask_normal || ps->dm_mloopuv_clone) {
                        &nb