Re: patch: introduce simple actions

[Patrick McHardy <kaber@trash.net>]

Jamal Hadi Salim wrote:
> On Sun, 2005-03-20 at 14:22, Patrick McHardy wrote:
>
>>One of the next things I wanted to do in this area was
>>moving all the large inline functions to act_generic.c,
>>so if possible I would prefer not to put these in a header
>>file.
> 
> These are small functions. Take a look at the simple action i attached.
> If you can give me the same functionality and still move things
> de_inlined to act_generic.c i would be fine with it.

The functions are not inlined anyway, so the question is whether any
#define magic is done as with pkt_act.h, otherwise there is no reason
to have them in a header file. Looking at your patch, it seems
tc_defact.h uses the redefined pkt_act.h functions, so it does need to
be in a header file, but frankly, I find this horrible. The patch
doesn't conflict with cleanup of pkt_act.h, but it means more work when
doing it, so I think this is what should be done first. Especially
since this is only an example, not useful for users, but it will be
possibly used to create more actions that also need to be changed
afterwards.

Regards
Patrick