mirror of
https://gitlab.freedesktop.org/cairo/cairo.git
synced 2026-05-07 16:48:12 +02:00
Remove some bogus XXX marks
These are all perfectly correct code. Most are simply there because when we support vertical text writing mode we need to update there, but that's pretty trivial. No special markers needed. /me is trying to make user-font clean of XXX and TODO marks
This commit is contained in:
parent
4e10241148
commit
e8e2386233
4 changed files with 5 additions and 6 deletions
|
|
@ -190,7 +190,6 @@ _cairo_arc_in_direction (cairo_t *cr,
|
|||
/* Recurse if drawing arc larger than pi */
|
||||
if (angle_max - angle_min > M_PI) {
|
||||
double angle_mid = angle_min + (angle_max - angle_min) / 2.0;
|
||||
/* XXX: Something tells me this block could be condensed. */
|
||||
if (dir == CAIRO_DIRECTION_FORWARD) {
|
||||
_cairo_arc_in_direction (cr, xc, yc, radius,
|
||||
angle_min, angle_mid,
|
||||
|
|
|
|||
|
|
@ -601,7 +601,7 @@ _compute_transform (cairo_ft_font_transform_t *sf,
|
|||
|
||||
status = _cairo_matrix_compute_scale_factors (scale,
|
||||
&x_scale, &y_scale,
|
||||
/* XXX */ 1);
|
||||
1);
|
||||
if (status)
|
||||
return status;
|
||||
|
||||
|
|
|
|||
|
|
@ -675,7 +675,7 @@ _cairo_scaled_font_set_metrics (cairo_scaled_font_t *scaled_font,
|
|||
|
||||
status = _cairo_matrix_compute_scale_factors (&scaled_font->font_matrix,
|
||||
&font_scale_x, &font_scale_y,
|
||||
/* XXX */ 1);
|
||||
1);
|
||||
if (status)
|
||||
return status;
|
||||
|
||||
|
|
@ -1957,7 +1957,7 @@ _cairo_scaled_glyph_lookup (cairo_scaled_font_t *scaled_font,
|
|||
}
|
||||
|
||||
_cairo_scaled_glyph_set_index(scaled_glyph, index);
|
||||
scaled_glyph->cache_entry.size = 1; /* XXX */
|
||||
scaled_glyph->cache_entry.size = 1; /* We currently don't differentiate on glyph size at all */
|
||||
scaled_glyph->scaled_font = scaled_font;
|
||||
scaled_glyph->surface = NULL;
|
||||
scaled_glyph->path = NULL;
|
||||
|
|
|
|||
|
|
@ -127,7 +127,7 @@ _cairo_user_scaled_glyph_init (void *abstract_font,
|
|||
* representation. */
|
||||
|
||||
matrix = scaled_font->base.scale_inverse;
|
||||
status = _cairo_matrix_compute_scale_factors (&matrix, &x_scale, &y_scale, /* XXX */ 1);
|
||||
status = _cairo_matrix_compute_scale_factors (&matrix, &x_scale, &y_scale, 1);
|
||||
if (status)
|
||||
return status;
|
||||
|
||||
|
|
@ -356,7 +356,7 @@ _cairo_user_font_face_scaled_font_create (void *abstract_
|
|||
user_scaled_font->default_glyph_extents.width = 0.;
|
||||
user_scaled_font->default_glyph_extents.height = font_extents.ascent + font_extents.descent;
|
||||
user_scaled_font->default_glyph_extents.x_advance = font_extents.max_x_advance;
|
||||
user_scaled_font->default_glyph_extents.y_advance = 0.; /* XXX */
|
||||
user_scaled_font->default_glyph_extents.y_advance = 0.;
|
||||
|
||||
*scaled_font = &user_scaled_font->base;
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue