Re: [PATCH 2/7] [gdbsupport] Add parameterless iterator_range constructor

[Tom de Vries <tdevries@suse.de>]

On 4/23/26 4:34 PM, Tom Tromey wrote:
>>>>>> "Tom" == Tom de Vries <tdevries@suse.de> writes:
> 
> Tom> I noticed that there's no convenient way to create an empty range in
> Tom> iterator_range.  Add this.
>   
> Tom> +  /* Create an empty range.  */
> Tom> +  explicit iterator_range ()
> Tom> +    : iterator_range (IteratorType {}, IteratorType {})
> Tom> +  {}
> 
> I don't think the 'explicit' is needed here.
> 
> Also there's no real need to use constructor delegation here but OTOH it
> doesn't really matter I suppose.

Thanks for the review.

I ended up committing:
...
+  /* Create an empty range.  */
+  iterator_range () = default;
...

Thanks,
- Tom

> 
> Tom