Thomas Lockhart <lockhart(at)fourpalms(dot)org> writes:
> I've just committed changes which implement three SQL99 functions and
> operators.
I'm getting
gcc -O1 -Wall -Wmissing-prototypes -Wmissing-declarations -g -I../../../../src/include -c -o regexp.o regexp.c
regexp.c: In function `textregexsubstr':
regexp.c:314: warning: unused variable `result'
The code seems to be rather undecided about whether it intends to return
NULL or an empty string --- would you make up your mind and remove the
other case entirely?
regards, tom lane