On Tue, May 13, 2008 at 9:17 PM, via RT Geoffrey Broadwell <[EMAIL PROTECTED]> wrote: > # New Ticket Created by Geoffrey Broadwell > # Please include the string: [perl #54148] > # in the subject line of all future correspondence about this issue. > # <URL: http://rt.perl.org/rt3/Ticket/Display.html?id=54148 > > > > The attached patch adds the new utility script tools/util/dump-pbc.pl, > which produces a (mostly) easy to read weave of PBC disassembly and PIR > source (from possibly multiple files). It also enables compile of the > 'disassemble' program in the Makefile. (All of the structure was there, > it just wasn't enabled.) > > > -'f > >
Nifty! Minor nits: - can we use _ instead of - in the filename? - can we make the error message more graceful if the disassemble executable hasn't been built? - it's not always going to be called "disassemble" - you'll need to poke into Parrot::Config and pull out the executable suffix and use that. (Might be disassemble.exe on windows, for example). -- Will "Coke" Coleda
