develooper Front page | perl.perl5.porters | Postings from June 2013

Re: [perl #118603] t/op/stash.t: 3 new test failures on Darwin/PPC

Thread Previous | Thread Next
From:
Craig A. Berry
Date:
June 23, 2013 03:31
Subject:
Re: [perl #118603] t/op/stash.t: 3 new test failures on Darwin/PPC
Message ID:
CA+vYcVyfkxuDMo-SaRO2YnS=U2PUdfFHrCJK7rA1CA5ggsNqLg@mail.gmail.com
On Sat, Jun 22, 2013 at 6:24 PM, Father Chrysostomos via RT
<perlbug-followup@perl.org> wrote:
> On Sat Jun 22 14:33:02 2013, nicholas wrote:

>> Pushed as origin/smoke-me/nicholas/RT-118603
>
> Wouldn’t a more robust approach like the attached be better?  (I have
> not actually tested my patch.)

The fact that it doesn't compile makes it a bit less robust, actually :-(.

Read <http://gcc.gnu.org/onlinedocs/cpp/If.html#If>, notably the bit
that says, "The preprocessor does not know anything about types in the
language. Therefore, sizeof operators are not recognized in ‘#if’."

And while defensive programming has its places, I'm not sure the size
of cv_flags_t is going to be system-dependent (given the general
obsolescence of <32-bit systems, or, in other words U32 being
available always and everywhere) or would change with any frequency.

Thread Previous | Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About