Re: Update EnableTimeoutParams timeout type comment

On Wed, Jul 1, 2026 at 11:41 AM Xuneng Zhou <xunengzhou@gmail.com> wrote:
> On Wed, Jul 1, 2026 at 12:01 PM John Naylor <johncnaylorls@gmail.com> 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