Skip to content

Bracket limit and enhance run in db2 - #588

Open
buzzia2001 wants to merge 2 commits into
mainfrom
runsqlbracketlimit
Open

Bracket limit and enhance run in db2#588
buzzia2001 wants to merge 2 commits into
mainfrom
runsqlbracketlimit

Conversation

@buzzia2001

Copy link
Copy Markdown
Member

Changes

This PR adds a limit to the bracket matcher, set to 6000 by default. When that limit is exceeded, the mechanism will not work. Additionally, in SQLRPGLE source files that begin with **FREE, the RUN IN DB2 FOR I action will now also be present for statements other than SELECT, etc.

How to test

  1. Open a source file with more than 6000 rows or change Vscode-rpgle: Bracket Highlighting Max Lines
  2. Create a SQLRPGLE with this content, you'll see a lamp with RUN IN DB2 FOR I:
**FREE

EXEC SQL 
    SELECT * FROM DB.FILE;

EXEC SQL 
    VALUES 'DBFILE';

EXEC SQL 
    CALL DB.FILE();

EXEC SQL
    INSERT INTO DB.FILE
    VALUES('CIAODB');

EXEC SQL
    DELETE FROM DB.FILE;

EXEC SQL
    UPDATE DB.FILE;
image

Checklist

  • have tested my change
  • updated relevant documentation
  • Remove any/all console.logs I added
  • eslint is not complaining
  • have added myself to the contributors' list in the README

@buzzia2001
buzzia2001 requested a review from bobcozzi September 9, 2026 19:04
@buzzia2001 buzzia2001 self-assigned this Sep 9, 2026
@buzzia2001 buzzia2001 added the enhancement New feature or request label Sep 9, 2026
@buzzia2001 buzzia2001 changed the title add bracket limit and support for other sql commands for run in db2 f… Bracket limit and enhance run in db2 Sep 9, 2026
@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

👋 A new build is available for this PR based on 78d7088.

@buzzia2001

Copy link
Copy Markdown
Member Author

@bobcozzi any plan for this one?

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant