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.

Sign up for GitHub

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

Jump to bottom

add a %no_suppress_warnings directive #1027

Merged
merged 6 commits into from
Jan 6, 2023
Merged

add a %no_suppress_warnings directive #1027

merged 6 commits into from
Jan 6, 2023

Commits on Jan 6, 2023

  1. jflex AbstractLexScan: remove TODO 

    Char classes still need to be mutable at this point. They can only be
made immutable after the semantic analysis phase at the end of parsing.
    @lsf37
    lsf37 committed Jan 6, 2023
    Configuration menu
    Copy the full SHA
    8c44333 View commit details
    Browse the repository at this point in the history

  2. jflex: add %no_suppress_warnings directive 

    By default JFlex emits @SuppressWarnings("fallthrough") at the class
level. This directive removes that annotation and leaves it up to the
user to define their own.

Fixes #762
    @lsf37
    lsf37 committed Jan 6, 2023
    Configuration menu
    Copy the full SHA
    f44691e View commit details
    Browse the repository at this point in the history

  3. docs: document %no_suppress_warnings

    @lsf37
    lsf37 committed Jan 6, 2023
    Configuration menu
    Copy the full SHA
    c6a9a3e View commit details
    Browse the repository at this point in the history

  4. test: add test for %no_suppress_warnings

    @lsf37
    lsf37 committed Jan 6, 2023
    Configuration menu
    Copy the full SHA
    afb5a92 View commit details
    Browse the repository at this point in the history

  5. tests: enable missed tests 

    Looks like BUILD.bzl wasn't picked up; renamed to BUILD.bazel and fixed
output drift.
    @lsf37
    lsf37 committed Jan 6, 2023
    Configuration menu
    Copy the full SHA
    6c03a66 View commit details
    Browse the repository at this point in the history

  6. github: suppress more codeql warnings 

    - LexScan.java: uncaught-number-format-exception:
  Format is checked by the lexer spec.

- LexScan.java: misleading-indentation: in comment only.
    @lsf37
    lsf37 committed Jan 6, 2023
    Configuration menu
    Copy the full SHA
    e2f5d00 View commit details
    Browse the repository at this point in the history