Patches item #3421209, was opened at 2011-10-10 17:54 Message generated for change (Comment added) made by cybevnm You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=790132&aid=3421209&group_id=154028
Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: cybevnm (cybevnm) Assigned to: Nobody/Anonymous (nobody) Summary: GetMultiLineTextAsLines unicode support Initial Comment: Patch add support for unicode-encoded (UTF8, UTF16) PdfString in GetMultiLineTextAsLines (and consequently for DrawMultilineText) at expence of lower performance and slightly changed PdfPainter interface. New GetMultiLineTextAsLines converts input to utf8. In next step utf8 string converted to utf16 string which allows us easily iterate through it. Utf8 step required only to reduce input in variable encodings to a common denominator.. Also patch has some fixes to improve support for USE_CXX_LOCALE ---------------------------------------------------------------------- Comment By: cybevnm (cybevnm) Date: 2011-10-11 16:42 Message: I fixed problem. Use unicode_multiline_fix_2.patch. Actually I not absolutelly sure that it has not big-little endian issues, so look at result a while please. This patch also contains small fix: in PdfPainter code in some places was added m_pFont->EmbedFont(); line. I think that it shouldn't be here, so I commented this code out. If you know purpose for this code you can enable it again, or completelly remove it otherwise. Separated CXX_USE_LOCALE improvement can be found in no_locale_header_improv.patch I think that it is pretty confusing to manipulate UTF-16 strings in podofo. Because of using explicit big-endian format the way of interpretion of each char in string depends on platform endianess (on BE machines we can use pdf16be array as is, but on LE machines we need to swap bytes before doing any computation with it), but I can be wrong of course. NOTE: please use unicode_multiline_fix_2.patch, not unicode_multiline_fix.patch because latter has some potential bugs ---------------------------------------------------------------------- Comment By: Dominik Seichter (domseichter) Date: 2011-10-10 21:46 Message: Hi, Sorry, your patch gives me different results than I get with out the patch. Please look at page 2 of the two attached PDFs (both generated by the same code, see new version of CreationTest in SVN). The version with your patch does not use the full width of the page! Can you fix the issues in your patch? I think it is very intersting and I also lile the CXX_USE_LOCALE patch (maybe you can provide a seperate patch for that?). Cheers, Dominik ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=790132&aid=3421209&group_id=154028 ------------------------------------------------------------------------------ All the data continuously generated in your IT infrastructure contains a definitive record of customers, application performance, security threats, fraudulent activity and more. Splunk takes this data and makes sense of it. Business sense. IT sense. Common sense. http://p.sf.net/sfu/splunk-d2d-oct _______________________________________________ Podofo-users mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/podofo-users
