Re: 10% drop in code line count in PG 17

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Daniel Gustafsson <daniel(at)yesql(dot)se>
Cc: PostgreSQL-development <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: 10% drop in code line count in PG 17
Date: 2025-11-21 00:49:12
Message-ID: aR-3CMIPEDPncVig@momjian.us
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Nov 20, 2025 at 03:30:15PM -0500, Bruce Momjian wrote:
> On Thu, Nov 20, 2025 at 10:38:49AM +0100, Daniel Gustafsson wrote:
> > > On 19 Nov 2025, at 20:59, Bruce Momjian <bruce(at)momjian(dot)us> wrote:
> >
> > > While working on a talk, I studied the number of code line changes in
> > > each major release,
> >
> > This script will only pick up C, but will pick up C in src/test but not any
> > Perl code using the C modules in src/test etc. These days we also have C++ and
> > some Python in the tree. Maybe it's time to revise it for todays codebase
> > which is quite different from when it was written 20 years ago?
>
> Yeah, that's part of a larger discussion. In an email I just sent I
> suggested we are trying to count files that are part of a cluster
> install, rather than testing files, but again, needs discussion.

Actually, another discussion would be why we have src/tools/codelines in
the git tree at all. I added it in 2005 to use in counting code lines,
and I thought we could consider it our standard method, but I am not
sure anyone aside from me even uses it, and it is clear there are
multiple methods people consider valid. Should we just remove it?

--
Bruce Momjian <bruce(at)momjian(dot)us> https://momjian.us
EDB https://enterprisedb.com

Do not let urgent matters crowd out time for investment in the future.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Samuel Thibault 2025-11-21 00:54:29 Re: GNU/Hurd portability patches
Previous Message Andreas Karlsson 2025-11-21 00:44:31 Re: Changing the state of data checksums in a running cluster