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

Better dumping #1993

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Better dumping #1993

wants to merge 6 commits into from

Commits on Sep 20, 2020

  1. Factor out constants for print indentation control 

    Also commented to make the implementation clearer.
    @bb010g
    bb010g committed Sep 20, 2020
    Configuration menu
    Copy the full SHA
    c9f92b0 View commit details
    Browse the repository at this point in the history

  2. jvp_dump_string: Consolidate flags usage

    @bb010g
    bb010g committed Sep 20, 2020
    Configuration menu
    Copy the full SHA
    9d320e1 View commit details
    Browse the repository at this point in the history

  3. Factor out jvp_dump_char from jvp_dump_string

    @bb010g
    bb010g committed Sep 20, 2020
    Configuration menu
    Copy the full SHA
    7c047a2 View commit details
    Browse the repository at this point in the history

  4. Handle RAW_OUTPUT through JV_PRINT_RAW flag 

    Behavior is the same, but this form is easier to understand and make
robust.
    @bb010g
    bb010g committed Sep 20, 2020
    Configuration menu
    Copy the full SHA
    2a82116 View commit details
    Browse the repository at this point in the history

  5. Fix stderr/0 builtin to match documentation 

    It's supposed to print raw, compact, and with no decoration or trailing
newline. Previously, it would print JSON formatted, compact, with no
trailing newline and no easy way to obtain one, and ignoring the
--ascii-output / -a flag.
    @bb010g
    bb010g committed Sep 20, 2020
    Configuration menu
    Copy the full SHA
    e50bb3f View commit details
    Browse the repository at this point in the history

  6. Bring --raw-output and --ascii-output to agreement 

    Escapes are still printed whenever characters
outside the ASCII plane are encountered. To avoid
ambiguity, backslash is the only ASCII character
escaped (as `\\`).

Fixes jqlang#1788, properly this time. Closes jqlang#1789.
    @bb010g
    bb010g committed Sep 20, 2020
    Configuration menu
    Copy the full SHA
    5f0f353 View commit details
    Browse the repository at this point in the history