Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a section on Strings and Unicode #349

Merged
merged 2 commits into from Jul 8, 2013

Conversation

DmitryOlshansky
Copy link
Member

I'm not proud of wording but we need clarifications on how exactly builtins correlate to Unicode standard (if at all).
The "obvious meaning" present there previously is not good enough definition.

Along the way I fixed a bug in docs - H5 macro was used but not defined, effectively throwing away level-5 titles.

Why wouldn't DDoc complain about undefined macros instead of sliently eating them, that's a topic in its own right.

Preview of that page:
http://blackwhale.github.io/arrays.html#strings_unicode

I'm not proud of wording but we need clarifications
on this subject.
@denis-sh
Copy link
Contributor

denis-sh commented Jul 7, 2013

Thanks a lot!
But htere is no std.uni.normalize and icmp is in std.string module.

@denis-sh
Copy link
Contributor

denis-sh commented Jul 7, 2013

Also could you open a ticket and another pull for macro fix? And another ticket for lack of unicode collation algorithm in D?

@DmitryOlshansky
Copy link
Member Author

But htere is no std.uni.normalize and icmp is in std.string module.

It's going to be there soon:
dlang/phobos#1347

see its slightly outdated docs: http://blackwhale.github.io/phobos/uni.html

Tickets are as usual a good idea:
http://d.puremagic.com/issues/show_bug.cgi?id=10565
http://d.puremagic.com/issues/show_bug.cgi?id=10566

@@ -839,6 +839,28 @@ w = "\r"[0]; // w is assigned the carriage return wchar character
d = 'd'; // d is assigned the character 'd'
---------

$(H5 $(LNAME2 strings_unicode, Strings and Unicode))
$(P Note that built-in comparison operators operate on
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

...operate on a

@DmitryOlshansky
Copy link
Member Author

@andralex fixed these, ready for another round?

(by transparently decoding, see $(PHOBOS algorithm, cmp, std.algorithm.cmp)),
$(PHOBOS uni, icmp, case-insensitive comparison) and $(PHOBOS uni, normalize, normalization).
)
$(P The last but not least - a desired string sorting order differs
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/The last/Last/

@andralex
Copy link
Member

andralex commented Jul 8, 2013

I'll merge this and fix online.

andralex added a commit that referenced this pull request Jul 8, 2013
Add a section on Strings and Unicode
@andralex andralex merged commit aa27402 into dlang:master Jul 8, 2013
@DmitryOlshansky DmitryOlshansky deleted the arrays-page branch July 8, 2013 08:20
@DmitryOlshansky
Copy link
Member Author

Thanks!

denis-sh added a commit to denis-sh/dlang.org that referenced this pull request Jul 20, 2013
This is a fixup for pull request dlang#349.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants