[Bf-blender-cvs] [dd1be8db19a] temp-ui-cpp: Cleanup: Move interface View2D files to C++
Hans Goudey
noreply at git.blender.org
Sun Apr 3 04:32:36 CEST 2022
Commit: dd1be8db19a86d044a573317b8c57bc8d25b7b08
Author: Hans Goudey
Date: Sat Apr 2 21:32:25 2022 -0500
Branches: temp-ui-cpp
https://developer.blender.org/rBdd1be8db19a86d044a573317b8c57bc8d25b7b08
Cleanup: Move interface View2D files to C++
===================================================================
M source/blender/editors/interface/CMakeLists.txt
R097 source/blender/editors/interface/view2d.c source/blender/editors/interface/view2d.cc
R092 source/blender/editors/interface/view2d_draw.c source/blender/editors/interface/view2d_draw.cc
R099 source/blender/editors/interface/view2d_edge_pan.c source/blender/editors/interface/view2d_edge_pan.cc
R088 source/blender/editors/interface/view2d_gizmo_navigate.c source/blender/editors/interface/view2d_gizmo_navigate.cc
R095 source/blender/editors/interface/view2d_ops.c source/blender/editors/interface/view2d_ops.cc
M source/blender/windowmanager/gizmo/WM_gizmo_types.h
===================================================================
diff --git a/source/blender/editors/interface/CMakeLists.txt b/source/blender/editors/interface/CMakeLists.txt
index a1ee5c38838..0bc52fd3a38 100644
--- a/source/blender/editors/interface/CMakeLists.txt
+++ b/source/blender/editors/interface/CMakeLists.txt
@@ -69,11 +69,11 @@ set(SRC
interface_widgets.c
resources.c
tree_view.cc
- view2d.c
- view2d_draw.c
- view2d_edge_pan.c
- view2d_gizmo_navigate.c
- view2d_ops.c
+ view2d.cc
+ view2d_draw.cc
+ view2d_edge_pan.cc
+ view2d_gizmo_navigate.cc
+ view2d_ops.cc
interface_eyedropper_intern.h
interface_intern.h
diff --git a/source/blender/editors/interface/view2d.c b/source/blender/editors/interface/view2d.cc
similarity index 97%
rename from source/blender/editors/interface/view2d.c
rename to source/blender/editors/interface/view2d.cc
index bf3c8362ce6..66171dc13e4 100644
--- a/source/blender/editors/interface/view2d.c
+++ b/source/blender/editors/interface/view2d.cc
@@ -5,10 +5,10 @@
* \ingroup edinterface
*/
-#include <float.h>
-#include <limits.h>
-#include <math.h>
-#include <string.h>
+#include <cfloat>
+#include <climits>
+#include <cmath>
+#include <cstring>
#include "MEM_guardedalloc.h"
@@ -123,7 +123,7 @@ static void view2d_masks(View2D *v2d, const rcti *mask_scroll)
/* mask - view frame */
UI_view2d_mask_from_win(v2d, &v2d->mask);
- if (mask_scroll == NULL) {
+ if (mask_scroll == nullptr) {
mask_scroll = &v2d->mask;
}
@@ -356,7 +356,7 @@ void UI_view2d_region_reinit(View2D *v2d, short type, int winx, int winy)
v2d->winx = winx;
v2d->winy = winy;
- view2d_masks(v2d, NULL);
+ view2d_masks(v2d, nullptr);
if (do_init) {
/* Visible by default. */
@@ -820,7 +820,7 @@ static void ui_view2d_curRect_validate_resize(View2D *v2d, bool resize)
}
/* set masks */
- view2d_masks(v2d, NULL);
+ view2d_masks(v2d, nullptr);
}
void UI_view2d_curRect_validate(View2D *v2d)
@@ -834,7 +834,7 @@ void UI_view2d_curRect_changed(const bContext *C, View2D *v2d)
ARegion *region = CTX_wm_region(C);
- if (region->type->on_view2d_changed != NULL) {
+ if (region->type->on_view2d_changed != nullptr) {
region->type->on_view2d_changed(C, region);
}
}
@@ -1662,7 +1662,8 @@ void UI_view2d_region_to_view(
void UI_view2d_region_to_view_rctf(const View2D *v2d, const rctf *rect_src, rctf *rect_dst)
{
const float cur_size[2] = {BLI_rctf_size_x(&v2d->cur), BLI_rctf_size_y(&v2d->cur)};
- const float mask_size[2] = {BLI_rcti_size_x(&v2d->mask), BLI_rcti_size_y(&v2d->mask)};
+ const float mask_size[2] = {(float)BLI_rcti_size_x(&v2d->mask),
+ (float)BLI_rcti_size_y(&v2d->mask)};
rect_dst->xmin = (v2d->cur.xmin +
(cur_size[0] * (rect_src->xmin - v2d->mask.xmin) / mask_size[0]));
@@ -1737,7 +1738,8 @@ void UI_view2d_view_to_region_fl(
void UI_view2d_view_to_region_rcti(const View2D *v2d, const rctf *rect_src, rcti *rect_dst)
{
const float cur_size[2] = {BLI_rctf_size_x(&v2d->cur), BLI_rctf_size_y(&v2d->cur)};
- const float mask_size[2] = {BLI_rcti_size_x(&v2d->mask), BLI_rcti_size_y(&v2d->mask)};
+ const float mask_size[2] = {(float)BLI_rcti_size_x(&v2d->mask),
+ (float)BLI_rcti_size_y(&v2d->mask)};
rctf rect_tmp;
/* Step 1: express given coordinates as proportional values. */
@@ -1766,7 +1768,8 @@ void UI_view2d_view_to_region_m4(const View2D *v2d, float matrix[4][4])
bool UI_view2d_view_to_region_rcti_clip(const View2D *v2d, const rctf *rect_src, rcti *rect_dst)
{
const float cur_size[2] = {BLI_rctf_size_x(&v2d->cur), BLI_rctf_size_y(&v2d->cur)};
- const float mask_size[2] = {BLI_rcti_size_x(&v2d->mask), BLI_rcti_size_y(&v2d->mask)};
+ const float mask_size[2] = {(float)BLI_rcti_size_x(&v2d->mask),
+ (float)BLI_rcti_size_y(&v2d->mask)};
rctf rect_tmp;
BLI_assert(rect_src->xmin <= rect_src->xmax && rect_src->ymin <= rect_src->ymax);
@@ -1805,11 +1808,11 @@ View2D *UI_view2d_fromcontext(const bContext *C)
ScrArea *area = CTX_wm_area(C);
ARegion *region = CTX_wm_region(C);
- if (area == NULL) {
- return NULL;
+ if (area == nullptr) {
+ return nullptr;
}
- if (region == NULL) {
- return NULL;
+ if (region == nullptr) {
+ return nullptr;
}
return &(region->v2d);
}
@@ -1819,15 +1822,15 @@ View2D *UI_view2d_fromcontext_rwin(const bContext *C)
ScrArea *area = CTX_wm_area(C);
ARegion *region = CTX_wm_region(C);
- if (area == NULL) {
- return NULL;
+ if (area == nullptr) {
+ return nullptr;
}
- if (region == NULL) {
- return NULL;
+ if (region == nullptr) {
+ return nullptr;
}
if (region->regiontype != RGN_TYPE_WINDOW) {
ARegion *region_win = BKE_area_find_region_type(area, RGN_TYPE_WINDOW);
- return region_win ? &(region_win->v2d) : NULL;
+ return region_win ? &(region_win->v2d) : nullptr;
}
return &(region->v2d);
}
@@ -1997,7 +2000,7 @@ char UI_view2d_rect_in_scrollers(const ARegion *region, const View2D *v2d, const
/** \name View2D Text Drawing Cache
* \{ */
-typedef struct View2DString {
+struct View2DString {
struct View2DString *next;
union {
uchar ub[4];
@@ -2008,11 +2011,11 @@ typedef struct View2DString {
/* str is allocated past the end */
char str[0];
-} View2DString;
+};
/* assumes caches are used correctly, so for time being no local storage in v2d */
-static MemArena *g_v2d_strings_arena = NULL;
-static View2DString *g_v2d_strings = NULL;
+static MemArena *g_v2d_strings_arena = nullptr;
+static View2DString *g_v2d_strings = nullptr;
void UI_view2d_text_cache_add(
View2D *v2d, float x, float y, const char *str, size_t str_len, const uchar col[4])
@@ -2023,13 +2026,13 @@ void UI_view2d_text_cache_add(
if (UI_view2d_view_to_region_clip(v2d, x, y, &mval[0], &mval[1])) {
const int alloc_len = str_len + 1;
- View2DString *v2s;
- if (g_v2d_strings_arena == NULL) {
+ if (g_v2d_strings_arena == nullptr) {
g_v2d_strings_arena = BLI_memarena_new(MEM_SIZE_OPTIMAL(1 << 14), __func__);
}
- v2s = BLI_memarena_alloc(g_v2d_strings_arena, sizeof(View2DString) + alloc_len);
+ View2DString *v2s = static_cast<View2DString *>(
+ BLI_memarena_alloc(g_v2d_strings_arena, sizeof(View2DString) + alloc_len));
BLI_LINKS_PREPEND(g_v2d_strings, v2s);
@@ -2053,13 +2056,13 @@ void UI_view2d_text_cache_add_rectf(
if (UI_view2d_view_to_region_rcti_clip(v2d, rect_view, &rect)) {
const int alloc_len = str_len + 1;
- View2DString *v2s;
- if (g_v2d_strings_arena == NULL) {
+ if (g_v2d_strings_arena == nullptr) {
g_v2d_strings_arena = BLI_memarena_new(MEM_SIZE_OPTIMAL(1 << 14), __func__);
}
- v2s = BLI_memarena_alloc(g_v2d_strings_arena, sizeof(View2DString) + alloc_len);
+ View2DString *v2s = static_cast<View2DString *>(
+ BLI_memarena_alloc(g_v2d_strings_arena, sizeof(View2DString) + alloc_len));
BLI_LINKS_PREPEND(g_v2d_strings, v2s);
@@ -2116,11 +2119,11 @@ void UI_view2d_text_cache_draw(ARegion *region)
BLF_disable(font_id, BLF_CLIPPING);
}
}
- g_v2d_strings = NULL;
+ g_v2d_strings = nullptr;
if (g_v2d_strings_arena) {
BLI_memarena_free(g_v2d_strings_arena);
- g_v2d_strings_arena = NULL;
+ g_v2d_strings_arena = nullptr;
}
}
diff --git a/source/blender/editors/interface/view2d_draw.c b/source/blender/editors/interface/view2d_draw.cc
similarity index 92%
rename from source/blender/editors/interface/view2d_draw.c
rename to source/blender/editors/interface/view2d_draw.cc
index 600aa3b522b..d76230ba99c 100644
--- a/source/blender/editors/interface/view2d_draw.c
+++ b/source/blender/editors/interface/view2d_draw.cc
@@ -5,22 +5,22 @@
* \ingroup edinterface
*/
-#include <float.h>
-#include <limits.h>
-#include <math.h>
-#include <string.h>
+#include <cfloat>
+#include <climits>
+#include <cmath>
+#include <cstring>
#include "MEM_guardedalloc.h"
#include "DNA_scene_types.h"
#include "DNA_userdef_types.h"
-#include "BLI_array.h"
#include "BLI_math.h"
#include "BLI_rect.h"
#include "BLI_string.h"
#include "BLI_timecode.h"
#include "BLI_utildefines.h"
+#include "BLI_vector.hh"
#include "GPU_immediate.h"
#include "GPU_matrix.h"
@@ -97,40 +97,38 @@ static float view2d_major_step_x__time(const View2D *v2d, const Scene *scene)
{
const double fps = FPS;
- float *possible_distances = NULL;
- BLI_array_staticdeclare(possible_distances, 32);
-
- for (uint step = 1; step < fps; step *= 2) {
- BLI_array_append(possible_distances, step);
- }
- BLI_array_append(possible_distances, fps);
- BLI_array_append(possible_distances, 2 * fps);
- BLI_array_append(possible_distances, 5 * fps);
- BLI_array_append(possible_distances, 10 * fps);
- BLI_array_append(possible_distances, 30 * fps);
- BLI_array_append(possible_distances, 60 * fps);
- BLI_array_append(possible_distances, 2 * 60 * fps);
- BLI_array_append(possible_distances, 5 * 60 * fps);
- BLI_array_append(possible_distances, 10 * 60 * fps);
- BLI_array_append(possible_distances, 30 * 60 * fps);
- BLI_array_append(possible_distances, 60 * 60 * fps);
-
- float distance = select_major_distance(possible_distances,
- BLI_array_len(possible_distances),
+ blender::Vector<float, 32> possible_distances;
+
+ for (int step = 1; step < fps; step *= 2) {
+ possible_distances.append(step);
+ }
+ possible_distances.append(fps);
+ possible_distances.append(2 * fps);
+ possible_distances.append(5 * fps);
+ possible_distances.append(10 * fps);
+ possible_distances.append(30 * fps);
+ possible_distances.append(60 * fps);
+ possible_distances.append(2 * 60 * fps);
+ possible_distances.append(5 * 60 * fps);
+ possible_distances.append(10 * 60 * fps);
+ possible_distances.append(30 * 60 * fps);
+ possible_distances.append(60 * 60 * fps);
+
+ float distance = select_major_distance(possible_distances.data(),
+ possible_distances.size(),
BLI_rcti_size_x(&v2d->mask),
BLI_rctf_size_x(&v2d->cur));
- BLI_array_free(possibl
@@ Diff output truncated at 10240 characters. @@
More information about the Bf-blender-cvs
mailing list