[Documentation Bug] Tap dance example 5, missing cases on switch

[cobac]

Hey,

I've implemented the behaviour explained in the 5th example of the tap dance section, but the compiler threw the following errors:

error: enumeration value 'TD_NONE' not handled in switch
error: enumeration value 'TD_UNKNOWN' not handled in switch

I've fixed it adding

...	
			
case TD_NONE:
    break;
case TD_UNKNOWN:
    break;
	}
}

to both functions with a case switch.

qmk_firmware/docs/feature_tap_dance.md

Line 384 in 2353232

void altlp_finished(qk_tap_dance_state_t *state, void *user_data) {

qmk_firmware/docs/feature_tap_dance.md

Line 399 in 2353232

void altlp_reset(qk_tap_dance_state_t *state, void *user_data) {

I'm not submitting a PR because I can't write C and I don't know if my fix is a good solution.

Thanks for maintaining this package!

[ForsakenRei]

I think you can add

default:
    break;

instead of those two cases. I didn't try it for this example but I think I encountered somthing similar before. For the reset section, I didn't use the defult case and it works fine. I checked my keymap and it actually has the default case in both td_finished and td_reset section.

[ForsakenRei]

After checking the doc, in example 6 there is another solution:
ql_tap_state.state = TD_NONE;
I'm not sure what's the difference XD but it should also work.

[joukewitteveen]

I believe this can be closed given that #14893 is merged.