openrw icon indicating copy to clipboard operation
openrw copied to clipboard

Opcode documentation convention

Open ghost opened this issue 6 years ago • 4 comments

Recently, my PRs were being reviewed. They contained some changes to opcode documentation. So that brought up the discussion in irc. This is why I want to hear people thoughts about current situation and how this should be handled. Currently what we have is very weird formatting(in my opinion) and not very descriptive information about arguments or even opcode itself, broken tabulation, missing names (0412 for example), etc. There is been a leak of original names in GTA:SA mobile version, but @JayFoxRox brought a good point that they shouldn't be used due to legal stuff. Maybe they can be changed in some way, so it would be fine.

ghost avatar May 05 '18 15:05 ghost

Maybe something like this: https://gist.github.com/ShFil119/646e8e00b693070806ce32e0eb9119c0

With some parser it can generate quite good documentation.

ghost avatar May 12 '18 20:05 ghost

How about Doxygen?

Lihis avatar May 12 '18 20:05 Lihis

Can you elaborate @ShFil119, like demonstrate an opcode with that formatting that example isn't very clear to me. Also it seems to be missing args.

ghost avatar May 13 '18 07:05 ghost

I would like to have inline comments alongside the opcode. Using Doxygen we could generate the opcode documentation without much trouble.

danhedron avatar Jun 21 '18 21:06 danhedron