diff options
author | Tom Lane | 2025-07-12 15:50:35 +0000 |
---|---|---|
committer | Tom Lane | 2025-07-12 15:50:37 +0000 |
commit | 84ce2587075c9418deda6a2f04b9768e375ebe43 (patch) | |
tree | 63ba4f32d04b2f535f435e60492a5414dae0bd30 /src/interfaces/ecpg/preproc/nls.mk | |
parent | f2c87ac04e73511a19dc36343f102eed3332573c (diff) |
The values of the "result" variables in these functions are
always integers; using a float8 variable accomplishes nothing
except to incur useless conversions to and from float. While
that wastes a few nanoseconds, these functions aren't all that
time-critical. But it seems worth fixing to remove possible
reader confusion.
Also, in the case of date2isoyear(), "result" is a very poorly
chosen variable name because it is *not* the function's result.
Rename it to "week", and do the same in date2isoweek() for
consistency.
Since this is mostly cosmetic, there seems little need
for back-patch.
Author: Sergey Fukanchik <[email protected]>
Reviewed-by: Tom Lane <[email protected]>
Discussion: https://postgr.es/m/6323a-68726500-1-7def9d00@137821581
Diffstat (limited to 'src/interfaces/ecpg/preproc/nls.mk')
0 files changed, 0 insertions, 0 deletions