First version of Codecs and Converters.

First version of Codecs and Converters.

No chunked conversions, yet.

R=asiva@google.com, lrn@google.com

Committed: https://code.google.com/p/dart/source/detail?r=24999

[Lasse Reichstein Nielsen, 2013-07-12 11:50:46]

lgtm

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/codec.dart
File sdk/lib/codec/codec.dart (right):

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/codec.dart#n...
sdk/lib/codec/codec.dart:19: * Fused codecs generally work in an optimized way
and can be faster than
generally work in an optimized way ->
generally attempt to optimize the operations

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/codec.dart#n...
sdk/lib/codec/codec.dart:30: Converter<S, T> get encoder;
Documentation!
E.g.:
This encoder converts from S to T. It may be statefull(?), so you should get a
new decoder for each [S] you want to convert.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/codec.dart#n...
sdk/lib/codec/codec.dart:43: * them correctly.
Elaborate. Is this just a case of "output of this codec's encoder must match
input of other codec's decoder and vice versa"?

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/codec.dart#n...
sdk/lib/codec/codec.dart:50: *     assert(decoded is List && decoded[0] ==
"json-object");
Example using [inverted]?

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/codec.dart#n...
sdk/lib/codec/codec.dart:59: * The [encoder] and [decoder] of the resulting
codec are inversed.
inversed -> swapped.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/codec.dart#n...
sdk/lib/codec/codec.dart:64: class _FusedCodec<S, M, T> extends Codec<S, T> {
Documentation for class, please.
This seems to be a general class used by all codecs, because the real fusing
happens at the Converter level.

Will there be fused codecs that know that their parts' encoder/decoder are
stateless, so it can create the fused Converter only once?

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/codec.dart#n...
sdk/lib/codec/codec.dart:71: _FusedCodec(Codec<S, M> first, Codec<M, T> second)
_FusedCodec(this._first, this._second);

I don't usually bother with prettier names for private classes. If I can read
"_FusedCodec" without problems, I can read "_first" too.
For public classes, the names are important to the API.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/codec.dart#n...
sdk/lib/codec/codec.dart:76: class _InvertedCodec<S, T> extends Codec<S, T> {
Consider swapping S and T here, so it reads better when compared to Codec
itself.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/encoding.dart
File sdk/lib/codec/encoding.dart (right):

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/encoding.dar...
sdk/lib/codec/encoding.dart:12: // TODO(floitsch): add other encodings, like
ASCII and ISO_8859_1.
These are partial encodings of strings (where strings are seen as UTF-16 +
unmatched surrogates).
They could be added in a package instead of here. I'd prefer to just have the
total encodings (UTF-8, UTF-16, UTF-32) in the core library, unless we see it as
a way to optimize.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/encoding.dar...
sdk/lib/codec/encoding.dart:16: static const UTF8 = const Utf8Codec();
Put this outside the class. Since it's an open-ended enum, others will have to
add other encodings, and they can't put it inside the class, so we will get the
inconsistent:
 Encoding.UTF8
but
 WIN1252
from another package.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/encoding.dar...
sdk/lib/codec/encoding.dart:20: * A [Utf8Codec] encodes strings to utf-8
codeunits (bytes) and decodes
codeunits -> code units

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/encoding.dar...
sdk/lib/codec/encoding.dart:21: * utf-8 codeunits to strings.
utf-8 -> UTF-8
codeunits -> code units

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/encoding.dar...
sdk/lib/codec/encoding.dart:23: // TODO(floitsch): Needs a way to specify a
replacement character for decoding.
Default replacement character is U+FFFD (aka. REPLACEMENT CHARACTER).
Using a different replacement character is really not part of UTF-8 decoding,
but should be handled by a following character replacement step.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/json.dart
File sdk/lib/codec/json.dart (right):

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/json.dart#ne...
sdk/lib/codec/json.dart:10: * A [JsonCodec] encodes Json-objects to strings and
decodes strings to
Json-objects -> JSON objects

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/json.dart#ne...
sdk/lib/codec/json.dart:11: * Json-objects.
JSON objects.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/json.dart#ne...
sdk/lib/codec/json.dart:17: * Creates a JsonCodec with the given reviver.
`JsonCodec`

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/json.dart#ne...
sdk/lib/codec/json.dart:19: * that has been parsed.
... has been parsed during decoding.
The key passed to the [reviver] function is the integer list index for a list
property, the map string for object properties, and `null` for the final result.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/json.dart#ne...
sdk/lib/codec/json.dart:21: factory JsonCodec.withReviver(reviver(var key, var
value)) =
The reviver is only for the decoder.
The encoder should have a 'replacer' function instead: 
  Object replacer(var key, var value) 
that provides the opposite operation. The returned object must be a List, Map or
basic type.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/json.dart#ne...
sdk/lib/codec/json.dart:25: // aliases for encode/decode?
It would be redundant, and we would probably want to remove it in the long run,
so I think not.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/json.dart#ne...
sdk/lib/codec/json.dart:46: reviver = reviver == null ? _reviver : reviver;
Parentheses around "reviver == null"

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/converter....
File sdk/lib/convert/converter.dart (right):

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/converter....
sdk/lib/convert/converter.dart:31: class _FusedConverter<S, M, T> extends
Converter<S, T> {
Comment for class.
I assume this is just the naïve/default fused converter that does one before the
other.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/converter....
sdk/lib/convert/converter.dart:35: _FusedConverter(Converter<S, M> first,
Converter<M, T> second)
this._first, this._second

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/json.dart
File sdk/lib/convert/json.dart (right):

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/json.dart#...
sdk/lib/convert/json.dart:14: * Converts the given Json-object [o] and returns
its stringified
Specify whay "Json-object" is (JSON object).
It's a structure of 
  number, bool, null, string,
  List of JSON objects or
  Maps from String to JSON objects.
It's really a mis-nomer. JSON is a text format, a "JSON object" is just one that
can be directly converted to a JSON string using some heuristic, so what it is
depends on what we allow.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/json.dart#...
sdk/lib/convert/json.dart:15: * Json representation.
Json -> JSON.
Drop "stringified", JSON is a string.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/json.dart#...
sdk/lib/convert/json.dart:24: * A [JsonDecoder] parses Json strings and builds
the corresponding
JSON strings
JSON objects (if defined above)

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/json.dart#...
sdk/lib/convert/json.dart:37: * Converts the given Json-string [input] to the
corresponding object.
JSON string.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/utf.dart
File sdk/lib/convert/utf.dart (right):

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/utf.dart#n...
sdk/lib/convert/utf.dart:8: * A [Utf8Encoder] converts strings to their utf-8
code units (a list of
utf -> UTF

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/utf.dart#n...
sdk/lib/convert/utf.dart:9: * 8-bit integers).
unsigned 8-bit integers,
or just integer in the range 0..255.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/utf.dart#n...
sdk/lib/convert/utf.dart:13: * Converts [str] to its utf-8 code units (a list of
str -> string
utf-8 -> UTF-8

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/utf.dart#n...
sdk/lib/convert/utf.dart:20: * A [Utf8Decoder] converts utf-8 code units (lists
of 8-bit ints) to strings.
UTF-8
unsigned
"to a string"

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/utf.dart#n...
sdk/lib/convert/utf.dart:24: * Converts the utf-8 [codeUnits] (a list of 8-bit
integers) to the
ditto

[srdjan, 2013-07-12 16:03:16]

https://codereview.chromium.org/19000006/diff/14001/sdk/lib/codec/json.dart
File sdk/lib/codec/json.dart (right):

https://codereview.chromium.org/19000006/diff/14001/sdk/lib/codec/json.dart#n...
sdk/lib/codec/json.dart:46: reviver = (reviver == null) ? _reviver : reviver;
Why not:
if (reviver == null) {
  reviver = reviver_;
}

[floitsch, 2013-07-12 16:09:15]

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/codec.dart
File sdk/lib/codec/codec.dart (right):

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/codec.dart#n...
sdk/lib/codec/codec.dart:19: * Fused codecs generally work in an optimized way
and can be faster than
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> generally work in an optimized way ->
> generally attempt to optimize the operations

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/codec.dart#n...
sdk/lib/codec/codec.dart:30: Converter<S, T> get encoder;
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> Documentation!
> E.g.:
> This encoder converts from S to T. It may be statefull(?), so you should get a
> new decoder for each [S] you want to convert.

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/codec.dart#n...
sdk/lib/codec/codec.dart:43: * them correctly.
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> Elaborate. Is this just a case of "output of this codec's encoder must match
> input of other codec's decoder and vice versa"?

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/codec.dart#n...
sdk/lib/codec/codec.dart:50: *     assert(decoded is List && decoded[0] ==
"json-object");
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> Example using [inverted]?

Added one, but the best example would be with the line-splitter which is not yet
in the library. Added TODO.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/codec.dart#n...
sdk/lib/codec/codec.dart:59: * The [encoder] and [decoder] of the resulting
codec are inversed.
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> inversed -> swapped.

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/codec.dart#n...
sdk/lib/codec/codec.dart:64: class _FusedCodec<S, M, T> extends Codec<S, T> {
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> Documentation for class, please.
> This seems to be a general class used by all codecs, because the real fusing
> happens at the Converter level.
correct.
> 
> Will there be fused codecs that know that their parts' encoder/decoder are
> stateless, so it can create the fused Converter only once?
wasn't planned, but individual classes will implement their own Fused converters
(for example to chunk the non-chunked input).

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/codec.dart#n...
sdk/lib/codec/codec.dart:71: _FusedCodec(Codec<S, M> first, Codec<M, T> second)
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> _FusedCodec(this._first, this._second);
> 
> I don't usually bother with prettier names for private classes. If I can read
> "_FusedCodec" without problems, I can read "_first" too.
> For public classes, the names are important to the API.

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/codec.dart#n...
sdk/lib/codec/codec.dart:76: class _InvertedCodec<S, T> extends Codec<S, T> {
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> Consider swapping S and T here, so it reads better when compared to Codec
> itself.

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/encoding.dart
File sdk/lib/codec/encoding.dart (right):

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/encoding.dar...
sdk/lib/codec/encoding.dart:12: // TODO(floitsch): add other encodings, like
ASCII and ISO_8859_1.
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> These are partial encodings of strings (where strings are seen as UTF-16 +
> unmatched surrogates).
> They could be added in a package instead of here. I'd prefer to just have the
> total encodings (UTF-8, UTF-16, UTF-32) in the core library, unless we see it
as
> a way to optimize.

If we intend to replace the Encoding enum in dart:io we should have at least
what they are having.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/encoding.dar...
sdk/lib/codec/encoding.dart:16: static const UTF8 = const Utf8Codec();
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> Put this outside the class. Since it's an open-ended enum, others will have to
> add other encodings, and they can't put it inside the class, so we will get
the
> inconsistent:
>  Encoding.UTF8
> but
>  WIN1252
> from another package.

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/encoding.dar...
sdk/lib/codec/encoding.dart:20: * A [Utf8Codec] encodes strings to utf-8
codeunits (bytes) and decodes
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> codeunits -> code units

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/encoding.dar...
sdk/lib/codec/encoding.dart:21: * utf-8 codeunits to strings.
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> utf-8 -> UTF-8
> codeunits -> code units

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/encoding.dar...
sdk/lib/codec/encoding.dart:23: // TODO(floitsch): Needs a way to specify a
replacement character for decoding.
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> Default replacement character is U+FFFD (aka. REPLACEMENT CHARACTER).
> Using a different replacement character is really not part of UTF-8 decoding,
> but should be handled by a following character replacement step.

Changed comment.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/json.dart
File sdk/lib/codec/json.dart (right):

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/json.dart#ne...
sdk/lib/codec/json.dart:10: * A [JsonCodec] encodes Json-objects to strings and
decodes strings to
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> Json-objects -> JSON objects

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/json.dart#ne...
sdk/lib/codec/json.dart:11: * Json-objects.
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> JSON objects.

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/json.dart#ne...
sdk/lib/codec/json.dart:17: * Creates a JsonCodec with the given reviver.
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> `JsonCodec`

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/json.dart#ne...
sdk/lib/codec/json.dart:19: * that has been parsed.
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> ... has been parsed during decoding.
> The key passed to the [reviver] function is the integer list index for a list
> property, the map string for object properties, and `null` for the final
result.

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/json.dart#ne...
sdk/lib/codec/json.dart:21: factory JsonCodec.withReviver(reviver(var key, var
value)) =
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> The reviver is only for the decoder.
> The encoder should have a 'replacer' function instead: 
>   Object replacer(var key, var value) 
> that provides the opposite operation. The returned object must be a List, Map
or
> basic type.

Yes. But the current JSON implementation doesn't provide this (afaics).

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/json.dart#ne...
sdk/lib/codec/json.dart:25: // aliases for encode/decode?
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> It would be redundant, and we would probably want to remove it in the long
run,
> so I think not.

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/codec/json.dart#ne...
sdk/lib/codec/json.dart:46: reviver = reviver == null ? _reviver : reviver;
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> Parentheses around "reviver == null"

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/converter....
File sdk/lib/convert/converter.dart (right):

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/converter....
sdk/lib/convert/converter.dart:31: class _FusedConverter<S, M, T> extends
Converter<S, T> {
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> Comment for class.
> I assume this is just the naïve/default fused converter that does one before
the
> other.

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/converter....
sdk/lib/convert/converter.dart:35: _FusedConverter(Converter<S, M> first,
Converter<M, T> second)
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> this._first, this._second

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/json.dart
File sdk/lib/convert/json.dart (right):

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/json.dart#...
sdk/lib/convert/json.dart:14: * Converts the given Json-object [o] and returns
its stringified
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> Specify whay "Json-object" is (JSON object).
> It's a structure of 
>   number, bool, null, string,
>   List of JSON objects or
>   Maps from String to JSON objects.
> It's really a mis-nomer. JSON is a text format, a "JSON object" is just one
that
> can be directly converted to a JSON string using some heuristic, so what it is
> depends on what we allow.

copied the documentation from `stringify`.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/json.dart#...
sdk/lib/convert/json.dart:15: * Json representation.
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> Json -> JSON.
> Drop "stringified", JSON is a string.

removed.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/json.dart#...
sdk/lib/convert/json.dart:24: * A [JsonDecoder] parses Json strings and builds
the corresponding
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> JSON strings
> JSON objects (if defined above)

Changed to
"JSON strings and builds the corresponding objects."

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/json.dart#...
sdk/lib/convert/json.dart:37: * Converts the given Json-string [input] to the
corresponding object.
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> JSON string.

copied the original description.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/utf.dart
File sdk/lib/convert/utf.dart (right):

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/utf.dart#n...
sdk/lib/convert/utf.dart:8: * A [Utf8Encoder] converts strings to their utf-8
code units (a list of
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> utf -> UTF

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/utf.dart#n...
sdk/lib/convert/utf.dart:9: * 8-bit integers).
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> unsigned 8-bit integers,
> or just integer in the range 0..255.

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/utf.dart#n...
sdk/lib/convert/utf.dart:13: * Converts [str] to its utf-8 code units (a list of
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> str -> string
> utf-8 -> UTF-8

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/utf.dart#n...
sdk/lib/convert/utf.dart:20: * A [Utf8Decoder] converts utf-8 code units (lists
of 8-bit ints) to strings.
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> UTF-8
> unsigned
> "to a string"

Done.

https://codereview.chromium.org/19000006/diff/4001/sdk/lib/convert/utf.dart#n...
sdk/lib/convert/utf.dart:24: * Converts the utf-8 [codeUnits] (a list of 8-bit
integers) to the
On 2013/07/12 11:50:47, Lasse Reichstein Nielsen wrote:
> ditto

Done.

https://codereview.chromium.org/19000006/diff/14001/sdk/lib/codec/json.dart
File sdk/lib/codec/json.dart (right):

https://codereview.chromium.org/19000006/diff/14001/sdk/lib/codec/json.dart#n...
sdk/lib/codec/json.dart:46: reviver = (reviver == null) ? _reviver : reviver;
On 2013/07/12 16:03:16, srdjan wrote:
> Why not:
> if (reviver == null) {
>   reviver = reviver_;
> }

Done.

[siva, 2013-07-12 17:19:45]

the VM C++ parts LGTM, I haven't reviewed the dart files but I did notice that
there are no new tests for converter.

Is there a Bug which documents how requested these libraries and why they are
being added?

[siva, 2013-07-12 17:20:22]

I meant "who requested these libraries"

[floitsch, 2013-07-15 10:25:19]

On 2013/07/12 17:19:45, siva wrote:
> the VM C++ parts LGTM, I haven't reviewed the dart files but I did notice that
> there are no new tests for converter.
The converters are indirectly tested through the codecs. They will get more
tests in future CLs.
> 
> Is there a Bug which documents how requested these libraries and why they are
> being added?
This is part of the library cleanup.
A related bug: http://dartbug.com/6284

[floitsch, 2013-07-15 10:26:51]

Committed patchset #6 manually as r24999 (presubmit successful).

[Ivan Posva, 2013-07-16 00:02:35]

You need to find a name for dart:codec that does not imply video or audio
compression.

-Ivan

[dgrove, 2013-07-16 00:03:26]

+1 to Ivan's point. I find this library name to be neither intuitive nor
discoverable.
On 2013/07/16 00:02:35, Ivan Posva wrote:
> You need to find a name for dart:codec that does not imply video or audio
> compression.
> 
> -Ivan