On Wed, Jul 1, 2026 at 11:41 AM Xuneng Zhou <[email protected]> wrote: > On Wed, Jul 1, 2026 at 12:01 PM John Naylor <[email protected]> wrote: > > It seems pointless to have a comment whose entire content is to list > > every current enum member of that type. I'd just delete the comment, > > since it's likely to just going to get outdated again. If any other > > objects of type enum have this, they should get the same treatment. > > Agreed. The comment just duplicates nearby code and can easily go > stale again. Removing it also makes sense to me. Here's the patch with > the treatment.
My last sentence was meant to imply that I'd rather find and remove all such comments, rather than fix only one and leave the rest for someone else to stumble across. I had Claude whip up a throwaway script to find candidate useless comments, and didn't see any others like it, only those documenting a value that can have only a subset of the enum values. I've pushed your patch. -- John Naylor Amazon Web Services
