On 05/10/2017 05:05 AM, Richard Biener wrote:
On Tue, May 9, 2017 at 9:00 PM, Nathan Sidwell <nat...@acm.org> wrote:

+
+#define TDF_ADDRESS    (1 << 3)        /* dump node addresses */

this leaves 1 << 2 unused.

Yes, that was intentional (though I suspect my note about it was hidden). As you say, I expect further cleanup and didn't want gratuitous churn. I'll add a comment about bit 2 being free.

Otherwise looks like a great cleanup.  You might want to coordinate with
Martin a bit here.  It also looks like with this we can start re-using
bits when they are restricted to one TDF_KIND.

Indeed, we coordinated a bit yesterday.  Thanks for review!

nathan

--
Nathan Sidwell

Reply via email to