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.


Reply via email to