obsolete example

Started by Pavel Stehuleover 6 years ago6 messageshackers
Jump to latest
#1Pavel Stehule
pavel.stehule@gmail.com

Hi

isn't src/tutorial/func.c obsolete? There is not any benefit for users.

Regards

Pavel

In reply to: Pavel Stehule (#1)
Re: obsolete example

Em qui., 21 de nov. de 2019 às 15:59, Pavel Stehule
<pavel.stehule@gmail.com> escreveu:

isn't src/tutorial/func.c obsolete? There is not any benefit for users.

version-0 calling conventions were removed in v10. It seems an
oversight at commit 5ded4bd2140. Tutorial needs some care (I'm not
volunteering to improve it). I suggest unbreak the funcs module with
'mv funcs_new.c func.c'.

--
Euler Taveira Timbira -
http://www.timbira.com.br/
PostgreSQL: Consultoria, Desenvolvimento, Suporte 24x7 e Treinamento

#3Michael Paquier
michael@paquier.xyz
In reply to: Euler Taveira de Oliveira (#2)
Re: obsolete example

On Thu, Nov 21, 2019 at 08:19:56PM -0300, Euler Taveira wrote:

Em qui., 21 de nov. de 2019 às 15:59, Pavel Stehule
<pavel.stehule@gmail.com> escreveu:

isn't src/tutorial/func.c obsolete? There is not any benefit for users.

version-0 calling conventions were removed in v10. It seems an
oversight at commit 5ded4bd2140. Tutorial needs some care (I'm not
volunteering to improve it). I suggest unbreak the funcs module with
'mv funcs_new.c func.c'.

No objections from here, let's get rid of it. The docs actually make
use of the V1 versions, and funcs_new.c is not even compiled (it does
compile). Any objections to the attached? On top of moving the file,
there is one comment to update and a sentence to remove. Some
progress is always better than no progress.
--
Michael

Attachments:

tutorial-func-cleanup.patchtext/x-diff; charset=us-asciiDownload+61-175
#4Tom Lane
tgl@sss.pgh.pa.us
In reply to: Michael Paquier (#3)
Re: obsolete example

Michael Paquier <michael@paquier.xyz> writes:

No objections from here, let's get rid of it. The docs actually make
use of the V1 versions, and funcs_new.c is not even compiled (it does
compile). Any objections to the attached? On top of moving the file,
there is one comment to update and a sentence to remove. Some
progress is always better than no progress.

+1

regards, tom lane

#5Pavel Stehule
pavel.stehule@gmail.com
In reply to: Michael Paquier (#3)
Re: obsolete example

pá 22. 11. 2019 v 1:13 odesílatel Michael Paquier <michael@paquier.xyz>
napsal:

On Thu, Nov 21, 2019 at 08:19:56PM -0300, Euler Taveira wrote:

Em qui., 21 de nov. de 2019 às 15:59, Pavel Stehule
<pavel.stehule@gmail.com> escreveu:

isn't src/tutorial/func.c obsolete? There is not any benefit for users.

version-0 calling conventions were removed in v10. It seems an
oversight at commit 5ded4bd2140. Tutorial needs some care (I'm not
volunteering to improve it). I suggest unbreak the funcs module with
'mv funcs_new.c func.c'.

No objections from here, let's get rid of it. The docs actually make
use of the V1 versions, and funcs_new.c is not even compiled (it does
compile). Any objections to the attached? On top of moving the file,
there is one comment to update and a sentence to remove. Some
progress is always better than no progress.

+1

Pavel

Show quoted text

--
Michael

#6Michael Paquier
michael@paquier.xyz
In reply to: Pavel Stehule (#5)
Re: obsolete example

On Fri, Nov 22, 2019 at 06:11:32AM +0100, Pavel Stehule wrote:

+1

Okay, done. I have added a .gitignore while on it in the path for the
files generated.
--
Michael