mirror of
https://gitlab.gnome.org/GNOME/gtk.git
synced 2024-12-27 22:20:24 +00:00
214f5a6f98
When we allocate a graphene_point_t on the stack, there's no guarantee that it will be aligned at an 8-byte boundary, which is an assumption made by gsk_pathop_encode() (which wants to use the lowest 3 bits to encode the operation). In the places where it matters, force the points on the stack and embedded in structs to be nicely aligned. By using a distinct type for this (a union with a suitable size and alignment), we ensure that the compiler will warn or error whenever we can't prove that a particular point is, in fact, suitably aligned. We can go from a `GskAlignedPoint *` to a `graphene_point_t *` (which is always valid, because the `GskAlignedPoint` is aligned) via &aligned_points[0].pt, but we cannot go back the other way (which is not always valid, because the `graphene_point_t` is not necessarily aligned nicely) without a cast. In practice, it seems that a graphene_point_t on x86_64 *is* usually placed at an 8-byte boundary, but this is not the case on 32-bit architectures or on s390x. In many cases we can avoid needing an explicit reference to the more complicated type by making use of a transparent union. There's already at least one transparent union in GSK's public API, so it's presumably portable enough to match GTK's requirements. Increasing the alignment of GskAlignedPoint also requires adjusting how a GskStandardContour is allocated and initialized. This data structure allocates extra memory to hold an array of GskAlignedPoint outside the bounds of the struct itself, and that array now needs to be aligned suitably. Previously the array started with at next byte after the flexible array of gskpathop, but the alignment of a gskpathop is only 4 bytes on 32-bit architectures, so depending on the number of gskpathop in the trailing flexible array, that pointer might be an unsuitable location to allocate a GskAlignedPoint. Resolves: https://gitlab.gnome.org/GNOME/gtk/-/issues/6395 Signed-off-by: Simon McVittie <smcv@debian.org>
205 lines
9.7 KiB
C
205 lines
9.7 KiB
C
/*
|
|
* Copyright © 2020 Benjamin Otte
|
|
*
|
|
* This library is free software; you can redistribute it and/or
|
|
* modify it under the terms of the GNU Lesser General Public
|
|
* License as published by the Free Software Foundation; either
|
|
* version 2.1 of the License, or (at your option) any later version.
|
|
*
|
|
* This library is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
|
* Lesser General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU Lesser General Public
|
|
* License along with this library. If not, see <http://www.gnu.org/licenses/>.
|
|
*
|
|
* Authors: Benjamin Otte <otte@gnome.org>
|
|
*/
|
|
|
|
|
|
#pragma once
|
|
|
|
#include "gskpathopprivate.h"
|
|
#include "gskpath.h"
|
|
#include "gskboundingboxprivate.h"
|
|
|
|
G_BEGIN_DECLS
|
|
|
|
typedef gpointer gskpathop;
|
|
|
|
typedef union _GskCurve GskCurve;
|
|
|
|
typedef struct _GskLineCurve GskLineCurve;
|
|
typedef struct _GskQuadCurve GskQuadCurve;
|
|
typedef struct _GskCubicCurve GskCubicCurve;
|
|
typedef struct _GskConicCurve GskConicCurve;
|
|
|
|
struct _GskLineCurve
|
|
{
|
|
GskPathOperation op;
|
|
|
|
gboolean padding;
|
|
|
|
union {
|
|
graphene_point_t points[2];
|
|
GskAlignedPoint aligned_points[2];
|
|
};
|
|
};
|
|
|
|
struct _GskQuadCurve
|
|
{
|
|
GskPathOperation op;
|
|
|
|
gboolean has_coefficients;
|
|
|
|
union {
|
|
graphene_point_t points[3];
|
|
GskAlignedPoint aligned_points[3];
|
|
};
|
|
|
|
graphene_point_t coeffs[3];
|
|
};
|
|
|
|
struct _GskCubicCurve
|
|
{
|
|
GskPathOperation op;
|
|
|
|
gboolean has_coefficients;
|
|
|
|
union {
|
|
graphene_point_t points[4];
|
|
GskAlignedPoint aligned_points[4];
|
|
};
|
|
|
|
graphene_point_t coeffs[4];
|
|
};
|
|
|
|
struct _GskConicCurve
|
|
{
|
|
GskPathOperation op;
|
|
|
|
gboolean has_coefficients;
|
|
|
|
/* points[0], points[1], points[3] are the control points,
|
|
* points[2].x is the weight
|
|
*/
|
|
union {
|
|
graphene_point_t points[4];
|
|
GskAlignedPoint aligned_points[4];
|
|
};
|
|
|
|
graphene_point_t num[3];
|
|
graphene_point_t denom[3];
|
|
};
|
|
|
|
union _GskCurve
|
|
{
|
|
GskPathOperation op;
|
|
GskLineCurve line;
|
|
GskQuadCurve quad;
|
|
GskCubicCurve cubic;
|
|
GskConicCurve conic;
|
|
};
|
|
|
|
typedef enum {
|
|
GSK_CURVE_LINE_REASON_STRAIGHT,
|
|
GSK_CURVE_LINE_REASON_SHORT
|
|
} GskCurveLineReason;
|
|
|
|
typedef gboolean (* GskCurveAddLineFunc) (const graphene_point_t *from,
|
|
const graphene_point_t *to,
|
|
float from_progress,
|
|
float to_progress,
|
|
GskCurveLineReason reason,
|
|
gpointer user_data);
|
|
|
|
typedef gboolean (* GskCurveAddCurveFunc) (GskPathOperation op,
|
|
const graphene_point_t *pts,
|
|
gsize n_pts,
|
|
float weight,
|
|
gpointer user_data);
|
|
|
|
void gsk_curve_init (GskCurve *curve,
|
|
gskpathop op);
|
|
void gsk_curve_init_foreach (GskCurve *curve,
|
|
GskPathOperation op,
|
|
const graphene_point_t *pts,
|
|
gsize n_pts,
|
|
float weight);
|
|
|
|
void gsk_curve_print (const GskCurve *curve,
|
|
GString *string);
|
|
|
|
char * gsk_curve_to_string (const GskCurve *curve);
|
|
|
|
gskpathop gsk_curve_pathop (const GskCurve *curve);
|
|
const graphene_point_t *gsk_curve_get_start_point (const GskCurve *curve);
|
|
const graphene_point_t *gsk_curve_get_end_point (const GskCurve *curve);
|
|
void gsk_curve_get_start_tangent (const GskCurve *curve,
|
|
graphene_vec2_t *tangent);
|
|
void gsk_curve_get_end_tangent (const GskCurve *curve,
|
|
graphene_vec2_t *tangent);
|
|
void gsk_curve_get_point (const GskCurve *curve,
|
|
float progress,
|
|
graphene_point_t *pos);
|
|
void gsk_curve_get_tangent (const GskCurve *curve,
|
|
float progress,
|
|
graphene_vec2_t *tangent);
|
|
void gsk_curve_reverse (const GskCurve *curve,
|
|
GskCurve *reverse);
|
|
void gsk_curve_split (const GskCurve *curve,
|
|
float progress,
|
|
GskCurve *start,
|
|
GskCurve *end);
|
|
void gsk_curve_segment (const GskCurve *curve,
|
|
float start,
|
|
float end,
|
|
GskCurve *segment);
|
|
gboolean gsk_curve_decompose (const GskCurve *curve,
|
|
float tolerance,
|
|
GskCurveAddLineFunc add_line_func,
|
|
gpointer user_data);
|
|
gboolean gsk_curve_decompose_curve (const GskCurve *curve,
|
|
GskPathForeachFlags flags,
|
|
float tolerance,
|
|
GskCurveAddCurveFunc add_curve_func,
|
|
gpointer user_data);
|
|
|
|
#define gsk_curve_builder_to(curve, builder) gsk_path_builder_pathop_to ((builder), gsk_curve_pathop (curve))
|
|
|
|
float gsk_curve_get_curvature (const GskCurve *curve,
|
|
float t,
|
|
graphene_point_t *center);
|
|
void gsk_curve_get_bounds (const GskCurve *curve,
|
|
GskBoundingBox *bounds);
|
|
void gsk_curve_get_tight_bounds (const GskCurve *curve,
|
|
GskBoundingBox *bounds);
|
|
|
|
void gsk_curve_get_derivative_at (const GskCurve *curve,
|
|
float t,
|
|
graphene_point_t *value);
|
|
int gsk_curve_get_crossing (const GskCurve *curve,
|
|
const graphene_point_t *point);
|
|
gboolean gsk_curve_get_closest_point (const GskCurve *curve,
|
|
const graphene_point_t *point,
|
|
float threshold,
|
|
float *out_dist,
|
|
float *out_t);
|
|
float gsk_curve_get_length (const GskCurve *curve);
|
|
float gsk_curve_get_length_to (const GskCurve *curve,
|
|
float t);
|
|
float gsk_curve_at_length (const GskCurve *curve,
|
|
float distance,
|
|
float epsilon);
|
|
|
|
int gsk_curve_get_curvature_points (const GskCurve *curve,
|
|
float t[3]);
|
|
|
|
int gsk_curve_get_cusps (const GskCurve *curve,
|
|
float t[2]);
|
|
|
|
|
|
G_END_DECLS
|
|
|