cleaned up the checkbox creation code (~70 lines of code became 3)

[wxWidgets.git] / src / os2 / fontutil.cpp

diff --git a/src/os2/fontutil.cpp b/src/os2/fontutil.cpp
index 31bbcaac3d2a7eed998ebbe9c48ef7fab8e1a915..805b4eeb9e716b6ed6efac35cfd1481ec5ff79e0 100644 (file)
--- a/src/os2/fontutil.cpp
+++ b/src/os2/fontutil.cpp
@@ -262,6 +262,7 @@ void wxFillLogFont(
   LOGFONT*                          pFattrs  // OS2 GPI FATTRS
 , PFACENAMEDESC                     pFaceName
 , HPS*                              phPS
   LOGFONT*                          pFattrs  // OS2 GPI FATTRS
 , PFACENAMEDESC                     pFaceName
 , HPS*                              phPS

+, bool*                             pbInternalPS

 , long*                             pflId
 , wxString&                         sFaceName
 , wxFont*                           pFont
 , long*                             pflId
 , wxString&                         sFaceName
 , wxFont*                           pFont


@@ -324,6 +325,9 @@ void wxFillLogFont(
 
     wxString                        sVals;
 
 
     wxString                        sVals;
 

+    //
+    // For debugging, delete later
+    //

     for (int i = 0; i < lNumFonts; i++)
     {
          sVals << "Face: " << pFM[i].szFacename
     for (int i = 0; i < lNumFonts; i++)
     {
          sVals << "Face: " << pFM[i].szFacename


@@ -419,6 +423,7 @@ void wxFillLogFont(
         vError = ::WinGetLastError(vHabmain);
     }
     sFaceName = zFacename;
         vError = ::WinGetLastError(vHabmain);
     }
     sFaceName = zFacename;

+    *pbInternalPS = bInternalPS;

 
     //
     // That's it, we now have everything we need to actually create the font
 
     //
     // That's it, we now have everything we need to actually create the font


@@ -438,12 +443,12 @@ void wxOS2SelectMatchingFontByName(
     int                             nPointSize;
     int                             nDiff;
     int                             nIs;
     int                             nPointSize;
     int                             nDiff;
     int                             nIs;

-    int                             nIndex;

     int                             nMinDiff;
     int                             nMinDiff0;
     int                             nApirc;
     int                             anDiff[16];
     int                             anMinDiff[16];
     int                             nMinDiff;
     int                             nMinDiff0;
     int                             nApirc;
     int                             anDiff[16];
     int                             anMinDiff[16];

+    int                             nIndex = 0;

     STR8                            zFn;
     char                            zFontFaceName[FACESIZE];
     wxString                        sFaceName;
     STR8                            zFn;
     char                            zFontFaceName[FACESIZE];
     wxString                        sFaceName;


@@ -462,21 +467,24 @@ void wxOS2SelectMatchingFontByName(
 
         case wxDECORATIVE:
         case wxROMAN:
 
         case wxDECORATIVE:
         case wxROMAN:

-            sFaceName = wxT("Times New Roman");
+            sFaceName = wxT("Tms Rmn");

             break;
 
         case wxTELETYPE:
             break;
 
         case wxTELETYPE:

-        case wxMODERN:

             sFaceName = wxT("Courier") ;
             break;
 
             sFaceName = wxT("Courier") ;
             break;
 

+        case wxMODERN:
+            sFaceName = wxT("System VIO") ;
+            break;
+

         case wxSWISS:
         case wxSWISS:

-            sFaceName = wxT("WarpSans") ;
+            sFaceName = wxT("Helv") ;

             break;
 
         case wxDEFAULT:
         default:
             break;
 
         case wxDEFAULT:
         default:

-            sFaceName = wxT("Helv") ;
+            sFaceName = wxT("System VIO") ;

     }
 
     switch (pFont->GetWeight())
     }
 
     switch (pFont->GetWeight())


@@ -607,16 +615,8 @@ void wxOS2SelectMatchingFontByName(
     pFattrs->lMatch         = pFM[nIndex].lMatch;      // force match
     pFattrs->idRegistry     = pFM[nIndex].idRegistry;  // uses default registry
     pFattrs->usCodePage     = pFM[nIndex].usCodePage;  // code-page
     pFattrs->lMatch         = pFM[nIndex].lMatch;      // force match
     pFattrs->idRegistry     = pFM[nIndex].idRegistry;  // uses default registry
     pFattrs->usCodePage     = pFM[nIndex].usCodePage;  // code-page

-    if(pFM[nIndex].lMatch)
-    {
-        pFattrs->lMaxBaselineExt = pFM[nIndex].lMaxBaselineExt; // requested font height
-        pFattrs->lAveCharWidth   = pFM[nIndex].lAveCharWidth ;  // requested font width
-    }
-    else
-    {
-        pFattrs->lMaxBaselineExt = 0;
-        pFattrs->lAveCharWidth   = 0;
-    }
+    pFattrs->lMaxBaselineExt = 0;   // OUTLINE fonts need this set to 0 as they use other attributes to match
+    pFattrs->lAveCharWidth   = 0;   // OUTLINE fonts need this set to 0 as they use other attributes to match

     pFattrs->fsType    = 0;// pfm->fsType;              /* uses default type       */
     pFattrs->fsFontUse = 0;
 
     pFattrs->fsType    = 0;// pfm->fsType;              /* uses default type       */
     pFattrs->fsFontUse = 0;