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

hostvariable support for exec sql block #2310

Closed
ap891843 opened this issue May 24, 2024 · 2 comments
Closed

hostvariable support for exec sql block #2310

ap891843 opened this issue May 24, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@ap891843
Copy link
Contributor

ap891843 commented May 24, 2024

Description

Following program has multiple issue

Issues:

  • Hover and go to definition doesn't work for SET A_B_C = :QWE.D-E-F
  • commenting out variable definition for QWE, shows diagnostics at wrong location
       IDENTIFICATION DIVISION.
       PROGRAM-ID. hostvariales.
       ENVIRONMENT DIVISION.
       CONFIGURATION SECTION.
       SPECIAL-NAMES.
       INPUT-OUTPUT SECTION.
       FILE-CONTROL.
       DATA DIVISION.
        FILE SECTION.
        WORKING-STORAGE SECTION.
        01 ASD.
           05 O-P-R PIC X.
        01 QWE.
           05 D-E-F PIC X.
        LINKAGE SECTION.
       PROCEDURE DIVISION.
        MAIN-PROCESSING SECTION.
         MAINLINE-PARAGRAPH.
       EXEC SQL
        UPDATE QWE_STATUS
            SET A_B_C  = :QWE.D-E-F
            WHERE TKN_UNIQUE_REFERENCE = :ASD.O-P-R
       END-EXEC  
           STOP RUN.

Expected behavior

  • Hover and go to definition should work for host variables used in exec blocks
  • Diagnostics should be located at the error source

@ap891843 ap891843 added the bug Something isn't working label May 24, 2024
@VitGottwald
Copy link
Contributor

Do we have a PR fixing this @ap891843 ?

@ap891843
Copy link
Contributor Author

ap891843 commented Jun 5, 2024

Do we have a PR fixing this @ap891843 ?

We haven't started working on it yet

ap891843 added a commit to ap891843/che-che4z-lsp-for-cobol that referenced this issue Jul 18, 2024
ap891843 added a commit to ap891843/che-che4z-lsp-for-cobol that referenced this issue Jul 18, 2024
ap891843 added a commit to ap891843/che-che4z-lsp-for-cobol that referenced this issue Jul 18, 2024
Nurkambay pushed a commit that referenced this issue Jul 19, 2024
…2388)

Signed-off-by: ap891843 <aman.prashant@broadcom.com>
@zimlu02 zimlu02 closed this as completed Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants