Skip to content

Conversation

dscorbett
Copy link
Collaborator

U+20F0 COMBINING ASTERISK ABOVE can be used as a svara marker in Grantha, but since Unicode does not define its Indic categories, a dotted circle is inserted between it and a following svara marker. For example, the string ⟨𑌊⃰𑍧𑍧⟩ from L2/09-372 gets a dotted circle.

@jfkthame
Copy link
Collaborator

jfkthame commented Nov 6, 2017

What about the dash and caret marks that also appear in L2/09-372 samples -- should something similar be done about those?

@jfkthame
Copy link
Collaborator

jfkthame commented Nov 6, 2017

Hmm, I see there are U+A8FA and U+A8FB in the Devanagari Extended block, but I don't know enough about them to judge whether these are the same characters as seen in the Grantha examples, or behave differently in some way.

@dscorbett
Copy link
Collaborator Author

That document specifies that they are U+1CD0 VEDIC TONE KARSHANA and U+1CD2 VEDIC TONE PRENKHA.

@jfkthame
Copy link
Collaborator

jfkthame commented Nov 6, 2017

Ah, right - I forgot about those, sorry!

@behdad
Copy link
Member

behdad commented Nov 14, 2017

Waiting for @jfkthame to advise how to fix this.

@jfkthame
Copy link
Collaborator

Given this seems to be a valid use case, @dscorbett's fix seems reasonable to me; I think we can just merge it.

@behdad
Copy link
Member

behdad commented Nov 14, 2017

Thanks David. Can you please update the patch to insert URL to this PR in the code as well?

@@ -303,6 +304,9 @@ def map_to_use(data):
# TODO: https://github.com/behdad/harfbuzz/issues/525
if U == 0x1A7F: UISC = Consonant_Final; UIPC = Bottom

# TODO: Neither in Unicode 10.0 nor in USE's override list
Copy link
Member

Choose a reason for hiding this comment

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

Please add the PR URL here.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done.

@behdad behdad merged commit 87f0ad1 into harfbuzz:master Nov 14, 2017
@dscorbett dscorbett deleted the vedic-asterisk branch November 14, 2017 23:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants