On Tue, Nov 12, 2019 at 02:17:35PM +0000, Nicholas Johnson wrote: > > > if (!b_res) > > > return; > > > > I think it may be good to comment here that skip the resources that are > > assigned (->parent != NULL). > > Something like > > /* If resource is already assigned, nothing more to do. */ This one looks good to me.