Revision: 50064 http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=50064 Author: campbellbarton Date: 2012-08-20 23:06:17 +0000 (Mon, 20 Aug 2012) Log Message: ----------- code cleanup: use BLI_RCT_SIZE macro
Modified Paths: -------------- trunk/blender/source/blender/blenlib/intern/rct.c trunk/blender/source/blender/compositor/operations/COM_DilateErodeOperation.cpp trunk/blender/source/blender/editors/animation/anim_markers.c trunk/blender/source/blender/editors/interface/interface.c trunk/blender/source/blender/editors/interface/interface_draw.c trunk/blender/source/blender/editors/interface/interface_handlers.c trunk/blender/source/blender/editors/interface/interface_panel.c trunk/blender/source/blender/editors/interface/interface_regions.c trunk/blender/source/blender/editors/interface/interface_style.c trunk/blender/source/blender/editors/interface/interface_templates.c trunk/blender/source/blender/editors/interface/interface_widgets.c trunk/blender/source/blender/editors/interface/view2d.c trunk/blender/source/blender/editors/interface/view2d_ops.c trunk/blender/source/blender/editors/mask/mask_draw.c trunk/blender/source/blender/editors/mesh/editface.c trunk/blender/source/blender/editors/render/render_preview.c trunk/blender/source/blender/editors/screen/area.c trunk/blender/source/blender/editors/screen/screen_ops.c trunk/blender/source/blender/editors/space_action/action_draw.c trunk/blender/source/blender/editors/space_action/action_edit.c trunk/blender/source/blender/editors/space_action/action_select.c trunk/blender/source/blender/editors/space_clip/clip_buttons.c trunk/blender/source/blender/editors/space_clip/clip_dopesheet_draw.c trunk/blender/source/blender/editors/space_clip/clip_draw.c trunk/blender/source/blender/editors/space_clip/clip_editor.c trunk/blender/source/blender/editors/space_clip/clip_graph_ops.c trunk/blender/source/blender/editors/space_clip/clip_ops.c trunk/blender/source/blender/editors/space_clip/space_clip.c trunk/blender/source/blender/editors/space_console/space_console.c trunk/blender/source/blender/editors/space_file/filesel.c trunk/blender/source/blender/editors/space_file/space_file.c trunk/blender/source/blender/editors/space_graph/graph_draw.c trunk/blender/source/blender/editors/space_graph/graph_select.c trunk/blender/source/blender/editors/space_image/image_buttons.c trunk/blender/source/blender/editors/space_image/image_draw.c trunk/blender/source/blender/editors/space_image/image_edit.c trunk/blender/source/blender/editors/space_image/image_ops.c trunk/blender/source/blender/editors/space_image/space_image.c trunk/blender/source/blender/editors/space_nla/nla_edit.c trunk/blender/source/blender/editors/space_nla/nla_select.c trunk/blender/source/blender/editors/space_node/drawnode.c trunk/blender/source/blender/editors/space_node/node_draw.c trunk/blender/source/blender/editors/space_node/node_view.c trunk/blender/source/blender/editors/space_outliner/outliner_edit.c trunk/blender/source/blender/editors/space_sequencer/sequencer_draw.c trunk/blender/source/blender/editors/space_sequencer/sequencer_edit.c trunk/blender/source/blender/editors/space_time/time_ops.c trunk/blender/source/blender/editors/space_view3d/view3d_draw.c trunk/blender/source/blender/editors/space_view3d/view3d_edit.c trunk/blender/source/blender/editors/space_view3d/view3d_select.c trunk/blender/source/blender/editors/transform/transform.c trunk/blender/source/blender/windowmanager/intern/wm_subwindow.c trunk/blender/source/gameengine/BlenderRoutines/BL_KetsjiEmbedStart.cpp Modified: trunk/blender/source/blender/blenlib/intern/rct.c =================================================================== --- trunk/blender/source/blender/blenlib/intern/rct.c 2012-08-20 21:36:35 UTC (rev 50063) +++ trunk/blender/source/blender/blenlib/intern/rct.c 2012-08-20 23:06:17 UTC (rev 50064) @@ -382,11 +382,11 @@ void print_rctf(const char *str, const rctf *rect) { printf("%s: xmin %.3f, xmax %.3f, ymin %.3f, ymax %.3f (%.3fx%.3f)\n", str, - rect->xmin, rect->xmax, rect->ymin, rect->ymax, rect->xmax - rect->xmin, rect->ymax - rect->ymin); + rect->xmin, rect->xmax, rect->ymin, rect->ymax, BLI_RCT_SIZE_X(rect), BLI_RCT_SIZE_Y(rect)); } void print_rcti(const char *str, const rcti *rect) { printf("%s: xmin %d, xmax %d, ymin %d, ymax %d (%dx%d)\n", str, - rect->xmin, rect->xmax, rect->ymin, rect->ymax, rect->xmax - rect->xmin, rect->ymax - rect->ymin); + rect->xmin, rect->xmax, rect->ymin, rect->ymax, BLI_RCT_SIZE_X(rect), BLI_RCT_SIZE_Y(rect)); } Modified: trunk/blender/source/blender/compositor/operations/COM_DilateErodeOperation.cpp =================================================================== --- trunk/blender/source/blender/compositor/operations/COM_DilateErodeOperation.cpp 2012-08-20 21:36:35 UTC (rev 50063) +++ trunk/blender/source/blender/compositor/operations/COM_DilateErodeOperation.cpp 2012-08-20 23:06:17 UTC (rev 50064) @@ -79,7 +79,7 @@ const int miny = max(y - this->m_scope, rect->ymin); const int maxx = min(x + this->m_scope, rect->xmax); const int maxy = min(y + this->m_scope, rect->ymax); - const int bufferWidth = rect->xmax - rect->xmin; + const int bufferWidth = BLI_RCT_SIZE_X(rect); int offset; this->m_inputProgram->read(inputValue, x, y, NULL); @@ -199,7 +199,7 @@ const int miny = max(y - this->m_scope, rect->ymin); const int maxx = min(x + this->m_scope, rect->xmax); const int maxy = min(y + this->m_scope, rect->ymax); - const int bufferWidth = rect->xmax - rect->xmin; + const int bufferWidth = BLI_RCT_SIZE_X(rect); int offset; float value = 0.0f; @@ -273,7 +273,7 @@ const int miny = max(y - this->m_scope, rect->ymin); const int maxx = min(x + this->m_scope, rect->xmax); const int maxy = min(y + this->m_scope, rect->ymax); - const int bufferWidth = rect->xmax - rect->xmin; + const int bufferWidth = BLI_RCT_SIZE_X(rect); int offset; float value = 1.0f; Modified: trunk/blender/source/blender/editors/animation/anim_markers.c =================================================================== --- trunk/blender/source/blender/editors/animation/anim_markers.c 2012-08-20 21:36:35 UTC (rev 50063) +++ trunk/blender/source/blender/editors/animation/anim_markers.c 2012-08-20 23:06:17 UTC (rev 50064) @@ -354,7 +354,7 @@ xpos = marker->frame; /* no time correction for framelen! space is drawn with old values */ - ypixels = v2d->mask.ymax - v2d->mask.ymin; + ypixels = BLI_RCT_SIZE_Y(&v2d->mask); UI_view2d_getscale(v2d, &xscale, &yscale); glScalef(1.0f / xscale, 1.0f, 1.0f); Modified: trunk/blender/source/blender/editors/interface/interface.c =================================================================== --- trunk/blender/source/blender/editors/interface/interface.c 2012-08-20 21:36:35 UTC (rev 50063) +++ trunk/blender/source/blender/editors/interface/interface.c 2012-08-20 23:06:17 UTC (rev 50064) @@ -99,8 +99,8 @@ float gx, gy; int sx, sy, getsizex, getsizey; - getsizex = ar->winrct.xmax - ar->winrct.xmin + 1; - getsizey = ar->winrct.ymax - ar->winrct.ymin + 1; + getsizex = BLI_RCT_SIZE_X(&ar->winrct) + 1; + getsizey = BLI_RCT_SIZE_Y(&ar->winrct) + 1; sx = ar->winrct.xmin; sy = ar->winrct.ymin; @@ -145,8 +145,8 @@ float a, b, c, d, e, f, px, py; int sx, sy, getsizex, getsizey; - getsizex = ar->winrct.xmax - ar->winrct.xmin + 1; - getsizey = ar->winrct.ymax - ar->winrct.ymin + 1; + getsizex = BLI_RCT_SIZE_X(&ar->winrct) + 1; + getsizey = BLI_RCT_SIZE_Y(&ar->winrct) + 1; sx = ar->winrct.xmin; sy = ar->winrct.ymin; @@ -272,7 +272,7 @@ block->rect.ymax += block->bounds; } - block->rect.xmax = block->rect.xmin + maxf(block->rect.xmax - block->rect.xmin, block->minbounds); + block->rect.xmax = block->rect.xmin + maxf(BLI_RCT_SIZE_X(&block->rect), block->minbounds); /* hardcoded exception... but that one is annoying with larger safety */ bt = block->buttons.first; @@ -325,8 +325,8 @@ wm_window_get_size(window, &xmax, &ymax); - oldwidth = block->rect.xmax - block->rect.xmin; - oldheight = block->rect.ymax - block->rect.ymin; + oldwidth = BLI_RCT_SIZE_X(&block->rect); + oldheight = BLI_RCT_SIZE_Y(&block->rect); /* first we ensure wide enough text bounds */ if (bounds_calc == UI_BLOCK_BOUNDS_POPUP_MENU) { @@ -341,8 +341,8 @@ ui_bounds_block(block); /* and we adjust the position to fit within window */ - width = block->rect.xmax - block->rect.xmin; - height = block->rect.ymax - block->rect.ymin; + width = BLI_RCT_SIZE_X(&block->rect); + height = BLI_RCT_SIZE_Y(&block->rect); /* avoid divide by zero below, caused by calling with no UI, but better not crash */ oldwidth = oldwidth > 0 ? oldwidth : MAX2(1, width); @@ -2204,7 +2204,7 @@ /* safety is 4 to enable small number buttons (like 'users') */ - // okwidth= -4 + (but->rect.xmax - but->rect.xmin); // UNUSED + // okwidth= -4 + (BLI_RCT_SIZE_X(&but->rect)); // UNUSED /* name: */ switch (but->type) { @@ -2212,7 +2212,7 @@ case MENU: case ICONTEXTROW: - if (but->rect.xmax - but->rect.xmin > 24) { + if (BLI_RCT_SIZE_X(&but->rect) > 24.0f) { UI_GET_BUT_VALUE_INIT(but, value); ui_set_name_menu(but, (int)value); } Modified: trunk/blender/source/blender/editors/interface/interface_draw.c =================================================================== --- trunk/blender/source/blender/editors/interface/interface_draw.c 2012-08-20 21:36:35 UTC (rev 50063) +++ trunk/blender/source/blender/editors/interface/interface_draw.c 2012-08-20 23:06:17 UTC (rev 50064) @@ -493,8 +493,8 @@ charmax = G.charmax = 0xffff; /* Calculate the size of the button */ - width = abs(rect->xmax - rect->xmin); - height = abs(rect->ymax - rect->ymin); + width = absBLI_RCT_SIZE_X(rect); + height = absBLI_RCT_SIZE_Y(rect); butw = floor(width / 12); buth = floor(height / 6); @@ -642,8 +642,8 @@ glBlendFunc(GL_SRC_ALPHA, GL_ONE_MINUS_SRC_ALPHA); /* scale widget */ - scaler_x1 = rect->xmin + (rect->xmax - rect->xmin) / 2 - SCOPE_RESIZE_PAD; - scaler_x2 = rect->xmin + (rect->xmax - rect->xmin) / 2 + SCOPE_RESIZE_PAD; + scaler_x1 = rect->xmin + BLI_RCT_SIZE_X(rect) / 2 - SCOPE_RESIZE_PAD; + scaler_x2 = rect->xmin + BLI_RCT_SIZE_X(rect) / 2 + SCOPE_RESIZE_PAD; glColor4f(0.f, 0.f, 0.f, 0.25f); fdrawline(scaler_x1, rect->ymin - 4, scaler_x2, rect->ymin - 4); @@ -805,9 +805,9 @@ if (scopes->wavefrm_yfac < 0.5f) scopes->wavefrm_yfac = 0.98f; - w = rect.xmax - rect.xmin - 7; - h = (rect.ymax - rect.ymin) * scopes->wavefrm_yfac; - yofs = rect.ymin + (rect.ymax - rect.ymin - h) / 2.0f; + w = BLI_RCT_SIZE_X(&rect) - 7; + h = BLI_RCT_SIZE_Y(&rect) * scopes->wavefrm_yfac; + yofs = rect.ymin + (BLI_RCT_SIZE_Y(&rect) - h) / 2.0f; w3 = w / 3.0f; /* log scale for alpha */ @@ -1033,8 +1033,8 @@ rect.ymin = (float)recti->ymin + SCOPE_RESIZE_PAD + 2; rect.ymax = (float)recti->ymax - 1; - w = rect.xmax - rect.xmin; - h = rect.ymax - rect.ymin; + w = BLI_RCT_SIZE_X(&rect); + h = BLI_RCT_SIZE_Y(&rect); centerx = rect.xmin + w / 2; centery = rect.ymin + h / 2; diam = (w < h) ? w : h; @@ -1262,12 +1262,12 @@ /* transform to button */ glPushMatrix(); - glTranslatef(rect->xmin + 0.5f * (rect->xmax - rect->xmin), rect->ymin + 0.5f * (rect->ymax - rect->ymin), 0.0f); + glTranslatef(rect->xmin + 0.5f * BLI_RCT_SIZE_X(rect), rect->ymin + 0.5f * BLI_RCT_SIZE_Y(rect), 0.0f); - if (rect->xmax - rect->xmin < rect->ymax - rect->ymin) - size = (rect->xmax - rect->xmin) / 200.f; + if (BLI_RCT_SIZE_X(rect) < BLI_RCT_SIZE_Y(rect)) + size = BLI_RCT_SIZE_X(rect) / 200.f; else - size = (rect->ymax - rect->ymin) / 200.f; + size = BLI_RCT_SIZE_Y(rect) / 200.f; glScalef(size, size, size); @@ -1368,8 +1368,8 @@ glScissor(scissor_new.xmin, scissor_new.ymin, scissor_new.xmax - scissor_new.xmin, scissor_new.ymax - scissor_new.ymin); /* calculate offset and zoom */ - zoomx = (rect->xmax - rect->xmin - 2.0f * but->aspect) / (cumap->curr.xmax - cumap->curr.xmin); - zoomy = (rect->ymax - rect->ymin - 2.0f * but->aspect) / (cumap->curr.ymax - cumap->curr.ymin); + zoomx = (BLI_RCT_SIZE_X(rect) - 2.0f * but->aspect) / BLI_RCT_SIZE_X(&cumap->curr); + zoomy = (BLI_RCT_SIZE_Y(rect) - 2.0f * but->aspect) / BLI_RCT_SIZE_Y(&cumap->curr); offsx = cumap->curr.xmin - but->aspect / zoomx; offsy = cumap->curr.ymin - but->aspect / zoomy; @@ -1545,8 +1545,8 @@ rect.ymin = (float)recti->ymin + SCOPE_RESIZE_PAD + 2; rect.ymax = (float)recti->ymax - 1; - width = rect.xmax - rect.xmin + 1; - height = rect.ymax - rect.ymin; + width = BLI_RCT_SIZE_X(&rect) + 1; + height = BLI_RCT_SIZE_Y(&rect); glEnable(GL_BLEND); glBlendFunc(GL_SRC_ALPHA, GL_ONE_MINUS_SRC_ALPHA); @@ -1621,8 +1621,8 @@ glTranslatef(rect.xmin + track_pos[0], rect.ymin + track_pos[1], 0.f); glScissor(ar->winrct.xmin + rect.xmin, ar->winrct.ymin + rect.ymin, - rect.xmax - rect.xmin, - rect.ymax - rect.ymin); + BLI_RCT_SIZE_X(&rect), + BLI_RCT_SIZE_Y(&rect)); for (a = 0; a < 2; a++) { if (a == 1) { Modified: trunk/blender/source/blender/editors/interface/interface_handlers.c =================================================================== --- trunk/blender/source/blender/editors/interface/interface_handlers.c 2012-08-20 21:36:35 UTC (rev 50063) +++ trunk/blender/source/blender/editors/interface/interface_handlers.c 2012-08-20 23:06:17 UTC (rev 50064) @@ -690,7 +690,7 @@ if (but->imb) ; /* use button size itself */ else if (but->flag & UI_ICON_LEFT) { - rect.xmax = rect.xmin + (rect.ymax - rect.ymin); + rect.xmax = rect.xmin + (BLI_RCT_SIZE_Y(&rect)); } else { int delta = BLI_RCT_SIZE_X(&rect) - BLI_RCT_SIZE_Y(&rect); @@ -714,7 +714,7 @@ drag = WM_event_start_drag(C, but->icon, but->dragtype, but->dragpoin, ui_get_but_val(but)); if (but->imb) - WM_event_drag_image(drag, but->imb, but->imb_scale, but->rect.xmax - but->rect.xmin, but->rect.ymax - but->rect.ymin); @@ Diff output truncated at 10240 characters. @@ _______________________________________________ Bf-blender-cvs mailing list Bf-blender-cvs@blender.org http://lists.blender.org/mailman/listinfo/bf-blender-cvs