Created attachment 57915 [details] [review] list font encodings This patch makes pdffonts include the font encoding in the list of fonts.
GfxCIDFont::GfxCIDFont has some goto in case of error, that will result in encodingName = NULL; and encoding = font->getEncodingName()->copy(); crashing
Created attachment 57952 [details] [review] list font encodings I've updated the patch to initialize the encoding to an empty string in the GfxFont constructor so it will always be set to something. Would initializing it to something like "Unknown" or "???" be better or should I just leave it blank?
Don't have a strong opinion on that, do whatever you prefer. You have a indentation mismatch with + font->getEncoding()->getCString(), and the rest of the lines around it (tab vs spaces), yours is probably using the "right" poppler encoding, but please change it so that it uses what the lines above and below it use. Then feel free to commit.
pushed
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.