Move Bidi functions into methods to make the docs look better

Move Bidi functions into methods to make the docs look better
Committed: https://code.google.com/p/dart/source/detail?r=13261

[Alan Knight, 2012-10-04 17:38:45]

This moves the Bidi standalone functions and fields into statics on a Bidi
class. The main reason to do this is that if we have a single Intl library, we
don't want the top level docs for it dominated by all the Bidi functions.

Doing that left one public function from the current messages skeleton
implementation that was for internal use only, which also isn't great to have as
the only thing at the top level of Intl docs, so I moved that into an internal
library, which is better anyway.

[Emily Fortuna, 2012-10-04 18:37:54]

https://codereview.chromium.org/11032035/diff/1/pkg/intl/lib/bidi_utils.dart
File pkg/intl/lib/bidi_utils.dart (right):

https://codereview.chromium.org/11032035/diff/1/pkg/intl/lib/bidi_utils.dart#...
pkg/intl/lib/bidi_utils.dart:53: * text. #######
is this a marker for more documentation suggested? (if so, can you make a TODO
instead?)

https://codereview.chromium.org/11032035/diff/1/pkg/intl/lib/bidi_utils.dart#...
pkg/intl/lib/bidi_utils.dart:57: /** Unicode "Left-To-Right Embedding" (LRE)
character. */
can you fix the indentation of all of these lines now that they're inside of a
class?

https://codereview.chromium.org/11032035/diff/1/pkg/intl/lib/bidi_utils.dart#...
pkg/intl/lib/bidi_utils.dart:398: 
You're going to love this... delete this extra line, please.

[Alan Knight, 2012-10-04 19:34:22]

https://codereview.chromium.org/11032035/diff/1/pkg/intl/lib/bidi_utils.dart
File pkg/intl/lib/bidi_utils.dart (right):

https://codereview.chromium.org/11032035/diff/1/pkg/intl/lib/bidi_utils.dart#...
pkg/intl/lib/bidi_utils.dart:53: * text. #######
On 2012/10/04 18:37:55, Emily Fortuna wrote:
> is this a marker for more documentation suggested? (if so, can you make a TODO
> instead?)

Oops. That was a temporary marker that I should have come back to before
uploading. Removed, and changed the comment.

https://codereview.chromium.org/11032035/diff/1/pkg/intl/lib/bidi_utils.dart#...
pkg/intl/lib/bidi_utils.dart:57: /** Unicode "Left-To-Right Embedding" (LRE)
character. */
On 2012/10/04 18:37:55, Emily Fortuna wrote:
> can you fix the indentation of all of these lines now that they're inside of a
> class?

Done.

https://codereview.chromium.org/11032035/diff/1/pkg/intl/lib/bidi_utils.dart#...
pkg/intl/lib/bidi_utils.dart:398: 
On 2012/10/04 18:37:55, Emily Fortuna wrote:
> You're going to love this... delete this extra line, please.

Done.

[Emily Fortuna, 2012-10-04 21:11:47]

lgtm!