7954 Commits

Author SHA1 Message Date
Khaled Hosny
014991d710
Merge pull request #1815 from khaledhosny/fealib-flag-fix
[feaLib] Don’t reset lookupflag in nested lookups
2020-01-30 13:06:42 +02:00
Khaled Hosny
42b51f084f
Merge pull request #1814 from khaledhosny/fealib-mult-fix
[feaLib] fix mixed single/multiple substitutions
2020-01-30 13:02:39 +02:00
Cosimo Lupo
39b02fd224
fix a coiple of SyntaxWarning on python 3.8 2020-01-30 10:47:57 +00:00
Khaled Hosny
eed252cb92 [feaLib] Don’t reset lookupflag in nested lookups
In makeotf, lookups defined inside feature blocks inherit the current
lookupflag of the feature, so don’t reset the lookupflag for such
lookups.
2020-01-29 23:46:40 +02:00
Khaled Hosny
22bfc305ee [feaLib] fix mixed single/multiple substitutions
If the single substitution involved a glyph class, we were incorrectly
using only the first glyph in the class.

Broken since ec6ff821f0e72022d7aec8794b6bb589d8f81808, apparently no one
else uses this feature!
2020-01-29 22:36:54 +02:00
Cosimo Lupo
8c1654397d
Bump version: 4.2.5 → 4.2.6.dev0 2020-01-29 17:39:48 +00:00
Cosimo Lupo
fa77aac63c
Release 4.2.5 2020-01-29 17:39:47 +00:00
Cosimo Lupo
786e99f5a9
Update changelog [skip ci] 2020-01-29 17:39:39 +00:00
justvanrossum
0bb234d218 added test 2020-01-29 12:42:27 +01:00
Cosimo Lupo
f609293dc7
Merge pull request #1811 from dscorbett/add_multiple_subst-duplicate
[feaLib] Do not fail on duplicate multiple substitutions
2020-01-29 11:26:41 +00:00
justvanrossum
950314a5a5 undo the more invasive changes, and only keep the change for #1808; maybe we'll add a test later 2020-01-29 12:24:03 +01:00
justvanrossum
92e03c72b2 removed leftovers from previous edit 2020-01-29 12:02:26 +01:00
justvanrossum
69800b0db5 no need to explicitly close: varLib.build() will do that. 2020-01-29 12:01:05 +01:00
justvanrossum
49507a9de8 undo DesignSpaceDocument context manager changes, and explicitly (but opt-out) close master_fonts within varLib.build() 2020-01-29 11:55:39 +01:00
justvanrossum
703c2272bd ensure we don't call close on objects that don't have it 2020-01-29 11:27:56 +01:00
justvanrossum
e18f63cfa3 make DesignSpaceDocument a context manager that closes source fonts, and use it in the varLib tests. 2020-01-29 11:17:57 +01:00
justvanrossum
51ff4a5d69 Move ensureDecompiled() calls to mergeObjects(). This makes more sense, but still needs the hasattr checks. 2020-01-29 11:05:01 +01:00
justvanrossum
033145adf6 this fixes #1808, but I'm not sure this is the most elegant solution 2020-01-29 10:10:55 +01:00
David Corbett
62ed43ddb9 Do not fail on duplicate multiple substitutions 2020-01-26 10:30:17 -05:00
justvanrossum
874e74b834 demonstrate #1808 2020-01-24 10:47:48 +01:00
Cosimo Lupo
a37dab3824
Merge pull request #1802 from fonttools/subset-optimize-single-pos
[subset] Optimize SinglePos lookup format when all values are same
2020-01-17 11:18:11 -08:00
Cosimo Lupo
53c99df8f8
subset: downgrade SinglePos to format 1 if all ValueRecords are same
Fixes https://github.com/fonttools/fonttools/issues/1709
2020-01-16 18:34:12 -08:00
Cosimo Lupo
a32fd30c2d
subset_test: add test for optimizing SinglePos format when all subsetted values are same 2020-01-16 18:30:07 -08:00
Nikolaus Waxweiler
680a271b00 Bump version: 4.2.4 → 4.2.5.dev0 2020-01-09 15:49:50 +00:00
Nikolaus Waxweiler
66a1473be6 Release 4.2.4 2020-01-09 15:49:49 +00:00
Nikolaus Waxweiler
faad7f441d
Merge pull request #1799 from fonttools/unicodedata-add-11-12-rtl
Update RTL_SCRIPTS for Unicode 11 and 12
2020-01-09 16:00:31 +01:00
Nikolaus Waxweiler
131d62f467 Update changelog
[ci skip]
2020-01-09 14:26:49 +00:00
Nikolaus Waxweiler
d381609885 Update RTL_SCRIPTS for Unicode 11 and 12
Information taken from https://docs.google.com/spreadsheets/d/1Y90M0Ie3MUJ6UVCRDOypOtijlMDLNNyyLk36T6iMu0o.
2020-01-09 14:21:49 +00:00
Cosimo Lupo
a311e76338
Bump version: 4.2.3 → 4.2.4.dev0 2020-01-07 16:28:13 +00:00
Cosimo Lupo
e2de0a36da
Release 4.2.3 2020-01-07 16:28:12 +00:00
Cosimo Lupo
81cc20ccde
Update changelog [skip ci] 2020-01-07 16:27:58 +00:00
Cosimo Lupo
be1aba59b0
Merge pull request #1797 from anthrotype/fix-split-mark-base-pos
splitMarkBasePos should update MarkRecord.Class values after splitting
2020-01-07 15:02:05 +00:00
Cosimo Lupo
504450e053
otTables: update MarkRecord.Class in splitMarkBasePos
Fixes https://github.com/googlefonts/noto-source/issues/145
2020-01-07 13:24:40 +00:00
Cosimo Lupo
58cbf7b814
otTables_test: get XML dumps for expected MarkBasePos split subtables
This tests will fail because of this bug: https://github.com/googlefonts/noto-source/issues/145

