ping method almost there

[wxWidgets.git] / src / generic / dcpsg.cpp

diff --git a/src/generic/dcpsg.cpp b/src/generic/dcpsg.cpp
index f67a2ec4f8e3076179ebf91d3b14aea77c946889..88e33771745c459cfb924f54b387025bb3f575c8 100644 (file)
--- a/src/generic/dcpsg.cpp
+++ b/src/generic/dcpsg.cpp
@@ -1697,7 +1697,7 @@ void wxPostScriptDC::GetTextExtent( const wxString& string, long *x, long *y,
 
     const wxWX2MBbuf strbuf = string.mb_str();
 
 
     const wxWX2MBbuf strbuf = string.mb_str();
 

-#if !USE_AFM_FOR_POSTSCRIPT
+#if !wxUSE_AFM_FOR_POSTSCRIPT

     /* Provide a VERY rough estimate (avoid using it).
      * Produces accurate results for mono-spaced font
      * such as Courier (aka wxMODERN) */
     /* Provide a VERY rough estimate (avoid using it).
      * Produces accurate results for mono-spaced font
      * such as Courier (aka wxMODERN) */


@@ -1758,6 +1758,9 @@ void wxPostScriptDC::GetTextExtent( const wxString& string, long *x, long *y,
     static int lastWeight= INT_MIN;
     static int lastDescender = INT_MIN;
     static int lastWidths[256]; /* widths of the characters */
     static int lastWeight= INT_MIN;
     static int lastDescender = INT_MIN;
     static int lastWidths[256]; /* widths of the characters */

+    
+    double UnderlinePosition = 0.0;
+    double UnderlineThickness = 0.0;

 
     /* get actual parameters */
     const int Family = fontToUse->GetFamily();
 
     /* get actual parameters */
     const int Family = fontToUse->GetFamily();


@@ -1791,7 +1794,7 @@ void wxPostScriptDC::GetTextExtent( const wxString& string, long *x, long *y,
                     if ((Style == wxITALIC) && (Weight == wxBOLD)) name = "TimesBoO";
                     else if ((Style != wxITALIC) && (Weight == wxBOLD)) name = "TimesBo";
                     else if ((Style == wxITALIC) && (Weight != wxBOLD)) name = "TimesO";
                     if ((Style == wxITALIC) && (Weight == wxBOLD)) name = "TimesBoO";
                     else if ((Style != wxITALIC) && (Weight == wxBOLD)) name = "TimesBo";
                     else if ((Style == wxITALIC) && (Weight != wxBOLD)) name = "TimesO";

-                    else if name = "TimesRo";  /* no typo */
+                    else name = "TimesRo"; 

                 }
                 break;
             default:
                 }
                 break;
             default:


@@ -1810,7 +1813,7 @@ void wxPostScriptDC::GetTextExtent( const wxString& string, long *x, long *y,
        
        if (!m_printData.GetFontMetricPath().IsEmpty())
        {
        
        if (!m_printData.GetFontMetricPath().IsEmpty())
        {

-           strcpy( afmName, m_printData.GetFontMetricPath().fn_str() )
+           strcpy( afmName, m_printData.GetFontMetricPath().fn_str() );

        }
 
         /* 2. open and process the file
        }
 
         /* 2. open and process the file


@@ -1834,21 +1837,21 @@ void wxPostScriptDC::GetTextExtent( const wxString& string, long *x, long *y,
         FILE *afmFile = fopen(afmName,"r");
        
 #ifdef __UNIX__
         FILE *afmFile = fopen(afmName,"r");
        
 #ifdef __UNIX__

-       if (afmFile==NULL)
-       {
-           strcpy( afmName, "/usr/local/share/wx/gs_afm/" );
-            strcat(afmName,name);
-            strcat(afmName,".afm");
-            FILE *afmFile = fopen(afmName,"r");
-       }
-       
-       if (afmFile==NULL)
-       {
-           strcpy( afmName, "/usr/share/wx/gs_afm/" );
-            strcat(afmName,name);
-            strcat(afmName,".afm");
-            FILE *afmFile = fopen(afmName,"r");
-       }
+            if (afmFile==NULL)
+            {
+                strcpy( afmName, "/usr/local/share/wx/gs_afm/" );
+             strcat(afmName,name);
+             strcat(afmName,".afm");
+             afmFile = fopen(afmName,"r");
+            }
+
+            if (afmFile==NULL)
+            {
+                strcpy( afmName, "/usr/share/wx/gs_afm/" );
+             strcat(afmName,name);
+             strcat(afmName,".afm");
+             afmFile = fopen(afmName,"r");
+            }

 #endif
        
         if (afmFile==NULL)
 #endif
        
         if (afmFile==NULL)


@@ -1951,14 +1954,20 @@ void wxPostScriptDC::GetTextExtent( const wxString& string, long *x, long *y,
     }
 
     /* JC: calculate UnderlineThickness/UnderlinePosition */
     }
 
     /* JC: calculate UnderlineThickness/UnderlinePosition */

-    m_underlinePosition = m_underlinePosition * fontToUse->GetPointSize() / 1000.0f;
-    m_underlineThickness = m_underlineThickness * fontToUse->GetPointSize() / 1000.0f * m_scaleFactor;
+    {
+        // VS: dirty, but is there any better solution?
+        double *pt;
+        pt = (double*) &m_underlinePosition;
+        *pt = UnderlinePosition * fontToUse->GetPointSize() / 1000.0f;
+        pt = (double*) &m_underlineThickness;
+        *pt = UnderlineThickness * fontToUse->GetPointSize() / 1000.0f * m_scaleFactor;
+    }

 
     /* 3. now the font metrics are read in, calc size this
        /  is done by adding the widths of the characters in the
        /  string. they are given in 1/1000 of the size! */
 
 
     /* 3. now the font metrics are read in, calc size this
        /  is done by adding the widths of the characters in the
        /  string. they are given in 1/1000 of the size! */
 

-    long widthSum=0;
+    double widthSum=0;

     long height=Size; /* by default */
     unsigned char *p;
     for(p=(unsigned char *)strbuf; *p; p++)
     long height=Size; /* by default */
     unsigned char *p;
     for(p=(unsigned char *)strbuf; *p; p++)


@@ -1966,22 +1975,22 @@ void wxPostScriptDC::GetTextExtent( const wxString& string, long *x, long *y,
         if(lastWidths[*p]== INT_MIN)
         {
             wxLogDebug("GetTextExtent: undefined width for character '%c' (%d)\n", *p,*p);
         if(lastWidths[*p]== INT_MIN)
         {
             wxLogDebug("GetTextExtent: undefined width for character '%c' (%d)\n", *p,*p);

-            widthSum += (long)(lastWidths[' ']/1000.0F * Size); /* assume space */
+            widthSum += /*(long)*/(lastWidths[' ']/1000.0F * Size); /* assume space */

         }
         else
         {
         }
         else
         {

-            widthSum += (long)((lastWidths[*p]/1000.0F)*Size);
+            widthSum += /*(long)*/((lastWidths[*p]/1000.0F)*Size);

         }
     }
 
     /* add descender to height (it is usually a negative value) */
         }
     }
 
     /* add descender to height (it is usually a negative value) */

-    if (lastDescender!=INT_MIN)
+    if (lastDescender != INT_MIN)

     {
         height += (long)(((-lastDescender)/1000.0F) * Size); /* MATTHEW: forgot scale */
     }
 
     /* return size values */
     {
         height += (long)(((-lastDescender)/1000.0F) * Size); /* MATTHEW: forgot scale */
     }
 
     /* return size values */

-    *x = widthSum;
+    *x = (long)widthSum;

     *y = height;
 
     /* return other parameters */
     *y = height;
 
     /* return other parameters */