Re: [PATCH 2/2] rust_binder: account for deferred work size in Thread::read()

From: Chris Wiseman

Date: Mon Nov 09 2026 - 17:50:50 EST


On Mon, Nov 09, 2026 at 11:35:28PM +0100, Alice Ryhl wrote:
> On Sun, Nov 8, 2026 at 4:11 PM Chris Wiseman <chriswiseman@xxxxxxxxxx> wrote:
> > + deferred_size = deferred_size.saturating_sub(work.size());
>
> If deferred_size is zero, then this evaluates to zero. It'd be ideal
> to avoid calling size() here since it's a dynamic call.

Good point, no need to pay for the dynamic call when deferred_size is
already zero. I will guard it with if deferred_size != 0 in v2.

Thanks,
Chris