Thread (19 messages) 19 messages, 3 authors, 2022-01-12

Re: [PATCH v7 4/8] worktree: simplify find_shared_symref() memory ownership model

flat view

From: Anders Kaseorg <hidden>
Date: 2021-12-01 23:48:14

On 12/1/21 15:10, Eric Sunshine wrote:
As far as I can see, this code only cares whether find_shared_symref()
returned a result; it doesn't actually consult the returned worktree
at all, thus it semantically considers `worktree` as a boolean, not as
a `struct worktree`.
No, the update_worktree(new_oid->hash, worktree) call uses it in a non-boolean way, so we do need to keep it around.

Anders
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help