[Date Prev][Date Next][Thread Prev][Thread Next][Thread Index]

Re: [XaraXtreme-dev] Text rendering implementation (3)

In message <442CEB48.5060808@xxxxxxxx>
          Phil Martin <phil@xxxxxxxx> wrote:

> Martin Wuerthner wrote:
>>Oops, sorry, I only tried a debug build here, and the non-debug build
>>fails indeed. The problem is a debugging msg function that I forgot to
>>define for non-debug mode.
> Yes, I was doing a Debug build too (developer habit). Moral: When
> building to test new contributions, always do Release builds.
> I'll wait for your further patch, Martin, before checking any more in.

Find the patch attached below. Please excuse the inconvenience.

Index: wxOil/ftfonts.cpp
--- wxOil/ftfonts.cpp	(Revision 748)
+++ wxOil/ftfonts.cpp	(Arbeitskopie)
@@ -200,7 +200,7 @@
 	Author: 	Mike_Kenny (Xara Group Ltd) <camelotdev@xxxxxxxx>
 	Created:	01/9/95
-	Returns:	TRUE if the TrueType mangler is actually running and can be used on this OpSys
+	Returns:	TRUE if the FreeType manager is actually running and can be used on this OpSys
 	Purpose:	Determine whether we can make further calls to the TrueType Manager
@@ -212,13 +212,11 @@
 // debugging routine to conveniently output a String_64 string
-#ifdef _DEBUG
 static void DumpString64User(char* user, TCHAR* msg, String_64* pString)
 	TRACEUSER(user, msg);
 	TRACEUSER(user, (TCHAR*)(*pString));  // use TCHAR* conversion operator