the function pointer is itself a function argument

[guy-maurel]

is corrected
fixes #3259

[alexhenrie]

This is an improvement, but it doesn't fix the problem completely. For example, sp_after_tparen_close=add transforms this input:

static int i2c_probe_address(struct i2c_adapter *adapter, int addr, int kind,
			     int (*found_proc1)(struct i2c_adapter *, int, int),
			     int (*found_proc2)(struct i2c_adapter *, int, int));

Into this output:

static int i2c_probe_address(struct i2c_adapter *adapter, int addr, int kind,
			     int (*found_proc1)(struct i2c_adapter *, int, int),
			     int (*found_proc2) (struct i2c_adapter *, int, int));

A space should be added after both (*found_proc1) and (*found_proc2), but it is only added to the second one.

[alexhenrie]

Thanks for fixing it! We should probably also update the test to have two function pointer arguments, like in my comment above.

[guy-maurel]

we have stil a problem with the test c:00420

[gmaurel]

is not ready to be merge,
see test c:00420