[PATCH] D67277: [clangd] Replace HighlightingKind::NumKinds with LastKind. NFC

This revision was automatically updated to reflect the committed changes. Closed by commit rL371375: [clangd] Replace HighlightingKind::NumKinds with LastKind. NFC (authored by ibiryukov, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commi

[PATCH] D67277: [clangd] Replace HighlightingKind::NumKinds with LastKind. NFC

ilya-biryukov added a comment. In D67277#1662669 , @hokein wrote: > LGTM, though I think the current implementation is a common pattern. Thanks. It's indeed a common pattern. The problem is that it forces clients to handle this extra case in every swit

[PATCH] D67277: [clangd] Replace HighlightingKind::NumKinds with LastKind. NFC

hokein accepted this revision. hokein added a comment. This revision is now accepted and ready to land. LGTM, though I think the current implementation is a common pattern. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D67277/new/ https://reviews.ll

[PATCH] D67277: [clangd] Replace HighlightingKind::NumKinds with LastKind. NFC

ilya-biryukov created this revision. ilya-biryukov added a reviewer: hokein. Herald added subscribers: kadircet, arphaman, jkorous, MaskRay. Herald added a project: clang. The latter simplifies the client code by avoiding the need to handle it as a separate case statement. Repository: rG LLVM