On Fri, Jan 3, 2020 at 2:23 AM Xiang Xiao <xiaoxiang781...@gmail.com> wrote:
> > I made this change just because the usage is inconsistent through out the > code base. the choice to remove the cast, not to add the cast, because: > 1.It is more easier to remove the cast by command > 2.This change is smaller than to add the cast > 3.It make the code a little bit clean(e.g. memcpy... vs. (void)memcpy...) > 4.The return value from many function don't indicate the pass/fail(e.g. > memcpy return destination), it is reasonable to ignore it silently if the > caller don't use that information. Thanks for explaining that. The coding standard has been updated to match. Nathan
