<div dir="ltr">Hi,<div><br></div><div>Not 100% sure about initialization (last hunk GetSharedModuleList()), but as I see it, it was racy, unless it is always protected by something in Target.</div><div><br></div><div>Other two are minor things, with ModuleList::operator= priority inversion probably never occurring in current code as is. However it might fight back in future, and since I've noticed it...</div>
<div><br></div><div>Please commit if OK for trunk.</div><div><br></div><div>Cheers,</div><div>/Piotr</div></div>