On Mon, 11 Sept 2023 at 08:51, Hongxu Ma <inte...@outlook.com> wrote: > > I created a patch to fix it. > Really appreciate to anyone can help to review it. > Thanks.
I think "product" as a variable name isn't very descriptive. Let's call it total_cells (or something similar instead). Other than that I think it's a good change. content->cellsadded is also a long, So I agree that I don't think the limit of int cells was intended here.