Opcode 0xF8 in Opcodes.json has an ambiguous "increment" flag

[VinceIP]

In Opcodes.JSON:
"0xF8": { "mnemonic": "LD", "bytes": 2, "cycles": [ 12 ], "operands": [ { "name": "HL", "immediate": true }, { "name": "SP", "increment": true, "immediate": true },

You can see that the second operand SP has the "increment" flag set to true, implying SP should be incremented after the LD operation (as in LD [HL+], A/LD A, [HL+]). This is incorrect because this opcode should leave SP unchanged.

[nitro2k01]

The json data is intended to be consumed by _includes/optables/mnemonic.html. It seems like this was likely originally intended to be a formatting kludge to insert a plus sign. Before whoever worked on it gave up and just hardcoded the opcode in question. So for that reason, I think it should be possible to just remove the increment property. At least assuming there isn't anything else that consumes the json file and relies on it being there. The really proper solution might be to introduce a new property for specifying that the second operand has a signed 8 bit immediate operand applied to it, which of course only ends up being used for this opcode. So it doesn't have to be hardcoded as a special case.

And anyway, the attributes for operands should be considered syntactical data for opcode presentation, not instruction behavior data.