On 01.10.26 18:44, Rui Zhao wrote:
One small thing in 0005, in src/backend/utils/adt/float.c: the comment
above float8in_internal() still says:
"num" could validly be declared "const char *", but that results in an
unreasonable amount of extra casting both here and in callers,
so we don't.
Could we remove those two lines now that num is const char *?
I have committed a few of these patches that did not receive any other
comments, and I will incorporate your feedback into the updated patch
series for the rest. Thanks.