String handling in debug_standardize_address and standardize_address

[pramsey]

The memory handling in debug_standardize_address is not super clean in the string building, it could stand to be cleaned up. In particular, don't do a bunch of length counting and addition and then palloc, just use StringInfo. Also don't use a bare sprintf, and rework that area also into using StringInfo.

[robe]

Can you take a look a this pull request when you have the chance ​https://git.osgeo.org/gitea/postgis/postgis/pulls/162

Feel free to criticize to your hearts content. One thing that is frustrating me is the tab/space mix. You think I should apply astyle to it? Haven't used that in a long time. Do we still use that?

[strk]

The astyle script created by make astyle checks astyle version and refuses to run if you don' thave 1.23 (as we had different results with different versions). It'd be hard to have that version nowadays so I dubt anyone is using that.

If you changed all lines I guess you could give it a consistent style with astyle (updating that astyle.sh script?). Just make sure .editorconfig will keep the choosen style afterwards.

[Regina Obe <lr@…>]

In 283a06c6/git:

Address standardizer cleanup

• Replace macro string with StringInfo

in debug_address_standardizer
and address_standardiser functions

References #5535 for PostGIS 3.5.0

[Regina Obe <lr@…>]

In 5e73e13e/git:

Address standardizer cleanup

• Replace macro string with StringInfo

in debug_address_standardizer
and address_standardiser functions

References #5535 for PostGIS 3.5.0

[Regina Obe <lr@…>]

In 48fb165a/git:

Address standardizer cleanup

• Replace macro string with StringInfo

in debug_address_standardizer
and address_standardiser functions

References #5535 for PostGIS 3.4.1