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

x.json2.decoder2: use json_str and json_len, intead of just json #22704

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

x.json2.decoder2: use json_str and json_len, intead of just json #22704

wants to merge 8 commits into from
+288 −112

Commits on Oct 30, 2024

  1. perf: use json_str and json_len, intead of just json

    @enghitalo
    enghitalo committed Oct 30, 2024
    Configuration menu
    Copy the full SHA
    d170014 View commit details
    Browse the repository at this point in the history

Commits on Nov 1, 2024

  1. Merge remote-tracking branch 'upstream/master' into x.json2.decoder.j… 

    …son2.Any
    @enghitalo
    enghitalo committed Nov 1, 2024
    Configuration menu
    Copy the full SHA
    9b4fb9a View commit details
    Browse the repository at this point in the history

  2. apply review is suggestions

    @enghitalo
    enghitalo committed Nov 1, 2024
    Configuration menu
    Copy the full SHA
    3d06289 View commit details
    Browse the repository at this point in the history

  3. increase Stru in bench

    @enghitalo
    enghitalo committed Nov 1, 2024
    Configuration menu
    Copy the full SHA
    6f49ac9 View commit details
    Browse the repository at this point in the history

  4. fix: missin doc

    @enghitalo
    enghitalo committed Nov 1, 2024
    Configuration menu
    Copy the full SHA
    b765a23 View commit details
    Browse the repository at this point in the history

Commits on Nov 3, 2024

  1. Merge remote-tracking branch 'upstream/master' into x.json2.decoder.j… 

    …son2.Any
    @enghitalo
    enghitalo committed Nov 3, 2024
    Configuration menu
    Copy the full SHA
    a6b9c80 View commit details
    Browse the repository at this point in the history

  2. The compiler is optimizing for this. Since the change is longer to wr… 

    …ite and is a bit less expressive (checking the `len` is not exclusive to strings; while it is fairly clear in this context, in others it can create more mental overhead to grasp what is happening when reading the code) using `== ` has become something that v vet recommends to check for an empty string.
    @enghitalo
    enghitalo committed Nov 3, 2024
    Configuration menu
    Copy the full SHA
    7a6430e View commit details
    Browse the repository at this point in the history

  3. doc: remove duplicated

    @enghitalo
    enghitalo committed Nov 3, 2024
    Configuration menu
    Copy the full SHA
    a8db5e1 View commit details
    Browse the repository at this point in the history