1) I had to change your code a bit, because it otherwise gave a strange
error message ([error] Constant name 'HASH(...)' has invalid characters). I
changed the:

use constant ({ INC_NOTHING => '-nothing-',
   INC_REGFILES => 'RegFiles',
})

Into:

use constant INC_NOTHING => '-nothing-';
use constant INC_REGFILES => 'RegFiles';

And then it worked fine.

2) You should probably recognize a sequence of retlw statements as a table,
the way it looks now, with a comment before each retlw, makes it difficult
to see what is actually happening.

3) Similar with a series of gotos: that would be a jumptable. You could even
change the labels of that to read "jumptable_<>" if you want to.

In general, I think it looks good... Not that my code becomes more readable
with it, but it seems to do the job of disassembling.

Greetings,
Maarten Hofman.
-- 
http://www.piclist.com PIC/SX FAQ & list archive
View/change your membership options at
http://mailman.mit.edu/mailman/listinfo/piclist