Initialise return values in error cases, too: core code is horrifically lax at checking errors from these APIs.

This commit is contained in:
John-Mark Bell 2013-01-26 22:04:26 +00:00
parent 30dfa09008
commit 23e16e020e

View File

@ -255,6 +255,7 @@ bool nsfont_width(const plot_font_style_t *fstyle,
else
LOG(("rufl_width: 0x%x", code));
/* warn_user("MiscError", "font error"); */
*width = 0;
return false;
}
@ -303,6 +304,8 @@ bool nsfont_position_in_string(const plot_font_style_t *fstyle,
else
LOG(("rufl_x_to_offset: 0x%x", code));
/* warn_user("MiscError", "font error"); */
*char_offset = 0;
*actual_x = 0;
return false;
}
@ -355,6 +358,8 @@ bool nsfont_split(const plot_font_style_t *fstyle,
else
LOG(("rufl_split: 0x%x", code));
/* warn_user("MiscError", "font error"); */
*char_offset = 0;
*actual_x = 0;
return false;
}
@ -386,6 +391,8 @@ bool nsfont_split(const plot_font_style_t *fstyle,
else
LOG(("rufl_width: 0x%x", code));
/* warn_user("MiscError", "font error"); */
*char_offset = 0;
*actual_x = 0;
return false;
}