<div dir="ltr"><div dir="ltr">The GEOS codebase has several typedefs called GeometryPtr and GeomPtr.  They are aliases for a mixture of Geometry * and std::unique_Ptr<Geometry>. I'm thinking it's a good thing to purge them from the codebase in favour of explicit type signatures, because:</div><div dir="ltr"><br></div><div dir="ltr">- it's highly confusing that they can be aliases for different things</div><div>- it's complicated that they are defined in multiple places</div><div dir="ltr">- it can be hard to see what they represent</div><div dir="ltr">- they don't save much typing in the case of Geometry*, and using auto can simplify other uses</div><div dir="ltr"><div><br></div><div>This was discussed a while back.  The general feeling was that it's better to use explicit types.  Is this still the consensus?</div><div><br></div><div>Some notes on the process:</div><div>- I expect this will be done in multiple passes, to fit work schedules.  Will start with the least intrusive changes</div><div>- when this is done, it needs to be backported so that subsequent backports don't collide with unchanged code.</div><div>- If older versions use the aliases in dead code, it may not be removed, just patched to ensure it still compiles (i.e. by leaving the typedefs in place)</div><div> </div><div><br></div><div><br></div><div><div><br></div></div></div>
</div>