On Tue, Sep 27, 2016 at 5:58 PM, Duy Nguyen <pclouds@xxxxxxxxx> wrote: >> but then you also have to change the type of xdl_opts >> to uint64_t, which in turn means that you will have to change the >> definition of xpparam_t's "flags" field from unsigned long to uint64_t. > > I miss a connection here. This new flag is intended to be used in > "flags" field in struct diff_options. Is there any chance it can be > set on xdl_opts (of the same struct, I assume)? > >> Maybe this can be avoided? > > I don't see a good way to avoid it. We normally enable or disable diff > features as bit flags and now we run out of bits. Adding something > like "flags2" works, but not pretty. Any suggestion is welcome. Never mind. I think I found some way that does not look particularly bad. -- Duy