Merge remote-tracking branch 'davisp/master'

1.0.4

Fix binary leak when encoding large strings

This bug was due to an interaction between two optimizations. If we
attempt to flush the buffer before any bytes are used we refused.
However, in enc_ensure we were not checking whether the buffer was
actually flushed so we would allocate a new buffer for the request.

The easiest way to encounter this issue was by encoding a raw binary
longer than 2041 bytes (i.e., `jiffy:encode(<<"stuff...">>).`).

Merge remote-tracking branch 'davisp/master'

Version 1.0.3

Force Windows to export inlined functions

Merge remote-tracking branch 'davisp/master'

Only use -flto when supported

This adds a dynamic check to see if -flto is supported by the compiler.

Co-Authored-By: JianBo He <heeejianbo@gmail.com>

Update Travis-CI Matrix

Apparently Travis-CI has dropped some of the older Erlang versions. I've
also added a run on 22.x

Merge remote-tracking branch 'davisp/master'

1.0.1

Use from_array instead of positional versions

Fix backwards compatibility

Remove accidentally commited rebar state

Fix R14 support

Fix support of VMs pre-17.x

Fix support for Erlang 17.x

1.0.0

Update readme with the exception type change

Upgrade to double-converson 3.1.4

Change errors for bignums to be more explicit

Co-Authored-By: Stephan Renatus <srenatus@chef.io>

Remove -fno-strict-aliasing

We don't do fishy things like type punning so it really isn't
necessary, and supplying it prevents certain optimizations.

Get rid of separate unsigned/signed buffers

Never expand the encode buffer; emit and restart

This greatly increases the performance of long string encodes as
we won't need to copy intermediate results over and over.

Fix erroneous state check in the decoder

If the input contained a mismatched end-of-array/object, the stack
could become empty before a call to dec_curr, which would look
beyond the bounds of the stack. If the value at this invalid
position happened to be st_array, we would pop too much from the
stack and overwrite the data that came before it.

This commit fixes this by letting dec_pop return the previous
state or st_invalid if the stack is empty, letting us exit
gracefully if the state isn't what we expect it to be.

dec_pop_assert is identical to the old dec_pop, tearing down the
emulator on internal errors.

Enable link-time optimization on *nix platforms

Move all atom checks under enif_is_atom

Use the result map for key dedupe

This is a wee bit more cache-friendly than using a std::set with
string keys.

Refactor trapping and trap more often during decode

Skip erroneous UTF-8 validation for atoms

We requested atoms in latin1 and then handled them as utf-8,
erroring out on some valid atoms and performing pointless
validation on others.

Use enif_is_identical for equality checks

Replace sprintf with a dedicated integer print routine

Skip redundant enif_is_empty_list checks during encode

enif_get_list_cell fails when the list is empty.

Use realloc instead of doing it manually

Walk through strings once when encoding

Use an array for the position stack rather than an Erlang list

sizeof(char) == 1 by definition

0.15.2

Raise errors instead of throwing

Documentation improvement decode option copy_strings.

fix bug on hex escape table

number of items on `hexvals` is 128 while table size is 256, so
remaining 128 items are filled with zero. As a result, values in
\xf0-\xff will be treated as zero while should be rejected.

Fix decoding of "\uDBFF\uDFFF" surrogate pair.

add test erlang/otp 21

Explicitly list xmerl for relase to include it

0.15.1

Add missing option for copy strings

Without this, call to decode with copy_strings option in an
application will make Dialyzer complain when using the exported type for options

This is 0.15.0

fixed issue 162 regarding duplicate keys in objects

Add `copy_strings` feature

Some users of Jiffy have experienced issues when decoding large JSON
documents. Normally Jiffy expects smallish documents and returns any
strings as sub-binaries. When dealing with large documents these
sub-binary references can keep a large amount of RAM around unless the
user goes through and applies `binary:copy/1` on every string returned
from Jiffy. This however causes a large amount of CPU usage to do
something that Jiffy could do as it builds the JSON structure.

The `copy_strings` decoder option does exactly this. Instead of
returning sub-binaries Jiffy now copies every string into a newly
allocated binary. Users report that this fixes the memory issues while
also not negatively affecting performance significantly.

Merge remote-tracking branch 'davisp/master'

Bump to 0.14.13

Fix type spec

Bump to 0.14.12

Add tests for deduplication in child objects

Document dedupe_keys option

Merge remote-tracking branch 'davisp/master'

