Re: [iproute PATCH v2 1/7] devlink: No need for this self-assignment

On Thu, Aug 17, 2017 at 09:48:50PM +0200, Jiri Pirko wrote:

Thu, Aug 17, 2017 at 07:09:25PM CEST, phil@nwl.cc wrote:

quoted

dl_argv_handle_both() will either assign to handle_bit or error out in
which case the variable is not used by the caller.

I'm pretty sure that I did this to silence the compiler. If the compiler
bug is fixed now, good.

That might depend on the compiler you used, so maybe you just want to
give it a try in your environment? If it still happens, we can keep this
self-assignment of course since it shouldn't harm.

Thanks, Phil