Re: [ovs-dev] [PATCH 11/16] hindex: remove the next variable in safe loops

2022-02-14 Thread 0-day Robot
Bleep bloop. Greetings Adrian Moreno, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: ERROR: Use ovs_assert() in place of assert() #122 FILE: tests/test-hindex.c:296: a

[ovs-dev] [PATCH 11/16] hindex: remove the next variable in safe loops

2022-02-14 Thread Adrian Moreno
Using SHORT version of the *_SAFE loops makes the code cleaner and less error prone. So, use the SHORT version and remove the extra variable when possible for HINDEX_*_SAFE. In order to be able to use both long and short versions without changing the name of the macro for all the clients, overload