Skip to content

brotli: add brotli support #20458

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Hackzzila wants to merge 11 commits into from
Closed

brotli: add brotli support #20458

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
a295f2a
deps: add brotli
Hackzzila May 1, 2018
5d82880
brotli: add brotli
Hackzzila May 1, 2018
52e2070
benchmark,test: add brotli
Hackzzila May 1, 2018
2154080
doc, test: add brotli and fix tests
Hackzzila May 1, 2018
54aa2e7
doc: improve brotli docs
Hackzzila May 2, 2018
69cc504
doc: make acceptEncoding const
Hackzzila May 2, 2018
5eb194f
doc: small brotli doc fixes
Hackzzila May 2, 2018
72c2d4e
doc: small doc fixes
Hackzzila May 2, 2018
be82db1
brotli,doc,src,test: remove copyright headers, small doc/code fixes
Hackzzila May 2, 2018
ccc89f2
deps,tools: update license-builder.sh and LICENSE
Hackzzila May 2, 2018
fcae0ef
Merge branch 'master' into brotli-dev
Hackzzila Jun 4, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
doc: small brotli doc fixes
  • Loading branch information
@Hackzzila
Hackzzila committed May 2, 2018
commit 5eb194fe4e651d3c91cf0408884d6c4131f019e1
6 changes: 3 additions & 3 deletions doc/api/brotli.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -344,14 +344,14 @@ added: REPLACEME
-->

Reset the encoder/decoder to factory defaults. This will undo any
changes you have made via [`.setParameter()`][]
changes you have made via [`.setParameter()`][].

### brotli.setParameter(param, value)
<!-- YAML
added: REPLACEME
-->

- `param` {number} A valid parameter from [Parameters][]
- `param` {number} a valid parameter from [Parameters][]
- `value` {number}

Dynamically set a parameter.
Expand Down Expand Up @@ -474,7 +474,7 @@ Decompress a chunk of data.
[`.setParameter()`]: #brotli_brotli_setparameter_param_value
[`Accept-Encoding`]: https://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.3
[`ArrayBuffer`]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/ArrayBuffer
[`brotli.constants`]: #brotli_constants
[`brotli.constants`]: #brotli_brotli_constants
[`brotli/decode.h`]: https://github.com/google/brotli/blob/v1.0.4/c/include/brotli/decode.h
Copy link
Contributor

@vsemozhetbyt vsemozhetbyt May 1, 2018 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We usually sort bottom references in ASCII order, so this should go before the [`brotli/encode.h`]:.

Copy link
Contributor

@vsemozhetbyt vsemozhetbyt May 1, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

References below also need some resorting.

Copy link
Contributor Author

@Hackzzila Hackzzila May 2, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't know how I managed to mess up sorting 😆

[`brotli/encode.h`]: https://github.com/google/brotli/blob/v1.0.4/c/include/brotli/encode.h
[`Buffer`]: buffer.html#buffer_class_buffer
Expand Down