Differential D15793

Sort keys in json encoder
ClosedPublic
Actions

Authored by davidedmundson on Sep 27 2018, 10:53 AM.

Details

Reviewers
broulik
Group Reviewers
Plasma
Commits
R856:ef33a69d7ea6: Sort keys in json encoder
R856:2d0a99ce4c73: Sort keys in json encoder
Summary

Should supress scripty noise.

The previous manualy sorting above was ultimately useless as it's stored
as a map internally inside the json blob. We need to sort when writing out,
it turns out there's an argument for that.

Test Plan

Ran script
Still got some output.
Running twice didn't change output

Diff Detail

Repository
R856 Plasma Browser Integration
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
Restricted Application added a project: Plasma. · View Herald TranscriptSep 27 2018, 10:53 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.Sep 27 2018, 10:53 AM
broulik accepted this revision.Sep 27 2018, 10:55 AM
This revision is now accepted and ready to land.Sep 27 2018, 10:55 AM
Closed by commit R856:ef33a69d7ea6: Sort keys in json encoder (authored by davidedmundson). · Explain WhySep 27 2018, 11:03 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mutil/convertpottojson.py (4 lines)

Diff 42418

View Options

util/convertpottojson.py

Loading...