On 2026-09-08T17:50:33+08:00, Hao Ge <[email protected]> wrote: > On 2026/9/8 03:42, Daniel Gomez wrote:
> I skipped it because alloc_tag is the only codetag type and it > already cleans up after itself. > > With a second codetag type the problem Sashiko reported could indeed > be triggered. > > My thinking at the time was to do it in codetag itself, as a separate patch, > since it feels odd to unload something whose load just failed. Then, why returning an error at all? I don't see why not doing it inside codetag itself wouldn't work but it's a "bigger" change. IMO, the error is supposed to be handled at load_module() then, the unwinding as the rest.
