On 5/12/15 00:01, Richard Henderson wrote: > On 05/10/2015 03:42 PM, Chen Gang wrote: >> -static __inline unsigned int >> +static inline uint8_t >> get_BFEnd_X0(tilegx_bundle_bits num) > > Do not change these casts to uint8_t. It's unnecessary churn. >
For me, it is enough to return uint8_t, and the caller really treats it as uint8_t. So for the function declaration, uint8_t is more precise than unsigned int for return type. Thanks. -- Chen Gang Open, share, and attitude like air, water, and life which God blessed