Messages in this thread | | | From | Xiaomeng Tong <> | Subject | Re: [PATCH 2/6] list: add new MACROs to make iterator invisiable outside the loop | Date | Thu, 3 Mar 2022 11:31:22 +0800 |
| |
On Wed, 02 Mar 2022 08:02:23 -0500, James Bottomley <James.Bottomley@HansenPartnership.com> wrote: > pos shouldn't be an input to the macro since it's being declared inside > it. All that will do will set up confusion about the shadowing of pos. > The macro should still work as > > #define list_for_each_entry_inside(type, head, member) \ > ... > > For safety, you could > > #define POS __UNIQUE_ID(pos) > > and use POS as the loop variable .. you'll have to go through an > intermediate macro to get it to be stable. There are examples in > linux/rcupdate.h
The outer "pos" variable is no longer needed and thus the declare statement before the loop is removed, see the demostration in PATCH 3~6. Now, there is only one inner "pos" variable left. Thus, there should be no such *shadow* problem.
Please remind me if i missed something, thanks.
-- Xiaomeng Tong
| |