avoid two (out of three) gcc12 compiler warnings in PrLongLivedTracking

avoid moving temporary objects by creating them 'in situ' and removing if not OK instead of moving them to the right place when OK... (aside: moving an object containing a boost::small_vector is not really cheap and should be avoided if possible...)

@mveghel: FYI

added 1 commit

• 17c76c95 - avoid two gcc12 compiler warnings

Compare with previous version

changed the description

added RTA label

merged

mentioned in commit b7a96d91

@graven I've pushed merge by mistake here. Now having seen the diffs (and not finding where the revert button is), I'm inclined to keep it like this. Is it ok for you ?

I would say that the chance that this would turn out to be disruptive is 'very small' -- so I'd keep it as is, and only act if there is evidence that it doesn't quite do what I think it should do ;-)