id	summary	reporter	owner	description	type	status	priority	milestone	component	version	severity	resolution	keywords	cc	lang	patch	platform
266	Packfile API cleanup: rename Parrot_readbc to Parrot_pbc_read, _loadbc => _pbc_load	rurban	jkeenan	"Attached is a patch which:

- renames Parrot_readbc to Parrot_pbc_read (as heard on IRC)[[BR]]
- renames Parrot_loadbc to Parrot_pbc_load[[BR]]
- adds a third debug argument to Parrot_pbc_read which is stored in 
  the returned pf->options member, to be able to react in 
  parrot and in other pbc utils differently. [[BR]]
  Used for debugging pbc transformers.[[BR]]
- macrofy TRACE_PACKFILE printing (debug levels 1, 2 and 3)

Rationale: Debugging the pbc reader requires rebuilding libparrot and pdump, but this affects parrot also. parrot should not debug-print pbc transformers while testing. 64-bit pbc unpacking has still broken alignments, 64bit BE and 12-byte floats are untested, so there's still a way to go with this debugging stuff.

Also included is the TT #258 TODO patch ""pcb_disassemble options -[bho?], enable roundtrips"". I can apply them seperately.

The patch needs a reviewer."	todo	closed	minor		core		low	fixed				applied