The patch has been applied and modified as Sean suggested.

--Matt


On Jan 28, 2013, at 5:02 PM, Matt Leotta <matt.leotta@kitware.com> wrote:

Sean,

I will make your suggested changes before applying.  That makes sense to me.  It should not matter if it is Apple or not, only if it is Intel or not, and PPC on Mac OS X is not an exception to the rule because PPC is not Intel.

Thanks,
Matt


On Mon, Jan 28, 2013 at 3:57 PM, Sean McBride <sean@rogue-research.com> wrote:
On Mon, 28 Jan 2013 15:49:53 -0500, Matt Leotta said:

>The patch looks good to me.  However, it touches both vnl and vil, so I'd
>like to give Gehua, Peter, and Ian a chance to comment before I apply it.
> They are the lead maintainers of vnl and vil.  If I don't hear anything by
>tomorrow I will apply the patch.

OK.  The patch isn't quite perfect... I'd remove:

1) the remaining "&& (!defined(__APPLE__)  || !defined(__ppc__)"

2) the two "with the following exception: PPC with Mac OS X" comments

Cheers,

--
____________________________________________________________
Sean McBride, B. Eng                 sean@rogue-research.com
Rogue Research                        www.rogue-research.com
Mac Software Developer              Montréal, Québec, Canada