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

liujinyang Wed, 14 Jan 2026 23:59:20 -0800

Hi Alvora,&nbsp;
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 <[email protected]&gt;
发件时间：2026年1月14日 01:18
收件人：liujinyang <[email protected]&gt;
抄送：pgsql-hackers <[email protected]&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/[email protected]&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/

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

Reply via email to