Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1374)

Unified Diff: sdk/lib/codec/codec.dart

Issue 19000006: First version of Codecs and Converters. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Revert sdk/lib/json change. Created 7 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: sdk/lib/codec/codec.dart
diff --git a/sdk/lib/codec/codec.dart b/sdk/lib/codec/codec.dart
new file mode 100644
index 0000000000000000000000000000000000000000..20bc02ace9f971fa7c6c7c78dfb141c943dbf707
--- /dev/null
+++ b/sdk/lib/codec/codec.dart
@@ -0,0 +1,85 @@
+// Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+
+library dart.codec;
+
+import 'dart:convert';
+
+part 'encoding.dart';
+part 'json.dart';
+
+/**
+ * A [Codec] encodes and (if supported) decodes data.
+ *
+ * Codecs can be fused. For example fusing [JSON] and [Encoding.UTF8] produces
+ * an encoder that can convert Json objects directly to bytes, or can decode
+ * bytes directly to json objects.
+ *
+ * Fused codecs generally work in an optimized way and can be faster than
Lasse Reichstein Nielsen 2013/07/12 11:50:47 generally work in an optimized way -> generally at
floitsch 2013/07/12 16:09:15 Done.
+ * executing each step of an encoding separately.
+ *
+ * *Codecs are still experimental and are subject to change without notice.*
+ */
+abstract class Codec<S, T> {
+ const Codec();
+
+ T encode(S input) => encoder.convert(input);
+ S decode(T encoded) => decoder.convert(encoded);
+
+ Converter<S, T> get encoder;
Lasse Reichstein Nielsen 2013/07/12 11:50:47 Documentation! E.g.: This encoder converts from S
floitsch 2013/07/12 16:09:15 Done.
+ Converter<T, S> get decoder;
+
+ /**
+ * Fuses `this` with `other`.
+ *
+ * When encoding, the resulting codec encodes with `this` before
+ * encoding with [other].
+ *
+ * When decoding, the resulting codec decodes with [other] before decoding
+ * with `this`.
+ *
+ * In some cases one needs to use the [inverted] codecs to be able to fuse
+ * them correctly.
Lasse Reichstein Nielsen 2013/07/12 11:50:47 Elaborate. Is this just a case of "output of this
floitsch 2013/07/12 16:09:15 Done.
+ *
+ * Examples:
+ *
+ * final JSON_TO_BYTES = JSON.fuse(Encoding.UTF8);
+ * List<int> bytes = JSON_TO_BYTES.encode(["json-object"]);
+ * var decoded = JSON_TO_BYTES.decode(bytes);
+ * assert(decoded is List && decoded[0] == "json-object");
Lasse Reichstein Nielsen 2013/07/12 11:50:47 Example using [inverted]?
floitsch 2013/07/12 16:09:15 Added one, but the best example would be with the
+ */
+ Codec<S, dynamic> fuse(Codec<T, dynamic> other) {
+ return new _FusedCodec<S, T, dynamic>(this, other);
+ }
+
+ /**
+ * Inverts `this`.
+ *
+ * The [encoder] and [decoder] of the resulting codec are inversed.
Lasse Reichstein Nielsen 2013/07/12 11:50:47 inversed -> swapped.
floitsch 2013/07/12 16:09:15 Done.
+ */
+ Codec<T, S> get inverted => new _InvertedCodec<T, S>(this);
+}
+
+class _FusedCodec<S, M, T> extends Codec<S, T> {
Lasse Reichstein Nielsen 2013/07/12 11:50:47 Documentation for class, please. This seems to be
floitsch 2013/07/12 16:09:15 correct.
+ final Codec<S, M> _first;
+ final Codec<M, T> _second;
+
+ Converter<S, T> get encoder => _first.encoder.fuse(_second.encoder);
+ Converter<T, S> get decoder => _second.decoder.fuse(_first.decoder);
+
+ _FusedCodec(Codec<S, M> first, Codec<M, T> second)
Lasse Reichstein Nielsen 2013/07/12 11:50:47 _FusedCodec(this._first, this._second); I don't u
floitsch 2013/07/12 16:09:15 Done.
+ : this._first = first,
+ this._second = second;
+}
+
+class _InvertedCodec<S, T> extends Codec<S, T> {
Lasse Reichstein Nielsen 2013/07/12 11:50:47 Consider swapping S and T here, so it reads better
floitsch 2013/07/12 16:09:15 Done.
+ final Codec<T, S> _codec;
+
+ _InvertedCodec(Codec<T, S> codec) : _codec = codec;
+
+ Converter<S, T> get encoder => _codec.decoder;
+ Converter<T, S> get decoder => _codec.encoder;
+
+ Codec<T, S> get inverted => _codec;
+}

Powered by Google App Engine
This is Rietveld 408576698