Opened 6 years ago

Closed 6 years ago

#4022 closed enhancement (fixed)

Drop cstring2text and text2cstring in favor of Postgresql functions

Reported by: Algunenano Owned by: pramsey
Priority: medium Milestone: PostGIS 2.5.0
Component: postgis Version: master
Keywords: Cc:

Description

To avoid future issues (as discussed in ​https://github.com/postgis/postgis/pull/211) I've changed all calls of cstring2text / text2cstring for cstring_to_text / text_to_cstring from utils/builtins.h.

As lwgeom_in_geojson.c can't include both json-c/json.h and utils/builtins.h I've moved the old declarations there and left a warning since I'm pretty sure that error will come up in the future.

PR: ​https://github.com/postgis/postgis/pull/213

Change History (1)

comment:1 by komzpa, 6 years ago

Resolution: → fixed
Status: new → closed

In 16390:

Drop cstring2text/text2cstring in favor of PostgreSQL functions

Patch by Raúl Marín Rodríguez

Closes ​https://github.com/postgis/postgis/pull/213
Closes #4022

Note: See TracTickets for help on using tickets.