You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@pekko.apache.org by "mdedetrich (via GitHub)" <gi...@apache.org> on 2023/03/22 13:24:42 UTC
[GitHub] [incubator-pekko] mdedetrich commented on issue #265: Investigate fixing exhaustive match issues in Pekko codebase
mdedetrich commented on issue #265:
URL: https://github.com/apache/incubator-pekko/issues/265#issuecomment-1479563488
Sure we can do this, the point of making the issue is looking at it in more detail because there may be some legitimate situations where we should add the extra case and handle it rather than just doing `case _ => throw ...`).
I would also argue that if we are just going to add `case _ => throw new IllegalStateException()` in every single case (of which there are many) its then just better off to silence `strict-unsealed-patmat` just like what was done with `-infer-any`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org