- Avoid rewrapping LoadError inside LoadError.

- Avoid rewrapping LoadError inside LoadError.

R=rmacnak@google.com

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

[Ivan Posva, 2015-03-11 23:45:25]

iposva@google.com changed reviewers:
+ johnmccutchan@google.com, rmacnak@google.com

[rmacnak, 2015-03-11 23:50:52]

lgtm

https://codereview.chromium.org/1001733002/diff/1/runtime/bin/builtin.dart
File runtime/bin/builtin.dart (right):

https://codereview.chromium.org/1001733002/diff/1/runtime/bin/builtin.dart#ne...
runtime/bin/builtin.dart:301: var error = new LoadError(dataOrError.toString());
toString is redundant

[Ivan Posva, 2015-03-11 23:53:57]

Thanks!

-Ivan

https://codereview.chromium.org/1001733002/diff/1/runtime/bin/builtin.dart
File runtime/bin/builtin.dart (right):

https://codereview.chromium.org/1001733002/diff/1/runtime/bin/builtin.dart#ne...
runtime/bin/builtin.dart:301: var error = new LoadError(dataOrError.toString());
On 2015/03/11 23:50:52, rmacnak wrote:
> toString is redundant

Redundant, but defensive in case you are not running in checked mode. I will
leave it like this and I hope that in a good round of cleanup we can get rid of
this. Either the requirement that we get a String in the error case or the call
toString().

[Ivan Posva, 2015-03-11 23:54:16]

Committed patchset #1 (id:1) manually as r44419 (presubmit successful).