Netflix / dynomite

A generic dynamo implementation for different k-v storage engines

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Warning messages when building from source

WenningQiu opened this issue · comments

commented

When I build Dynomite from source on my RHEL 7 (see detail below) I am getting some warning messages that are concerning. I am attaching the entire make output if that helps.

My question is if there is any way to get a clean build, such as additional options/switches, an optimal platform on which to do the build or enhancing the source to suppress the warnings?

'''
qiuwen01-pa@accdevxacpx0001> uname -a
Linux accdevxacpx0001 3.10.0-1160.71.1.el7.x86_64 #1 SMP Wed Jun 15 08:55:08 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
[10:57:00][/nfs/home/qiuwen01-pa/dynomite]
'''

'''
dyn_crc32.c:129:3: warning: conversion to ‘int’ from ‘size_t’ may alter its value [-Wconversion]

dyn_jenkins.c:186:9: warning: conversion to ‘rstatus_t’ from ‘uint32_t’ may change the sign of the result [-Wsign-conversion]
return c; /* zero length strings require no mixing /
^
dyn_jenkins.c:188:9: warning: conversion to ‘rstatus_t’ from ‘uint32_t’ may change the sign of the result [-Wsign-conversion]
return c;
^
dyn_jenkins.c:246:9: warning: conversion to ‘rstatus_t’ from ‘uint32_t’ may change the sign of the result [-Wsign-conversion]
return c; /
zero length requires no mixing */
^
dyn_jenkins.c:248:9: warning: conversion to ‘rstatus_t’ from ‘uint32_t’ may change the sign of the result [-Wsign-conversion]
return c;
^

dyn_md5.c:260:24: warning: conversion to ‘unsigned char’ from ‘MD5_u32plus’ may alter its value [-Wconversion]
ctx->buffer[56] = ctx->lo;
^
dyn_md5.c:261:29: warning: conversion to ‘unsigned char’ from ‘MD5_u32plus’ may alter its value [-Wconversion]
ctx->buffer[57] = ctx->lo >> 8;
^
dyn_md5.c:262:29: warning: conversion to ‘unsigned char’ from ‘MD5_u32plus’ may alter its value [-Wconversion]
ctx->buffer[58] = ctx->lo >> 16;
^

../dyn_util.c:610:24: warning: conversion to ‘uint32_t’ from ‘long int’ may alter its value [-Wconversion]
return elem->tag_end - elem->tag_start;
^
../dyn_util.c: In function ‘argpos_elem_len’:
../dyn_util.c:615:20: warning: conversion to ‘uint32_t’ from ‘long int’ may alter its value [-Wconversion]
return elem->end - elem->start;
^
'''
make.output.gz