Re: remove the unneeded header file math.h in binaryheap.c

Hi Alvora, 
thanks for your review.
So wha't the next should I do for commit it, because this is my first patch.

Best Regards
liujinyang

原始邮件


发件人：Álvaro Herrera <alvherre(at)kurilemu(dot)de&gt;
发件时间：2026年1月14日 01:18
收件人：liujinyang <21043272(at)qq(dot)com&gt;
抄送：pgsql-hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org&gt;
主题：Re: remove the unneeded header file math.h in binaryheap.c

On&nbsp;2026-Jan-13,&nbsp;liujinyang&nbsp;wrote:

&gt;&nbsp;Hi&nbsp;Hackers,
&gt;&nbsp;
&gt;&nbsp;I&nbsp;found&nbsp;in&nbsp;file&nbsp;binaryheap.c，&nbsp;file&nbsp;"math.h"&nbsp;was&nbsp;included,&nbsp;but&nbsp;it&nbsp;is&nbsp;uncessary,&nbsp;so&nbsp;I&nbsp;removed&nbsp;it&amp;nbsp;
&gt;&nbsp;and&nbsp;filing&nbsp;a&nbsp;patch&nbsp;to&nbsp;address&nbsp;the&nbsp;issue.

Fun.&nbsp;&nbsp;This&nbsp;was&nbsp;already&nbsp;unnecessary&nbsp;at&nbsp;commit&nbsp;7a2fe9bd0371&nbsp;which
introduced&nbsp;the&nbsp;file.&nbsp;&nbsp;It&nbsp;seems&nbsp;Abhijit&nbsp;had&nbsp;that&nbsp;include&nbsp;in&nbsp;his&nbsp;first
version&nbsp;[1]&nbsp;because&nbsp;he&nbsp;was&nbsp;using&nbsp;floor().&nbsp;&nbsp;&nbsp;Robert&nbsp;rewrote&nbsp;it&nbsp;later&nbsp;[2]
and&nbsp;replaced&nbsp;that&nbsp;with&nbsp;straight&nbsp;arithmetic,&nbsp;making&nbsp;the&nbsp;include
unnecessary,&nbsp;but&nbsp;forgot&nbsp;to&nbsp;remove&nbsp;it.

[1]&nbsp;https://postgr.es/m/20121114131112.GA27771@toroid.org&nbsp;
[2]&nbsp;https://postgr.es/m/CA%2BTgmobvR7XW9fjj2RNY7sKK-VAG5nahfai_zV51rHVLDNvaBg%40mail.gmail.com

--&nbsp;
Álvaro&nbsp;Herrera&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;48°01'N&nbsp;7°57'E&nbsp;&nbsp;—&nbsp;&nbsp;https://www.EnterpriseDB.com/