Fixed various C90 compliance issues : variables declared after code instead of at the top of a block, true/false used instead of TRUE/FALSE, C++-style comments

Author: Bill-Gray

fb/drm.c

@@ -81,6 +81,8 @@ int get_framebuffer(const char *dri_device, const int connector_num, struct fram
     drmModeResPtr res;
     drmModeEncoderPtr encoder = 0;
     drmModeConnectorPtr connector = 0;
+    drmModeModeInfoPtr resolution = 0;
+    struct drm_mode_map_dumb mreq;
 
     /* Open the dri device /dev/dri/cardX */
     fd = open(dri_device, O_RDWR);
@@ -126,8 +128,7 @@ int get_framebuffer(const char *dri_device, const int connector_num, struct fram
         return FRAMEBUFFER_COULD_NOT_FIND_CONNECTOR;
 
     /* Get the preferred resolution */
-    drmModeModeInfoPtr resolution = 0;
-    for (int i = 0; i < connector->count_modes; i++) {
+    for( i = 0; i < connector->count_modes; i++) {
             resolution = &connector->modes[i];
             if (resolution->type & DRM_MODE_TYPE_PREFERRED)
                     break;
@@ -165,8 +166,6 @@ int get_framebuffer(const char *dri_device, const int connector_num, struct fram
     /* Get the crtc settings */
     fb->crtc = drmModeGetCrtc(fd, encoder->crtc_id);
 
-    struct drm_mode_map_dumb mreq;
-
     memset(&mreq, 0, sizeof(mreq));
     mreq.handle = fb->dumb_framebuffer.handle;
 

x11new/pdckbd.c

@@ -390,8 +390,8 @@ static int _modifier_key( KeySym *key)
 static bool check_key( int *c)
 {
    XEvent report;
-   bool rval = false;
-   bool wait_for_more_mouse = false;
+   bool rval = FALSE;
+   bool wait_for_more_mouse = FALSE;
 
 #ifndef NO_LEAKS
    if( !_xim)
@@ -419,13 +419,13 @@ static bool check_key( int *c)
             new_rows = attrib.height / PDC_font_height;
             if( new_cols != PDC_cols || new_rows != PDC_rows)
                {
-               bool resize_not_in_queue = true;
+               bool resize_not_in_queue = TRUE;
                int i;
 
                PDC_resize_screen( new_rows, new_cols);
                for( i = queue_low; i != queue_high; i = (i + 1) % QUEUE_SIZE)
                   if( key_queue[i] == KEY_RESIZE)
-                     resize_not_in_queue = false;
+                     resize_not_in_queue = FALSE;
                if( resize_not_in_queue)
                   {
                   SP->resized = TRUE;

x11new/pdcscrn.c

@@ -173,7 +173,7 @@ int PDC_scr_open(void)
     assert( win);
     wmDeleteMessage = XInternAtom( dis, "WM_DELETE_WINDOW", True);
 
-    //Allocate space for the hints
+    /* Allocate space for the hints */
     xshSize = XAllocSizeHints();
     xwmhHints = XAllocWMHints();
     xchClass = XAllocClassHint();

x11new/pdcsetsc.c

@@ -78,10 +78,9 @@ int PDC_set_bold(bool boldon)
 
 void PDC_set_title( const char *title)
 {
-    PDC_LOG(("PDC_set_title() - called:<%s>\n", title));
-
     char *temp_string = strdup( title);
 
+    PDC_LOG(("PDC_set_title() - called:<%s>\n", title));
     if( xtpWinName.value)
         XFree( xtpWinName.value);
     XStringListToTextProperty( &temp_string, 1, &xtpWinName);