Re: [PATCH 1/3] Refactor entry point to -Wmisleading-indentation

2015-07-31 Thread Jeff Law
On 07/27/2015 05:12 PM, Patrick Palka wrote: The only question in my mind is bootstrap & regression testing. From reading the thread for the earlier version of this patch I got the impression you had bootstrapped and regression tested earlier versions. If you could confirm that you've bootstra

Re: [PATCH 1/3] Refactor entry point to -Wmisleading-indentation

2015-07-27 Thread Patrick Palka
On Tue, Jun 23, 2015 at 3:05 PM, Patrick Palka wrote: > On Mon, Jun 22, 2015 at 2:56 PM, Patrick Palka wrote: >> On Mon, Jun 22, 2015 at 1:29 PM, Jeff Law wrote: >>> On 06/09/2015 11:31 AM, Patrick Palka wrote: This patch refactors the entry point of -Wmisleading-indentation from:

Re: [PATCH 1/3] Refactor entry point to -Wmisleading-indentation

2015-06-23 Thread Patrick Palka
On Mon, Jun 22, 2015 at 2:56 PM, Patrick Palka wrote: > On Mon, Jun 22, 2015 at 1:29 PM, Jeff Law wrote: >> On 06/09/2015 11:31 AM, Patrick Palka wrote: >>> >>> This patch refactors the entry point of -Wmisleading-indentation from: >>> >>>void >>>warn_for_misleading_indentation (location_

Re: [PATCH 1/3] Refactor entry point to -Wmisleading-indentation

2015-06-22 Thread Patrick Palka
On Mon, Jun 22, 2015 at 1:29 PM, Jeff Law wrote: > On 06/09/2015 11:31 AM, Patrick Palka wrote: >> >> This patch refactors the entry point of -Wmisleading-indentation from: >> >>void >>warn_for_misleading_indentation (location_t guard_loc, >> location_t

Re: [PATCH 1/3] Refactor entry point to -Wmisleading-indentation

2015-06-22 Thread Jeff Law
On 06/09/2015 11:31 AM, Patrick Palka wrote: This patch refactors the entry point of -Wmisleading-indentation from: void warn_for_misleading_indentation (location_t guard_loc, location_t body_loc, location_t next_stmt_

Re: [PATCH 1/3] Refactor entry point to -Wmisleading-indentation

2015-06-22 Thread Jeff Law
On 06/18/2015 10:39 AM, David Malcolm wrote: On Thu, 2015-06-18 at 11:41 -0400, Patrick Palka wrote: On Tue, Jun 9, 2015 at 1:31 PM, Patrick Palka wrote: This patch refactors the entry point of -Wmisleading-indentation from: void warn_for_misleading_indentation (location_t guard_loc,

Re: [PATCH 1/3] Refactor entry point to -Wmisleading-indentation

2015-06-18 Thread Patrick Palka
On Thu, Jun 18, 2015 at 12:39 PM, David Malcolm wrote: > On Thu, 2015-06-18 at 11:41 -0400, Patrick Palka wrote: >> On Tue, Jun 9, 2015 at 1:31 PM, Patrick Palka wrote: >> > This patch refactors the entry point of -Wmisleading-indentation from: >> > >> > void >> > warn_for_misleading_indentat

Re: [PATCH 1/3] Refactor entry point to -Wmisleading-indentation

2015-06-18 Thread David Malcolm
On Thu, 2015-06-18 at 11:41 -0400, Patrick Palka wrote: > On Tue, Jun 9, 2015 at 1:31 PM, Patrick Palka wrote: > > This patch refactors the entry point of -Wmisleading-indentation from: > > > > void > > warn_for_misleading_indentation (location_t guard_loc, > >

Re: [PATCH 1/3] Refactor entry point to -Wmisleading-indentation

2015-06-18 Thread Patrick Palka
On Tue, Jun 9, 2015 at 1:31 PM, Patrick Palka wrote: > This patch refactors the entry point of -Wmisleading-indentation from: > > void > warn_for_misleading_indentation (location_t guard_loc, >location_t body_loc, >locatio

[PATCH 1/3] Refactor entry point to -Wmisleading-indentation

2015-06-09 Thread Patrick Palka
This patch refactors the entry point of -Wmisleading-indentation from: void warn_for_misleading_indentation (location_t guard_loc, location_t body_loc, location_t next_stmt_loc, enum cpp_tt