创建附件303828 [详细信息] [审查]此修补程序实现了Gnome Bugzilla中描述的缺少的SVG功能错误号607931其中需要对简写字体属性的支持。此修补程序实现了Gnome Bugzilla中描述的缺少的SVG功能错误号607931需要支持速记字体属性。CSS2规范中描述了速记字体属性的结构。请参见http://www.w3.org/TR/CSS2/fonts.html#font-速记.此修补程序为速记字体属性添加了一个解析器，但对带引号的字体名称有限制。一般来说，librsvg存在Gnome Bugzilla中描述的问题错误#739329这在这里也被认为是可以接受的。

审查附件303828 [详细信息] [审查]以下为：代码对我来说看起来很好，但这里有一些（非维护人员）反馈，主要基于偏好。我会把同意/不同意的最后一句话留给维护人员。：：：rsvg-styles.c@@-461,0+461,6@@+/*+用于分析声明速记字体属性的字符串的函数。它有+qouted字体系列名称的限制，它需要所有引号…还有3个。。。我想你在这里之前只需要一个，而不是之前。@@ -461,0 +461,22 @@+/*+用于分析声明速记字体属性的字符串的函数。它有+qouted字体系列名称的限制，它需要所有引号…还有19个。。。我认为在if和else分支中都保留g_free（余数）没有意义，我想它可以简单地移动到if之前。@@ -461,0 +461,42 @@+/*+用于分析声明速记字体属性的字符串的函数。它有+qouted字体系列名称的限制，它需要所有引号…还有39个。。。这可以与前面的其他内容结合起来。。。如果分支带有||，因为两者使用相同的主体，带有rsvg_css_parse_font_style。@@ -461,0 +461,62 @@+/*+用于分析声明速记字体属性的字符串的函数。它有+qouted字体系列名称的限制，它需要所有引号…还有59个。。。前面的四个if…else分支也具有相同的主体，可以使用||将这些分支前置到这个else if分支条件。

创建附件305191 [详细信息] [审查]此修补程序实现了Gnome Bugzilla中描述的缺少的SVG功能错误#607931.此修补程序实现了Gnome Bugzilla中描述的缺少的SVG功能错误号607931需要支持速记字体属性。CSS2规范中描述了速记字体属性的结构。请参见http://www.w3.org/TR/CSS2/fonts.html#font-速记.此补丁为简写字体属性添加了一个解析器，但对带引号的字体名称有限制。一般来说，librsvg存在Gnome Bugzilla中描述的问题错误#739329这在这里也被认为是可以接受的。

你好，罗伯特！谢谢你的评论。你的三个评论完全正确，我在上一个补丁中已经更改了它们。我省略了您关于将g_free（）移到if-else分支之外的第二篇评论，因为这是一个处理内存分配和指针的函数。内存上的每一个直接操作都是危险的，所以我将空闲操作和替换操作放在一起。

创建附件306795 [详细信息]测试速记字体属性的各种组合@费德里科：我看到你在提交时在我的其他补丁上添加了测试。

我认为速记属性不能用作表示属性。在svg1.1测试套件中是对用作属性（painting-marker-04-f.svg）和类似“标记”的属性此处未列出“字体”速记：https://www.w3.org/TR/SVG/attindex.html#演示属性换言之，我会放弃这个大块头：https://bugzilla.gnome.org/attachment.cgi？id=305191&action=diff#a/rsvg-样式s.c_sec3但我没有测试这个补丁。

--GitLab迁移自动消息--此错误已迁移到GNOME的GitLab实例，并已关闭，无法继续活动。您可以通过此GitLab实例链接订阅并进一步参与新的bug：https://gitlab.gnome.org/gnome/librsvg/issues/34.