コミット: 7e79c030be542ca5e2c00e69a3395780418ec4f7 - luatexja (git)

ソースコードの管理場所

コミットメタ情報

• リビジョン: 7e79c030be542ca5e2c00e69a3395780418ec4f7 (tree)
• 日時: 2019-09-14 08:45:34
• 作者: Hironori Kitagawa <h_kitagawa2001@yaho...>
• コミッター: Hironori Kitagawa

ログメッセージ

jpotf was broken in recent commits

変更サマリ

• modified: src/ltj-jfont.lua
• modified: src/ltj-lotf_aux.lua

差分

--- a/src/ltj-jfont.lua
+++ b/src/ltj-jfont.lua
@@ -854,22 +854,7 @@ do
      [0x300C]=0xFE41, [0x300D]=0xFE42, [0x300E]=0xFE43, [0x300F]=0xFE44,
      [0xFF3B]=0xFE47, [0xFF3D]=0xFE48, 
   }
-  local vert_jpotf_table = {}
-  local function add_vform(i, k, vform, ft, add_vert, jpotf_vert)
-    if type(coverage)~='table' then return end
-    for i,v in pairs(vert_form_table) do
-       if not coverage[i] and ft.characters[v] then vform[i] = v end
-    end
-    if jpotf_vert then
-      for i,v in pairs(vert_jpotf_table) do
-	if ft.characters[v] then  vform[i] = coverage[v] or vform[v] or v end
-      end
-    end
-    if add_vert then -- vert feature が有効にならない場合
-      for i,v in pairs(coverage) do vform[i] = vform[i] or v end
-    end
-  end
-
+  local vert_jpotf_table, vert_feat = {}, {vert=true}
   local utfbyte, utfsub = utf.byte, utf.sub
   luatexja.jfont.register_vert_replace = function(t)
     for i,v in pairs(t) do
@@ -887,20 +872,25 @@ luatexbase.add_to_callback(
    function (fmtable, fnum)
       local vform = {}; fmtable.vform = vform
       local t = font_getfont(fnum)
-      if t.specification and t.resources then
-         local add_vert = not ltju.exist_feature(fnum, 'vert') and not ltju.exist_feature(fnum, 'vrt2')
-         local jpotf_vert = ltju.specified_feature(fnum, 'jpotf')
-         -- 現在の language, script で vert もvrt2 も有効にできない場合，強制的に vert 適用
-         for _,i in pairs(t.resources.sequences) do
-            if i.order[1]== 'vert' and i.type == 'gsub_single' and i.steps then
-               for _,j in pairs(i.steps) do
-                  if type(j)=='table' then
-		      add_vform(j.coverage,vform, t, add_vert, jpotf_vert)
-		  end
-               end
-            end
-          end
+      if not t then return fmtable end
+      for i,v in pairs(vert_form_table) do
+	if t.characters[v] then vform[i] = v end
+      end
+      if ltju.specified_feature(fnum, 'jpotf') then
+        for i,v in pairs(vert_jpotf_table) do
+          if t.characters[v] then vform[i] = vform[v] or v end
+        end
+      end
+      if not ltju.exist_feature(fnum, 'vert') and not ltju.exist_feature(fnum, 'vrt2') then
+        -- 現在の (script, lang) で vert もvrt2 も有効にできない場合，
+	-- 全 (script,lang) の vert を強制的に適用
+        ltju.loop_over_feat(t, vert_feat, function (i,k) vform[i] = vform[i] or k end, true)
       end
+      -- vform の中身を vert 適用結果に変える
+      ltju.loop_over_feat(t, vert_feat,
+        function (i,k)
+          for j,w in pairs(vform) do if w==k then vform[j]=nil elseif w==i then vform[j] = k end end
+        end)
       return fmtable
    end, 'ltj.get_vert_form', 1
 )

--- a/src/ltj-lotf_aux.lua
+++ b/src/ltj-lotf_aux.lua
@@ -83,16 +83,17 @@ local function loop_over_duplicates(id, func)
 end
 aux.loop_over_duplicates = loop_over_duplicates
 
-local function loop_over_feat(id, feature_name, func)
+local function loop_over_feat(id, feature_name, func, universal)
 -- feature_name: like { vert=true, vrt2 = true, ...}
 -- func: return non-nil iff abort this fn
+-- universal: true iff look up all (script, lang) pair
   local t = (type(id)=="table") and id or getfont(id)
   if t and t.resources and t.resources.sequences then
     for _,i in pairs(t.resources.sequences) do
       if i.order[1] and feature_name[i.order[1]] then
         local f = i.features and i.features[i.order[1]]
         if i.type == 'gsub_single' and i.steps 
-          and f and f[t.properties.script] and f[t.properties.script][t.properties.language] then
+          and f and (universal or (f[t.properties.script] and f[t.properties.script][t.properties.language])) then
           for _,j in pairs(i.steps) do
             if type(j)=='table' then 
               if type(j.coverage)=='table' then