The compiler is unable to check that this switch is exhaustive in reasonable time
zhanleewo opened this issue · 3 comments
zhanleewo commented
dzenbot commented
You’re right. Both of those var
need a default: return “”
. This may never be hit, but it will trick the compiler at least.
Should be super easy to add, in the following lines:
https://github.com/home-assistant/Iconic/blob/master/Source/iconic-default.stencil#L224
https://github.com/home-assistant/Iconic/blob/master/Source/iconic-default.stencil#L233
Want to give it shot @zhanleewo?
zhanleewo commented
Yes, I have done this. Since I think you will solve it in the future, so I did not provide a patch.
mitchins commented
Hi, this is still open... can we make it happen?
I see no risk in using a default in any of the auto generated code because we can trust it is exhaustive as it is automatically done.