Re^4: Layer does not match this perl (thanks)

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

Comment on Re^4: Layer does not match this perl (thanks) Select or Download Code

Replies are listed 'Best First'.
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]