Hi All I have attached an updated patch where I have taken care of some memory leak scenarios not handled in the previous patch(though very unlikely to occur)
Regards Ali From: Jaseem Ali K T Sent: Wednesday, September 07, 2016 11:29 PM To: podofo-users@lists.sourceforge.net Subject: RE: [Podofo-users] TTC font related issues in Windows Hi Thanks for the reply. <<<The question is, how do we know which index we want from the ttc if we have a user is asking for "Batang, bold" for example?>>> I have not looked at freetype source or spec to see how index is assigned to each font in the file. But I guess it should be in the same order as they appear in the ttc header. Since windows api can place us at the beginning of the font we want, I guess we can match the name of the font (fullname) with each of fonts in the file and obtain the index. This is what I can think of right now. (I cannot exactly be sure if this would be more or less complecated than the current patch I sent. ) The patch I sent is perfectly fine for windows as of now. Since this works fine I would suggest this to be merged now. If non-windows systems implement the part of obtaining correct face index for ttc files, the windows part also may be re-implemented for uniformity. Regards Ali From: dennis [mailto:dennis.v...@dots.de] Sent: Wednesday, August 31, 2016 12:43 PM To: podofo-users@lists.sourceforge.net<mailto:podofo-users@lists.sourceforge.net> Subject: Re: [Podofo-users] TTC font related issues in Windows Hey, Perfect! Thank you for testing this so in depth! You are right about your point of trusting the windows API; i just had too many problems working with it in other places so i'm always sceptical about it. I never used Freetype in PoDoFo (or in general), so my knowledge about it is minimal, but always passing index 0 to a ttc is definitly wrong if you want something else then ... well the first font in the ttc ^_^ The question is, how do we know which index we want from the ttc if we have a user is asking for "Batang, bold" for example? Greetings, Dennis PS: im happy to see someone else working on Fonts :-) Am 29.08.2016 um 19:28 schrieb Jaseem Ali K T: Hi Thanks for the reply I did some tests. I have a system with Japanese language. I set the system locale to Russian , Arabic etc and tested the code. I was able to get Meiryo font. I tested other fonts too and I was able to get others also. We do , after all ,depend on windows API, to get the font file buffer. If we can trust the API to give the correct file buffer(for the font), I think we can trust the same API(we are using the same function) in helping us to extract the correct font also.(The only problem is it is closed source, and we would not be able to debug. We'd just have to trust them. But , since the code is already running on windows we are already 'trusting' them many places) The pre-patch code didn't have fontstyle matching. Introducing this to pre-patch code, may require, that we have to take care of font style names in each language, locale etc. And we are not sure what other things we are going to miss. ....... May be a more refined approach would be pass the correct face index to freetype library (In place of the logic for extracting the font by adjusting offsets) . (Side Note: what I understand by reading podofo code is that, in non-windows systems, the fontmanager will give the path of the required font, and this is passed to freetype library as FT_New_Face( *pLibrary, pszFilename, 0, &m_pFace ); Here the face index is always zero. Which means we always load the first font of the ttc file, which may not be what we wanted. ) .......... Regards Ali ::DISCLAIMER:: ---------------------------------------------------------------------------------------------------------------------------------------------------- The contents of this e-mail and any attachment(s) are confidential and intended for the named recipient(s) only. E-mail transmission is not guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or may contain viruses in transmission. The e mail and its contents (with or without referred errors) shall therefore not attach any liability on the originator or HCL or its affiliates. Views or opinions, if any, presented in this email are solely those of the author and may not necessarily reflect the views or opinions of HCL or its affiliates. Any form of reproduction, dissemination, copying, disclosure, modification, distribution and / or publication of this message without the prior written consent of authorized representative of HCL is strictly prohibited. If you have received this email in error please delete it and notify the sender immediately. Before opening any email and/or attachments, please check them for viruses and other defects. ----------------------------------------------------------------------------------------------------------------------------------------------------
PdfFontCache.diff
Description: PdfFontCache.diff
------------------------------------------------------------------------------
_______________________________________________ Podofo-users mailing list Podofo-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/podofo-users