Re^3: Layer does not match this perl (sigh)

This patch now has a life at perl #73754

Update: The patch has been applied as http://perl5.git.perl.org/perl.git/commitdiff/0dc1749808f7658dd7e4af08e1854e0d622b105d.

Comment on Re^3: Layer does not match this perl (sigh)

Replies are listed 'Best First'.
Re^4: Layer does not match this perl (thanks) by tye (Sage) on Mar 22, 2010 at 21:16 UTC
Thanks very much for submitting that. I suggest the diagnostics be made more specific. Instead of: `+%s (%d) does not match %s (%d), +%s (%d) smaller than %s (%d),` [download] Use: `"PerlIO layer function table size (%d) does not match size expected by + this perl (%d)", "PerlIO layer instance size (%d) smaller than size expected by this pe +rl (%d)",` [download] But I also don't know if the format strings not matching the format strings used in the source code could present some type of problem. - tye	[reply] [d/l] [select]
Re^5: Layer does not match this perl (thanks) by Corion (Patriarch) on Mar 23, 2010 at 08:18 UTC
The old message wasn't properly detected by `perldiag.t` either, so either the extraction of (missing) messages from the source is faulty or I don't understand what `perldiag.t` is supposed to test. But changing the messages shouldn't affect anything. I'll send another patch unless somebody else sends it first (remember to mention the RT ticket number).	[reply] [d/l] [select]