fdo-mirrors/cairo
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/cairo/cairo.git synced 2026-05-07 08:38:02 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

path: Skip calls to zero-length memcpy

Browse source 
We attempt to copy 0 points onto the array of path points for a
close-path. This is pointless and an unnecessary function call under
MSVC at least.

Based on a patch by Steve Snyder, incorporating Behdad's review
comments.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=37836
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
This commit is contained in:
Chris Wilson 2011-09-16 16:11:42 +01:00
parent f09361cff1
commit 1ce5d4707c
1 changed files with 3 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/cairo-path-fixed.c
View file

@ -808,6 +808,9 @@ _cairo_path_buf_add_points (cairo_path_buf_t *buf,
const cairo_point_t *points, const cairo_point_t *points,
int num_points) int num_points)
{ {
if (buf->num_points == 0)
return;
memcpy (buf->points + buf->num_points, memcpy (buf->points + buf->num_points,
points, points,
sizeof (points[0]) * num_points); sizeof (points[0]) * num_points);