Skip to content
This repository has been archived by the owner on Nov 20, 2018. It is now read-only.

Extend WebEncoders API to avoid allocations within the methods #563

Closed
wants to merge 3 commits into from

Commits on Feb 17, 2016

  1. Extend WebEncoders API to avoid allocations within the methods

    - rewrite existing methods in terms of the new ones
    - don't allocate multiple 0-length arrays
    
    nit:
    - clarify a couple of doc comments e.g. using `<paramref/>`
    - move an error message into a resource
     - pass parameter names into new resource
    - rename parameters for consistency e.g. `inputLength` -> `count`
    dougbu committed Feb 17, 2016
    Configuration menu
    Copy the full SHA
    4623d92 View commit details
    Browse the repository at this point in the history
  2. PR comments: Move away from in-place updates to input array

    Also name literal `int` parameters.
    dougbu committed Feb 17, 2016
    Configuration menu
    Copy the full SHA
    d0c8adc View commit details
    Browse the repository at this point in the history

Commits on Feb 18, 2016

  1. PR comments:

    - more `var`
    - move `count` parameters to the end
    
    Also added a test covering the new overloads.
    dougbu committed Feb 18, 2016
    Configuration menu
    Copy the full SHA
    2bd6965 View commit details
    Browse the repository at this point in the history