Remove unnecessary StackTrace instantiation (#69)
Manually instantiating `StackTrace.current` when completing with an
error does not create a meaningfully different stack trace compared to
not passing one at all - it has an extra frame but not a meaningful one.
diff --git a/lib/src/client.dart b/lib/src/client.dart
index d89a623..ccc4c39 100644
--- a/lib/src/client.dart
+++ b/lib/src/client.dart
@@ -64,10 +64,8 @@
Client.withoutJson(this._channel) {
done.whenComplete(() {
for (var request in _pendingRequests.values) {
- request.completer.completeError(
- StateError(
- 'The client closed with pending request "${request.method}".'),
- StackTrace.current);
+ request.completer.completeError(StateError(
+ 'The client closed with pending request "${request.method}".'));
}
_pendingRequests.clear();
}).catchError((_) {
diff --git a/pubspec.yaml b/pubspec.yaml
index 5515221..c0c8008 100644
--- a/pubspec.yaml
+++ b/pubspec.yaml
@@ -1,5 +1,5 @@
name: json_rpc_2
-version: 2.2.2
+version: 2.2.3-dev
description: >-
Utilities to write a client or server using the JSON-RPC 2.0 spec.
homepage: https://github.com/dart-lang/json_rpc_2
