[Linaro-mm-sig] Re: [PATCH v10 4/5] rust: drm: gem: Introduce shmem::SGTable

Just so it doesn't come as a surprise in the next respin - I don't really see a reasonable way of using get_or_insert_with() here (though I had no idea this was a thing!). However - I can get rid of all of the ret = statements and also clean up the indenting a little bit with block expressions as I forgot those exist.

On Fri, 2026-04-10 at 16:55 +0900, Alexandre Courbot wrote:

nit: let's use `let ret = if let Some(sgt_res) ...` to avoid the multiple `ret = `statements?

Or maybe even better, you might be able to use `Option::get_or_insert_with`.