Add `dedupe_keys` option

You can no optionally request that keys are deduplicate inside of Jiffy
instead of having to perform that operation in Erlang.

Tighten string buffer size calculation in enc_string.

When "\u"-escaping a Unicode character, the esc_extra value doesn't
need to include the number of bytes in the input string.  That is, if
a three-byte UTF-8 character is being escaped to a six-byte "\uXXXX"
sequence, esc_extra only needs to be increased by 3.

test erlang 20

Merge remote-tracking branch 'davisp/master'

Merge pull request #151 from wohali/win32-cleanup

Cleanup after new enc lands

Merge remote-tracking branch 'davisp/master'

Update enc for Windows support

Cleanup after new enc lands

Merge remote-tracking branch 'davisp/master'

Update to latest Jiffy

Replace old test case with generated data

For some reason I left a 377KiB test case checked in. Replacing this
with a generated test case is obvious in hindsight.

Specify the escript executable for hooks

Windows appears to not enjoy relative path names for exectuables so
hopefully this fixes things there.

Fixes #83

Jiffy 0.14.11

Maintain my own file list for hex

Apparently hex only includes some files by default but not others
rather than a sane default of anything except for VCS directories. So
now I also get to maintain a list of files in my .app.src.

Bump to 0.14.10 because rebar3 is inept

Apparently rebar3 can't generate a sane version number from Git tags and
adds a bunch of BS build number and ref tags which hex.pm then rejects.
Given that I can't find any documentation on how that works and reading
code is an exercise in over engineered abstractionism I've succumbed to
the demoralizing conclusion that I'll just have to go back to the dark
ages and maintain my versions both in source and in version control.

Add `use_nil` to `encode_option`

A call to `jiffy:encode(Term, [use_nil]).` will currently cause dialyzer
to issue a warning because `use_nil` is not included in `encode_option`.
This appears to just be an omission.

Merge pull request #118 from davisp/feature-add-rebar3-support

Add support for rebar3

Add support for rebar3

Allow Jiffy to be used easily in projects using either rebar 2 or 3.

Update the list of Erlang VMs to use on Travis-CI

Replace PropEr with EQC

PropEr broke my support for R14. Turns out that EQC Mini is quite usable
so I've just switched to that. If EQC Mini exists it will be used, if
not the test is skipped gracefully.

Merge branch 'upstream'

Fix enc_long for 64-bit Windows

Originally reported by @NorthNick on apache/couchdb-jiffy.

Merge pull request #119 from egobrain/patch-1

Fixed wrong jiffy:decode/2 spec

Fixed wrong jiffy:decode/2 spec

Fix force_utf8 for object keys

Previously if a key was malformed UTF-8 and the user specified the
`force_utf8` option we would fail to try and encode a fixed up version
of the object. This was due to missing a clause to catch the
`invalid_object_member_key` exception. This adds the clause and a couple
tests to ensure it works.

Copy double-conversion license to root LICENSE

This was marked in the c_src/double-conversion directory but I've gone
ahead and added a copy to the root LICENSE file.

Ignore hexer.config

Update jiff.app.src for hex.pm

Time for Jiffy to enter the world of Erlang package management.

Remove ETAP license information

Document the use_nil option for encoding JSON

Remove old debug printing

Merge pull request #113 from jparise/map-iter-destroy

Destroy map iterators once we're done with them.

Destroy map iterators once we're done with them.

Each call to enif_map_iterator_create() must be paired with a call to
enif_map_iterator_destroy(). Otherwise, we'll leak memory.

Fixes #112

Fix jiffy:encode Dialyzer spec.

Merge pull request #104 from pinotree/hurd

rebar.config: set CFLAGS/CXXFLAGS/LDFLAGS on Hurd

Don't rely on an environment variable for Travis

The environment variable approach breaks other projects' Travis-CI runs
if they include other projects that depend on a different version of
PropEr.

Fixes: #106

Reduce C++ flags to -O2 to fix gcc5 build

fix Windows compile options

rebar.config: set CFLAGS/CXXFLAGS/LDFLAGS on Hurd

Use the common CFLAGS/CXXFLAGS/LDFLAGS used on Linux/FreeBSD/etc also
on GNU/Hurd.

Merge pull request #103 from vicglarson/master

Fix win32 build

Fix win32 build

Merge branch 'upstream-merge'

Fix compiler warning on gcc 5.1.0

Remove travis notification

Remove related path to rebar