[Libre-soc-bugs] [Bug 1125] split instructions' pseudo-code into separate files so they can be [[!inline]]-ed into the wiki

[bugzilla-daemon at libre-soc.org]

https://bugs.libre-soc.org/show_bug.cgi?id=1125

--- Comment #5 from Jacob Lifshay <programmerjake at gmail.com> ---
(In reply to Luke Kenneth Casson Leighton from comment #4)
> (In reply to Jacob Lifshay from comment #0)
> > I changed the pseudo-code parser so it can parse [[!inline]] directives, and
> > demoed changing one instruction (minmax. in av.mdwn),
> 
> ah sorry i wasn't clear enough: i meant the *entire* instruction.

I thought about doing that, but decided against it because we often quote just
the pseudo-code on the wiki and because the way instructions' different parts
are organized in rfcs don't match how the parser likes them (e.g. we have the
instruction form as a textual table, as well as having the full description
text). if we do want the whole instruction split out per file, we still need
just the pseudo-code in a separate file for the above reasons.


> the way that you chose to do it was a lot more work, wasn't it?

yes, but I think it is worth it.
> 
> all you actually had to do by doing the entire instruction
> was look for "#" in the split-insns.sh file then add a second
> level of subdirectories, or read "av.mdwn" for example and
> notice "av/minmax" in it.
> 
> that's about... 15-20 minutes of work.
> 
> what you are proposing requires far more work as it involves
> massive intrusive changes to the instruction parser.

actually that part is rather minor since it only supports [[!inline]] when the
pseudocode is nothing but [[!inline]] (plus prefix comments)
> 
> which we DO NOT have time for and i do not want time or money
> wasted on.

well, i already completed all those changes...the simulator part took like
30min.

-- 
You are receiving this mail because:
You are on the CC list for the bug.