| Summary: | fnttosfnt: too many characters in string initializition | ||
|---|---|---|---|
| Product: | xorg | Reporter: | Alan Coopersmith <alan.coopersmith> |
| Component: | App/other | Assignee: | Xorg Project Team <xorg-team> |
| Status: | RESOLVED FIXED | QA Contact: | Xorg Project Team <xorg-team> |
| Severity: | normal | ||
| Priority: | high | CC: | jch, Peter_Dyballa, ralf |
| Version: | git | Keywords: | janitor |
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | |||
| i915 platform: | i915 features: | ||
Sorry about the phenomenal bug spam, guys. Adding xorg-team@ to the QA contact so bugs don't get lost in future. *** Bug 14629 has been marked as a duplicate of this bug. *** *** Bug 14930 has been marked as a duplicate of this bug. *** This bug is from 2005! Nobody here that could answer the simple question if we should use XVENDORNAMESHORT or extend the size? Use SHORT. This particular string MUST be exactly 16 characters long. Its value is completely arbitrary -- I don't know of any contemporary software that uses the PCLT table at all. I suggest that you avoid using a macro, and choose a literal string that is exactly 16 characters long. Really, you should not worry about the contents, and avoid using a macro, just choose a random 16-byte-long string. I suggest "Szczebrzeszyn ". (I'll just mention that I'm the original author of this code.) --Juliusz P.S. Thanks to Alan for CC-ing me. commit bbab34919743990bf87935480092f7644f8f02fc Author: James Cloos <cloos@jhcloos.com> Date: Thu Mar 13 13:19:11 2008 -0400 I’m not sure the PCLT table is needed by the fonts generated here, but use a fixed 16 char string as the initializer for the PCLT Fontname field to silence the warning (or error for some compilers) as reported in bugzilla |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.
When building the Xorg modular tree, this warning is seen in the fnttosfnt build: "write.c", line 1033: warning: 9 extra byte(s) in string literal initializer ignored That line of write.c is: char name[16] = XVENDORNAME" font "; But the modular makefile.am has these definitions: -DXVENDORNAME=\"X.org\ Foundation\" -DXVENDORNAMESHORT=\"X.org\" Most of the other references in fnttosfnt source appear to be to XVENDORNAMESHORT. Should that be used here as well or does the length of the name string need to be extended?