Re: [PATCH] [CLEANUP] Wrap locally-used functions in anonymous namespaces

2014-04-16 Thread Richard Biener
On Tue, Apr 15, 2014 at 7:33 PM, Patrick Palka wrote: > On Tue, Apr 15, 2014 at 3:51 AM, Richard Biener > wrote: >> On Mon, Apr 14, 2014 at 4:51 PM, Patrick Palka wrote: >>> Hi everyone, >>> >>> This patch wraps a bunch of locally-used, non-debug functions in an >>> anonymous namespace. These f

Re: [PATCH] [CLEANUP] Wrap locally-used functions in anonymous namespaces

2014-04-15 Thread Patrick Palka
On Tue, Apr 15, 2014 at 3:51 AM, Richard Biener wrote: > On Mon, Apr 14, 2014 at 4:51 PM, Patrick Palka wrote: >> Hi everyone, >> >> This patch wraps a bunch of locally-used, non-debug functions in an >> anonymous namespace. These functions can't simply be marked as "static" >> because they are

Re: [PATCH] [CLEANUP] Wrap locally-used functions in anonymous namespaces

2014-04-15 Thread Richard Biener
On Mon, Apr 14, 2014 at 4:51 PM, Patrick Palka wrote: > Hi everyone, > > This patch wraps a bunch of locally-used, non-debug functions in an > anonymous namespace. These functions can't simply be marked as "static" > because they are used as template arguments to hash_table::traverse, and > the C