The mark records' Class values of the second subtable should be updated to match the new class count
(the two split subtable contain half of the original mark classes). Otherwise the base records would
implicitly reference the wrong or non-existent mark classes...
The fix is in the following commit.
2020-01-07 12:57:20 +00:00
Just van Rossum
c6d005131c
Merge pull request #1796 from anntzer/sw
Get rid of SyntaxWarning with py3.8.
2020-01-06 11:57:09 +01:00
Antony Lee
4fa68d1989 Get rid of SyntaxWarning with py3.8.
On a clean clone (make sure there are no pycs present yet),
```
python -c 'import fontTools.ttLib.tables._c_m_a_p'
```
results in
```
.../fontTools/ttLib/tables/_c_m_a_p.py:21: SyntaxWarning: "is" with a literal. Did you mean "=="?
  if gid is 0:
```

Fix that.
2020-01-06 11:30:30 +01:00
Ben Kiel
726cd67549
Clarify differences between XML and Python (#1792)
* Clarify differences between XML and Python

* Typo
2019-12-29 10:09:23 -06:00
Cosimo Lupo
fbf2b960cc
Merge pull request #1767 from fonttools/fealib-duplicate-sub-warning
[feaLib] Warning instead of error for duplicate identical substitutions
2019-12-18 13:23:50 +00:00
Jens Kutilek
864945df0b Change log call 2019-12-18 14:12:02 +01:00
Jens Kutilek
720488ed4b Merge branch 'master' into fealib-duplicate-sub-warning 2019-12-18 11:24:00 +01:00
Jens Kutilek
67da54fb5f Add a simple test 2019-12-18 11:09:23 +01:00
Jens Kutilek
b858f7bb3e Change log message, remove comments 2019-12-18 10:03:08 +01:00
Cosimo Lupo
251fd70b77
Merge pull request #1785 from anthrotype/glifLib-remove-comments
glifLib: ignore xml <!-- comments --> when parsing GLIF files with lxml
2019-12-16 12:19:09 +00:00
Cosimo Lupo
337bb66211
glifLib: strip comments when parsing with lxml
they are already ignored when parsing via built-in ElementTree
2019-12-16 12:05:13 +00:00
Cosimo Lupo
70c3eccb4e
glifLib_test: test xml containing <!-- comment --> 2019-12-16 12:01:40 +00:00
Cosimo Lupo
fd606ae813
Bump version: 4.2.2 → 4.2.3.dev0 2019-12-12 16:48:53 +00:00
Cosimo Lupo
632e065bcd
Release 4.2.2 2019-12-12 16:48:52 +00:00
Cosimo Lupo
0649663ebc
Update changelog 2019-12-12 16:48:42 +00:00
Cosimo Lupo
3b9a94d659
Merge pull request #1782 from anthrotype/subset-feature-vars-remap-indices
[subset] remap FeatureVariations SubstitutionRecord.FeatureIndex
2019-12-12 13:51:59 +00:00
Just van Rossum
cf0e43d6e5
[ttLib.glyf] make glyph.draw() skip redundant final lineTo() (#1775)
* [ttLib.glyf] make glyph.draw() skip redundant final lineTo()

This ensures that g.draw(pen) and g.drawPoints(PointToSegmentPen(pen)) are now 100% equivalent.
2019-12-12 13:44:10 +01:00