Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Use static JSONEncoders #3049

Merged
merged 2 commits into from
Apr 3, 2018
Merged

Use static JSONEncoders #3049

merged 2 commits into from
Apr 3, 2018

Conversation

richvdh
Copy link
Member

@richvdh richvdh commented Mar 29, 2018

using json.dumps with custom options requires us to create a new JSONEncoder on
each call. It's more efficient to create one upfront and reuse it.

using json.dumps with custom options requires us to create a new JSONEncoder on
each call. It's more efficient to create one upfront and reuse it.
json encoders have an encode method, not a dumps method.
@richvdh richvdh merged commit 8da39ad into develop Apr 3, 2018
@richvdh richvdh deleted the rav/use_staticjson branch May 2, 2018 15:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants