Changes between Version 7 and Version 8 of ItsABughunt

Show
Ignore:
Timestamp:
02/05/10 19:25:51 (4 years ago)
Author:
cotto
Comment:

some initial comments on freeze/thaw and packfile

Legend:

Unmodified
Added
Removed
Modified
  • ItsABughunt

    v7 v8  
    88These subsystems need rethinking and revision and refactoring, because: 
    99 
     10 * freeze/thaw and packfile 
     11   * The freeze/thaw and packfile code do similar but slightly different things.  They should be unified to cut down on duplication. 
     12   * In a packfile, a sub's bytecode is stored in the bytecode segment with a separate fixup segment indication which chunk of bytecode belongs to which sub. 
     13     - Storing a sub's bytecode as its data in the constant PMC segment (i.e. having a sub carry its code with it) would greatly simplify much of the code that deals with packfiles. 
     14 
    1015 * hashes 
    11    * the Hash PMC is too entertwined with the guts of src/hash.c which knows too much about the internals of Hash PMCs 
     16   * the Hash PMC is too intertwined with the guts of src/hash.c 
    1217   * it's not clear if the hash/bucket algorithm is sufficiently collision free 
    1318   * OrderedHash is a mess of semantics and implementation 
     19     * Question: Does this still apply since the OrderedHash revamp branch was merged? --cotto 
    1420   * some Parrot systems need C-like hashes while user-visible parts of Parrot need PMC-ish hashes 
    1521