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

Dealing with XML #108

Merged
merged 2 commits into from
Jan 28, 2018
Merged

Dealing with XML #108

merged 2 commits into from
Jan 28, 2018

Conversation

ForNeVeR
Copy link
Owner

I finally was able to generate most of the XML data from our font files. There're still some strange contradictions, but at least now they're documented!

Next step: #107.

Closes #70.

Copy link
Collaborator

@alexreg alexreg left a comment

Choose a reason for hiding this comment

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

Good job documenting all this! Just some small improvements to the language.


This document describes the correspondence between the font files we use and
data in `DefaultTexFont.xml`. The conrete approach (e.g. script files to
regenerate XML data) wasn't reproduced yet, although we were able to get nearly
Copy link
Collaborator

Choose a reason for hiding this comment

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

hasn't been reproduced yet *

======================================================

This document describes the correspondence between the font files we use and
data in `DefaultTexFont.xml`. The conrete approach (e.g. script files to
Copy link
Collaborator

Choose a reason for hiding this comment

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

i.e. * (I think you mean?)


Data in our XML files is often inaccurate and/or seemlingly generated from other
TTF files than the files in the repository. Be prepared for that. When in doubt,
refer to [the original tfm files][tfm] (they seem to correspond to our data
Copy link
Collaborator

Choose a reason for hiding this comment

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

the original TFM files *

refer to [the original tfm files][tfm] (they seem to correspond to our data
better than the TFM files generated from our TTF files).

There're two helpful tool sets to work with our TTF files:
Copy link
Collaborator

Choose a reason for hiding this comment

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

There are *
toolsets *


`ttf2tfm` and `tftopl` are parts of the standard TeX distribution (at least
TeX Live).
2. It may be helpful to use [ttx][] utility to get some additional information
Copy link
Collaborator

Choose a reason for hiding this comment

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

the ttx utility *

`CHARDP` corresponds to `depth`, and `CHARIC` corresponds to `italic`. Any of
them could be ommited.

Kerning section could also be reconstructed from the `*.tpl` file.
Copy link
Collaborator

Choose a reason for hiding this comment

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

The kerning section can also be... *


Kerning section could also be reconstructed from the `*.tpl` file.

How to know that `O 177` is the same as `code="196"`? To do that, first look
Copy link
Collaborator

Choose a reason for hiding this comment

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

How can we know... *

```

It means that `O 177` should be character `0xa8` or `168`, not `196`. The reason
for that contradiction is currently unknown.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think "inconsistency" is a better word here.

@ForNeVeR
Copy link
Owner Author

Thanks a lot for your review, Alex!

I'll fix the issues tomorrow.

@ForNeVeR ForNeVeR merged commit a017584 into master Jan 28, 2018
@ForNeVeR ForNeVeR deleted the docs/fonts branch January 28, 2018 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants