change smoke to use String instead of Symbol

dart-archive / smoke

Smoke is a Dart package that exposes a reduced reflective system API. This API includes accessing objects in a dynamic fashion (read properties, write properties, and call methods), inspecting types (for example, whether a method exists), and symbol/string convention.

https://pub.dartlang.org/packages/smoke

BSD 3-Clause "New" or "Revised" License

12 stars 3 forks source link

change smoke to use String instead of Symbol #7

Open DartBot opened 9 years ago

DartBot commented 9 years ago

Issue by sigmundch Originally opened as dart-lang/sdk#17863

Strings are easier to use and faster than symbol (e.g. lookup in tables), and if the only purpose of symbol is for dart:mirrors, we could change smoke to hide them under the smoke/mirrors.dart implementation.

In that case, should we also be changing package:observe and other packages to use String instead of symbol?

DartBot commented 9 years ago

Comment by sigmundch

Added Pkg-Smoke label.

DartBot commented 9 years ago

Comment by jmesserly