Page MenuHome
Differential D2360 Diff 7831 source/blender/editors/interface/view2d.c

Changeset View

Standalone View

View Options

source/blender/editors/interface/view2d.c

Show First 20 Lines • Show All 1,287 Lines • ▼ Show 20 LinesView2DGrid *UI_view2d_grid_calc(
return grid; return grid;
} }
/* Draw gridlines in the given 2d-region */ /* Draw gridlines in the given 2d-region */
void UI_view2d_grid_draw(View2D *v2d, View2DGrid *grid, int flag) void UI_view2d_grid_draw(View2D *v2d, View2DGrid *grid, int flag)
{ {
float vec1[2], vec2[2]; float vec1[2], vec2[2];
int a, step; int a, step;
int vertical_minor_step = (BLI_rcti_size_x(&v2d->mask) + 1) / (U.v2d_min_gridsize * UI_DPI_FAC),
krashUnsubmitted
Done
Inline Actions

I'm not fussed about style personally, but convention is to use style of surrounding code or file. This would be one of few functions in this file that would use camel case for local variables. @Mike Erwin (merwin) can decide how picky to be.

krash: I'm not fussed about style personally, but convention is to use style of surrounding code or…
horizontal_major_step = (BLI_rcti_size_y(&v2d->mask) + 1) / (U.v2d_min_gridsize * UI_DPI_FAC);
unsigned char draw_color[3];
krashUnsubmitted
Done
Inline Actions

You use float here for drawColor but then use "ubv" below for UI and attribute functions. I think either switch this to unsigned char or use "fv" functions below.

krash: You use float here for drawColor but then use "ubv" below for UI and attribute functions. I…
/* check for grid first, as it may not exist */ /* check for grid first, as it may not exist */
if (grid == NULL) if (grid == NULL)
return; return;
glBegin(GL_LINES); /* Count the needed vertices for the gridlines */
unsigned vertex_count = 0;
if (flag & V2D_VERTICAL_LINES) {
/* vertical lines */
vertex_count += 2 * vertical_minor_step; /* minor gridlines */
vertex_count += 2 * (vertical_minor_step + 2); /* major gridlines */
}
if (flag & V2D_HORIZONTAL_LINES) {
/* horizontal lines */
vertex_count += 2 * (horizontal_major_step + 1); /* major gridlines */
/* fine lines */
if (flag & V2D_HORIZONTAL_FINELINES)
vertex_count += 2 * (horizontal_major_step + 1);
}
/* axes */
if (flag & V2D_HORIZONTAL_AXIS)
vertex_count += 2;
if (flag & V2D_VERTICAL_AXIS)
vertex_count += 2;
/* If there is nothing to render, exit early */
if (vertex_count == 0)
return;
/* Setup Gawain */
merwinUnsubmitted
Done
Inline Actions

These comments are obvious & can be deleted:
Setup Gawain
Start Rendering
Stop Rendering

merwin: These comments are obvious & can be deleted: Setup Gawain Start Rendering Stop Rendering
VertexFormat *format = immVertexFormat();
unsigned pos = add_attrib(format, "pos", GL_FLOAT, 2, KEEP_FLOAT);
unsigned color = add_attrib(format, "color", GL_UNSIGNED_BYTE, 3, NORMALIZE_INT_TO_FLOAT);
krashUnsubmitted
Done
Inline Actions

I don't think you need smooth color between verts of single lines. Use GPU_SHADER_FLAT_COLOR.

krash: I don't think you need smooth color between verts of single lines. Use GPU_SHADER_FLAT_COLOR.
krashUnsubmitted
Done
Inline Actions

Not sure if vertexCount is ever zero in current use of grid drawing (I tried in several editors to get zero but it never happened), but it would debug assert if it was ever possible.

krash: Not sure if vertexCount is ever zero in current use of grid drawing (I tried in several editors…
/* Start Rendering */
immBindBuiltinProgram(GPU_SHADER_2D_FLAT_COLOR);
immBegin(GL_LINES, vertex_count);
/* vertical lines */ /* vertical lines */
if (flag & V2D_VERTICAL_LINES) { if (flag & V2D_VERTICAL_LINES) {
/* initialize initial settings */ /* initialize initial settings */
vec1[0] = vec2[0] = grid->startx; vec1[0] = vec2[0] = grid->startx;
vec1[1] = grid->starty; vec1[1] = grid->starty;
vec2[1] = v2d->cur.ymax; vec2[1] = v2d->cur.ymax;
/* minor gridlines */ /* minor gridlines */
step = (BLI_rcti_size_x(&v2d->mask) + 1) / (U.v2d_min_gridsize * UI_DPI_FAC); step = vertical_minor_step;
UI_ThemeColor(TH_GRID); if (step != 0) {
UI_GetThemeColor3ubv(TH_GRID, draw_color);
immAttrib3ubv(color, draw_color);
for (a = 0; a < step; a++) { for (a = 0; a < step; a++) {
glVertex2fv(vec1); immVertex2fv(pos, vec1);
glVertex2fv(vec2); immVertex2fv(pos, vec2);
vec2[0] = vec1[0] += grid->dx; vec2[0] = vec1[0] += grid->dx;
} }
}
/* major gridlines */ /* major gridlines */
vec2[0] = vec1[0] -= 0.5f * grid->dx; vec2[0] = vec1[0] -= 0.5f * grid->dx;
UI_ThemeColorShade(TH_GRID, 16);
UI_GetThemeColorShade3ubv(TH_GRID, 16, draw_color);
immAttrib3ubv(color, draw_color);
step++; step++;
for (a = 0; a <= step; a++) { for (a = 0; a <= step; a++) {
glVertex2fv(vec1); immVertex2fv(pos, vec1);
glVertex2fv(vec2); immVertex2fv(pos, vec2);
vec2[0] = vec1[0] -= grid->dx; vec2[0] = vec1[0] -= grid->dx;
} }
} }
/* horizontal lines */ /* horizontal lines */
if (flag & V2D_HORIZONTAL_LINES) { if (flag & V2D_HORIZONTAL_LINES) {
/* only major gridlines */ /* only major gridlines */
vec1[1] = vec2[1] = grid->starty; vec1[1] = vec2[1] = grid->starty;
vec1[0] = grid->startx; vec1[0] = grid->startx;
vec2[0] = v2d->cur.xmax; vec2[0] = v2d->cur.xmax;
step = (BLI_rcti_size_y(&v2d->mask) + 1) / (U.v2d_min_gridsize * UI_DPI_FAC); step = horizontal_major_step;
UI_GetThemeColor3ubv(TH_GRID, draw_color);
immAttrib3ubv(color, draw_color);
UI_ThemeColor(TH_GRID);
for (a = 0; a <= step; a++) { for (a = 0; a <= step; a++) {
glVertex2fv(vec1); immVertex2fv(pos, vec1);
glVertex2fv(vec2); immVertex2fv(pos, vec2);
vec2[1] = vec1[1] += grid->dy; vec2[1] = vec1[1] += grid->dy;
} }
/* fine grid lines */ /* fine grid lines */
vec2[1] = vec1[1] -= 0.5f * grid->dy; vec2[1] = vec1[1] -= 0.5f * grid->dy;
step++; step++;
if (flag & V2D_HORIZONTAL_FINELINES) { if (flag & V2D_HORIZONTAL_FINELINES) {
UI_ThemeColorShade(TH_GRID, 16); UI_GetThemeColorShade3ubv(TH_GRID, 16, draw_color);
immAttrib3ubv(color, draw_color);
for (a = 0; a < step; a++) { for (a = 0; a < step; a++) {
glVertex2fv(vec1); immVertex2fv(pos, vec1);
glVertex2fv(vec2); immVertex2fv(pos, vec2);
vec2[1] = vec1[1] -= grid->dy; vec2[1] = vec1[1] -= grid->dy;
} }
} }
} }
/* Axes are drawn as darker lines */ /* Axes are drawn as darker lines */
UI_ThemeColorShade(TH_GRID, -50); UI_GetThemeColorShade3ubv(TH_GRID, -50, draw_color);
immAttrib3ubv(color, draw_color);
/* horizontal axis */ /* horizontal axis */
if (flag & V2D_HORIZONTAL_AXIS) { if (flag & V2D_HORIZONTAL_AXIS) {
vec1[0] = v2d->cur.xmin; vec1[0] = v2d->cur.xmin;
vec2[0] = v2d->cur.xmax; vec2[0] = v2d->cur.xmax;
vec1[1] = vec2[1] = 0.0f; vec1[1] = vec2[1] = 0.0f;
glVertex2fv(vec1); immVertex2fv(pos, vec1);
glVertex2fv(vec2); immVertex2fv(pos, vec2);
} }
/* vertical axis */ /* vertical axis */
if (flag & V2D_VERTICAL_AXIS) { if (flag & V2D_VERTICAL_AXIS) {
vec1[1] = v2d->cur.ymin; vec1[1] = v2d->cur.ymin;
vec2[1] = v2d->cur.ymax; vec2[1] = v2d->cur.ymax;
vec1[0] = vec2[0] = 0.0f; vec1[0] = vec2[0] = 0.0f;
glVertex2fv(vec1); immVertex2fv(pos, vec1);
glVertex2fv(vec2); immVertex2fv(pos, vec2);
} }
glEnd(); /* stop rendering */
immEnd();
immUnbindProgram();
} }
/* Draw a constant grid in given 2d-region */ /* Draw a constant grid in given 2d-region */
void UI_view2d_constant_grid_draw(View2D *v2d) void UI_view2d_constant_grid_draw(View2D *v2d)
{ {
float start_x, start_y, step = 25.0f; float start_x, start_y, step = 25.0f;
int count_x, count_y; int count_x, count_y;
▲ Show 20 Lines • Show All 1,063 Lines • Show Last 20 Lines