On Sat, Feb 24, 2024 at 5:21 AM Tristen Raab <tristen.r...@highgo.ca> wrote:
>
> I've reviewed your patch, it applies correctly and the documentation builds 
> without any errors. As for the content of the patch itself, I think so far 
> it's good but would make two modifications. I like how the patch was worded 
> originally when referring to the subscription, stating these parameters were 
> 'in' the subscription rather than 'by' it. So I'd propose changing
>
> > parameters specified by the subscription. When creating the slot, ensure
>
> to
>
> > parameters specified in the subscription. When creating the slot, ensure
>

This suggestion makes sense, so I updated the patch for this and committed.

-- 
With Regards,
Amit Kapila.


Reply via email to