add json.tool option to avoid alphabetic sort of fields

[storchaka_gmail.com]

http://bugs.python.org/review/21650/diff/12162/Doc/library/json.rst
File Doc/library/json.rst (right):

http://bugs.python.org/review/21650/diff/12162/Doc/library/json.rst#newcode616
Doc/library/json.rst:616: .. cmdoption:: --unsorted
Just a nitpick. May be name it --no-sort-keys? Then we can change the default in
future and add the --sort-keys option.

http://bugs.python.org/review/21650/diff/12162/Lib/json/tool.py
File Lib/json/tool.py (right):

http://bugs.python.org/review/21650/diff/12162/Lib/json/tool.py#newcode28
Lib/json/tool.py:28: parser.add_argument('--unsorted', dest='sort_keys',
action='store_true',
This looks confusing. May be use

 action='store_const', const=False, default=True

?

http://bugs.python.org/review/21650/diff/12162/Lib/json/tool.py#newcode38
Lib/json/tool.py:38: obj = json.load(infile,
object_pairs_hook=collections.OrderedDict)
object_pairs_hook can slow down a loading. May be use it only when sort_keys is
false?

[Martin Panter]

https://bugs.python.org/review/21650/diff/12162/Doc/library/json.rst
File Doc/library/json.rst (right):

https://bugs.python.org/review/21650/diff/12162/Doc/library/json.rst#newcode616
Doc/library/json.rst:616: .. cmdoption:: --unsorted
Or you could just leave it as --unsorted, and in the future change the default
and add a --sorted option. :P --no-sort-keys is clumsier to type.

https://bugs.python.org/review/21650/diff/12162/Lib/json/tool.py
File Lib/json/tool.py (right):

https://bugs.python.org/review/21650/diff/12162/Lib/json/tool.py#newcode28
Lib/json/tool.py:28: parser.add_argument('--unsorted', dest='sort_keys',
action='store_true',
Highly confusing! I would simply drop the “dest” argument, allowing it to create
an attribute called “unsorted” by default instead.

[storchaka_gmail.com]

Besides one small nitpick the patch LGTM.

http://bugs.python.org/review/21650/diff/13192/Doc/whatsnew/3.5.rst
File Doc/whatsnew/3.5.rst (right):

http://bugs.python.org/review/21650/diff/13192/Doc/whatsnew/3.5.rst#newcode219
Doc/whatsnew/3.5.rst:219: of dictionaries by their key (contributed by Berker
Peksag in :issue:`21650`).
Please be consistent with the rest of the file.

  of dictionaries by their key. (Contributed by Berker Peksag in
:issue:`21650`.)

[berkerpeksag]

http://bugs.python.org/review/21650/diff/13192/Doc/whatsnew/3.5.rst
File Doc/whatsnew/3.5.rst (right):

http://bugs.python.org/review/21650/diff/13192/Doc/whatsnew/3.5.rst#newcode219
Doc/whatsnew/3.5.rst:219: of dictionaries by their key (contributed by Berker
Peksag in :issue:`21650`).
On 2014/11/02 19:22:46, storchaka wrote:
> Please be consistent with the rest of the file.
> 
>   of dictionaries by their key. (Contributed by Berker Peksag in
> :issue:`21650`.)

Done.