RE: [lustre-devel] [PATCH] Revert "Staging: lustre: o2iblnd: Use sizeof type *pointer instead of sizeof type."
From: Simmons, James A.
Date: Wed Mar 23 2016 - 16:30:00 EST
>> > > so the right code should be:
>> > >
>> > > sizeof(**net->ibn_tx_ps);
>> > > and the same for sizeof(**net->ibn_fmr_ps)
>> > That's a mess, isn't there some other way to fix this up to be more
>> > "obvious"?
>> This must have been encountered in the past. How was it handle in those
>> other cases?
>I fail to see why it's a mess. It's just **
>and someone making a mistake.
I have no trouble with **. If we revert it someone else will come along and
do the same mistake so I think we are stuck with the change to **.
>Removing the "typedef struct" uses from lustre
>would probably make a lot of this clearer though.
That along with a few hundred more patches heading Greg's way :-)