+-- On Thu, 21 Dec 2017, Jack Schwartz wrote --+ | Remove unused variables: mh_mode_type, mh_width, mh_height, mh_depth | | Signed-off-by: Jack Schwartz <jack.schwa...@oracle.com> | Reviewed-by: Daniel Kiper <daniel.ki...@oracle.com> | --- | hw/i386/multiboot.c | 6 ------ | 1 file changed, 6 deletions(-) | | diff --git a/hw/i386/multiboot.c b/hw/i386/multiboot.c | index ff2733d..964feaf 100644 | --- a/hw/i386/multiboot.c | +++ b/hw/i386/multiboot.c | @@ -255,12 +255,6 @@ int load_multiboot(FWCfgState *fw_cfg, | mb_kernel_size = mb_load_size; | } | | - /* Valid if mh_flags sets MULTIBOOT_HEADER_HAS_VBE. | - uint32_t mh_mode_type = ldl_p(header+i+32); | - uint32_t mh_width = ldl_p(header+i+36); | - uint32_t mh_height = ldl_p(header+i+40); | - uint32_t mh_depth = ldl_p(header+i+44); */ | -
+1 They are anyway comments. Reviewed-by: Prasad J Pandit <p...@fedoraproject.org> Thank you. -- Prasad J Pandit / Red Hat Product Security Team 47AF CE69 3A90 54AA 9045 1053 DD13 3D32 FE5B 041F