search

clarete / hackernews.el

Hacker News client for Emacs
GNU General Public License v3.0
248 stars 26 forks source link

Use native JSON support in Emacs 27 #37

Closed basil-conto closed 6 years ago

basil-conto commented 6 years ago

Obviously URL retrieval completely overshadows JSON parsing as a performance bottleneck, but every little helps. Quoth etc/NEWS:

The new configure option '--with-json' adds support for JSON using
the Jansson library.  It is on by default; use 'configure
--with-json=no' to build without Jansson support.  The new JSON
functions 'json-serialize', 'json-insert', 'json-parse-string', and
'json-parse-buffer' are typically much faster than their Lisp
counterparts from json.el.
basil-conto commented 6 years ago

I plan to merge this by the end of the week if there are no suggestions/objections.

clarete commented 6 years ago

Awesome sauce! Good to know about the new feature too! @basil-conto You don't know how much I learned about Emacs development just by reading your PRs!

basil-conto commented 6 years ago

You don't know how much I learned about Emacs development just by reading your PRs!

You and me both. :) Thank you for being so supportive!