summaryrefslogtreecommitdiff
path: root/src/cairo-scaled-font-subsets.c
diff options
context:
space:
mode:
authorUli Schlachter <psychon@znc.in>2012-05-01 14:48:43 +0200
committerUli Schlachter <psychon@znc.in>2012-05-05 10:14:54 +0200
commit138e595c1192c03ebc1b2c89851ce938b18abd2f (patch)
treee03e3a0eef36dd8421c08a9b5d58ecb1acf2813d /src/cairo-scaled-font-subsets.c
parent494cfd7eb9d994a5e2024e299986e66a3ef9a562 (diff)
downloadcairo-138e595c1192c03ebc1b2c89851ce938b18abd2f.tar.gz
Remove some dead code
This code could never be executed, because the status variable doesn't get set since the last time it is checked and errors get returned to the caller. This was noticed while looking into the build log provided by http://lists.cairographics.org/archives/cairo/2012-April/022993.html Reported-by: Ryan Schmidt <cairo-2012b@ryandesign.com> Signed-off-by: Uli Schlachter <psychon@znc.in>
Diffstat (limited to 'src/cairo-scaled-font-subsets.c')
-rw-r--r--src/cairo-scaled-font-subsets.c5
1 files changed, 0 insertions, 5 deletions
diff --git a/src/cairo-scaled-font-subsets.c b/src/cairo-scaled-font-subsets.c
index 355245a47..e78e0c283 100644
--- a/src/cairo-scaled-font-subsets.c
+++ b/src/cairo-scaled-font-subsets.c
@@ -522,11 +522,6 @@ _cairo_sub_font_add_glyph (cairo_sub_font_t *sub_font,
y_advance = scaled_glyph->metrics.y_advance;
_cairo_scaled_font_thaw_cache (sub_font->scaled_font);
- if (unlikely (status)) {
- _cairo_sub_font_glyph_destroy (sub_font_glyph);
- return status;
- }
-
if (!is_latin && sub_font->num_glyphs_in_current_subset == sub_font->max_glyphs_per_subset)
{
sub_font->current_subset++;