diff options
| author | Peter Jones <pjones@redhat.com> | 2015-06-16 11:41:32 -0400 |
|---|---|---|
| committer | Peter Jones <pjones@redhat.com> | 2015-06-16 11:41:32 -0400 |
| commit | 1fefa29c34b8177779a237e26984ae973adf23e9 (patch) | |
| tree | cc1b0ab7bfa78a25e4fb7feb6cfbf60d9e50ab2e /include/console.h | |
| parent | 0a7003faec146ff1db829cc2d896ab093da7def0 (diff) | |
| download | efi-boot-shim-1fefa29c34b8177779a237e26984ae973adf23e9.tar.gz efi-boot-shim-1fefa29c34b8177779a237e26984ae973adf23e9.zip | |
Fix console_print_box*() parameters.
When we made lib build with the correct CFLAGS, it inherited
-Werror=sign-compare, and I fixed up some parameters on
console_print_box() and console_print_box_at() to avoid sign comparison
errors.
The fixups were *completely wrong*, as some behavior relies on negative
values. So this fixes them in a completely different way, by casting
appropriately to signed types where we're doing comparisons.
Signed-off-by: Peter Jones <pjones@redhat.com>
Diffstat (limited to 'include/console.h')
| -rw-r--r-- | include/console.h | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/include/console.h b/include/console.h index ecaa73f5..d59e3b4f 100644 --- a/include/console.h +++ b/include/console.h @@ -4,12 +4,12 @@ EFI_STATUS console_get_keystroke(EFI_INPUT_KEY *key); void -console_print_box_at(CHAR16 *str_arr[], unsigned int highlight, - unsigned int start_col, unsigned int start_row, - unsigned int size_cols, unsigned int size_rows, - int offset, unsigned int lines); +console_print_box_at(CHAR16 *str_arr[], int highlight, + int start_col, int start_row, + int size_cols, int size_rows, + int offset, int lines); void -console_print_box(CHAR16 *str_arr[], unsigned int highlight); +console_print_box(CHAR16 *str_arr[], int highlight); int console_yes_no(CHAR16 *str_arr[]); int |