Dan Hipschman wrote:
On Tue, Sep 18, 2007 at 11:15:26PM +0100, Robert Shearman wrote:
Sorry, I missed your last three patches somehow. My patch not only uses RPC_FC_POINTER instead of RPC_FC_LONG here, but it also implements the pointer description section for complex structures.
Right, but I was really talking about the approach of passing a flag into write_member_types or passing the containing type in. I think it is slightly cleaner to do the former, but your patch works very well for the case I was trying to fix.
Well, it's such a small section of code it probably doesn't matter much either way. I tend to avoid boolean arguments because when you look at the call to the function and just see a bunch of TRUEs and FALSEs, it's hard to remember what they mean.
Sure, it doesn't really matter and your set of patches implements more functionality. I'm happy to drop my patches in favour of yours.