Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add GOTO command into PSQL [CORE1351] #1770

Closed
firebird-automations opened this issue Jul 10, 2007 · 8 comments
Closed

Add GOTO command into PSQL [CORE1351] #1770

firebird-automations opened this issue Jul 10, 2007 · 8 comments

Comments

@firebird-automations
Copy link
Collaborator

Submitted by: Martin Pavlik (quitec)

Add GOTO command to a procedural language.
I often need to execute the same code on more that one place inside a procedure.

example:

for select .... do
begin
....
goto label1;
label2:
...
end

label1:
...
goto label2:

Or even better add CALL and RETURN commands to allow execution of a subroutine inside a procedure.

@firebird-automations
Copy link
Collaborator Author

Modified by: Martin Pavlik (quitec)

summary: Add GOTO command in PSQL => Add GOTO command into PSQL

@firebird-automations
Copy link
Collaborator Author

Commented by: @dyemanov

Why don't you want to make the common code a separate procedure?

@firebird-automations
Copy link
Collaborator Author

Commented by: Martin Pavlik (quitec)

Because I do not want to pass and retrieve lots of local variables, let us say 30 or more.

@firebird-automations
Copy link
Collaborator Author

Modified by: @pcisar

Workflow: jira [ 12531 ] => Firebird [ 14144 ]

@firebird-automations
Copy link
Collaborator Author

Commented by: Fidel Viegas (araujofh)

From FB 2.1 onwards, you do have access to Global Temporary Tables, so I would suggest you use that to pass your data between the two procedures.

GO TO is really bad programming practice!

@firebird-automations
Copy link
Collaborator Author

Commented by: @asfernandes

I suggest close this ticket as "won't fix". Nobody wants goto.

@firebird-automations
Copy link
Collaborator Author

Modified by: @dyemanov

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Won't Fix [ 2 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @pcisar

status: Resolved [ 5 ] => Closed [ 6 ]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant