Changeset 2627

Show
Ignore:
Timestamp:
08/14/07 17:23:34
Author:
murrayc
Message:

2007-08-14 Murray Cumming <murrayc@murrayc.com>

* libtinymail-camel/tny-camel-folder.c:
(tny_camel_folder_transfer_msgs_default): Remove unused err variable
to fix a compiler warning.
* libtinymail-maemo/tny-maemo-conic-device.c: Formatting/syntax
changes requested by Philip.

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • trunk/ChangeLog

    r2623 r2627  
     12007-08-14  Murray Cumming  <murrayc@murrayc.com> 
     2 
     3        * libtinymail-camel/tny-camel-folder.c: 
     4        (tny_camel_folder_transfer_msgs_default): Remove unused err variable  
     5        to fix a compiler warning. 
     6        * libtinymail-maemo/tny-maemo-conic-device.c: Formatting/syntax  
     7        changes requested by Philip. 
     8 
    192007-08-14  Philip Van Hoof  <pvanhoof@gnome.org> 
    210 
  • trunk/libtinymail-camel/tny-camel-folder.c

    r2626 r2627  
    40014001        gint from = 0, to = 0, ufrom = 0, uto = 0; 
    40024002        TnyCamelFolderPriv *priv_src, *priv_dst; 
    4003         gboolean on_err = FALSE; 
    40044003 
    40054004        priv_src = TNY_CAMEL_FOLDER_GET_PRIVATE (self); 
  • trunk/libtinymail-maemo/tny-maemo-conic-device.c

    r2620 r2627  
    2828#include <gdk/gdk.h> /* For GDK_THREAD_ENTER/LEAVE */ 
    2929#include <string.h> /* For strcmp() */ 
    30 #include <stdio.h> /* for printf */ 
    3130 
    3231#ifdef MAEMO_CONIC_DUMMY  
     
    124123        TnyMaemoConicDevice *self; 
    125124        TnyMaemoConicDevicePriv *priv; 
     125        gboolean status_before = FALSE; 
     126         
    126127        g_return_if_fail (TNY_IS_DEVICE(device)); 
    127128        self = TNY_MAEMO_CONIC_DEVICE (device); 
    128129        priv = TNY_MAEMO_CONIC_DEVICE_GET_PRIVATE (self); 
    129130 
    130         const gboolean status_before = tny_maemo_conic_device_is_online (device); 
     131        status_before = tny_maemo_conic_device_is_online (device); 
    131132 
    132133        priv->forced = FALSE; 
     
    234235dummy_con_ic_connection_connect_by_id (TnyMaemoConicDevice *self, const gchar* iap_id) 
    235236{ 
     237        int response = 0; 
     238         
    236239        /* Show a dialog, because libconic would show a dialog here, 
    237240         * and give the user a chance to refuse a new connection, because libconic would allow that too. 
     
    240243                        GTK_MESSAGE_QUESTION, GTK_BUTTONS_OK_CANCEL,  
    241244                        "TnyMaemoConicDevice fake scratchbox implementation:\nThe application requested a connection. Make a fake connection?")); 
    242         printf ("DEBUG: %s before gtk_dialog_run()\n", __FUNCTION__); 
    243         const int response = gtk_dialog_run (dialog); 
    244         printf ("DEBUG: %s after gtk_dialog_run()\n", __FUNCTION__); 
     245 
     246        response = gtk_dialog_run (dialog); 
    245247        gtk_widget_hide (GTK_WIDGET (dialog)); 
    246248        gtk_widget_destroy (GTK_WIDGET (dialog)); 
    247249         
    248250        if (response == GTK_RESPONSE_OK) { 
     251                GError* error = NULL; 
    249252                /* Make a connection, by setting a name in our dummy text file, 
    250253                 * which will be read later: */ 
    251254                gchar *filename = get_dummy_filename (); 
    252255                 
    253                 GError* error = 0; 
    254256                g_file_set_contents (filename, "debug id0", -1, &error); 
    255257                if(error) { 
    256                         printf("%s: error from g_file_set_contents(): %s\n", __FUNCTION__, error->message); 
     258                        g_warning("%s: error from g_file_set_contents(): %s\n", __FUNCTION__, error->message); 
    257259                        g_error_free (error); 
    258260                        error = NULL; 
     
    288290        #ifdef MAEMO_CONIC_DUMMY  
    289291        return dummy_con_ic_connection_connect_by_id (self, iap_id); 
    290         #endif /* MAEMO_CONIC_DUMMY */ 
    291          
    292         TnyMaemoConicDevicePriv *priv
     292        #else 
     293        TnyMaemoConicDevicePriv *priv = NULL; 
     294        gboolean request_failed = FALSE
    293295 
    294296        g_return_val_if_fail (TNY_IS_DEVICE(self), FALSE); 
     
    302304        priv->loop = g_main_loop_new(NULL, FALSE /* not running immediately. */); 
    303305 
    304         gboolean request_failed = FALSE; 
    305306        if (iap_id) { 
    306307                if (!con_ic_connection_connect_by_id (priv->cnx, iap_id, CON_IC_CONNECT_FLAG_NONE)) { 
     
    333334 
    334335        return priv->is_online; 
     336        #endif /* MAEMO_CONIC_DUMMY */ 
    335337} 
    336338 
     
    375377#endif /* MAEMO_CONIC_DUMMY*/ 
    376378 
    377         printf ("DEBUG: %s: end.\n", __FUNCTION__); 
    378379        return TRUE; 
    379380} 
     
    384385static gboolean on_dummy_connection_check (gpointer user_data) 
    385386{ 
    386         TnyMaemoConicDevice *self = TNY_MAEMO_CONIC_DEVICE (user_data); 
    387         TnyMaemoConicDevicePriv *priv = TNY_MAEMO_CONIC_DEVICE_GET_PRIVATE (self); 
     387        TnyMaemoConicDevice *self = NULL; 
     388        TnyMaemoConicDevicePriv *priv = NULL; 
     389        gchar *filename = NULL; 
     390        gchar *contents = NULL; 
     391        GError* error = NULL; 
     392        gboolean test = FALSE; 
     393                 
     394        self = TNY_MAEMO_CONIC_DEVICE (user_data); 
     395 
     396        priv = TNY_MAEMO_CONIC_DEVICE_GET_PRIVATE (self); 
     397         
    388398        /* Check whether the enviroment variable has changed,  
    389399         * so we can fake a connection change: */ 
    390         gchar *filename = get_dummy_filename (); 
    391         gchar *contents = 0; 
    392         GError* error = 0; 
    393         gboolean test = g_file_get_contents (filename, &contents, NULL, &error); 
     400        filename = get_dummy_filename (); 
     401         
     402        test = g_file_get_contents (filename, &contents, NULL, &error); 
    394403 
    395404        if(error) { 
    396                 /* printf("%s: error from g_file_get_contents(): %s\n", __FUNCTION__, error->message); */ 
     405                /* g_debug("%s: error from g_file_get_contents(): %s\n", __FUNCTION__, error->message); */ 
    397406                g_error_free (error); 
    398407                error = NULL; 
     
    400409         
    401410        if (!test || !contents) { 
    402                 /* printf ("DEBUG1: %s: priv->iap = %s\n", priv->iap); */ 
     411                /* g_debug ("DEBUG1: %s: priv->iap = %s\n", priv->iap); */ 
    403412                /* Default to the first debug connection: */ 
    404413                contents = g_strdup ("debug id0"); 
     
    419428                if (strcmp (priv->iap, MAEMO_CONIC_DUMMY_IAP_ID_NONE) == 0) { 
    420429                        priv->is_online = FALSE; 
    421                         printf ("DEBUG: TnyMaemoConicDevice: %s:\n  Dummy connection changed to no connection.\n", __FUNCTION__); 
     430                        g_debug ("DEBUG: TnyMaemoConicDevice: %s:\n  Dummy connection changed to no connection.\n", __FUNCTION__); 
    422431                } else { 
    423432                        priv->is_online = TRUE; 
    424                         printf ("DEBUG: TnyMaemoConicDevice: %s:\n  Dummy connection changed to '%s\n", __FUNCTION__, priv->iap); 
     433                        g_debug ("DEBUG: TnyMaemoConicDevice: %s:\n  Dummy connection changed to '%s\n", __FUNCTION__, priv->iap); 
    425434                } 
    426435                 
    427                 printf ("DEBUG1: %s: emitting is_online=%d\n", __FUNCTION__, priv->is_online); 
     436                g_debug ("DEBUG1: %s: emitting is_online=%d\n", __FUNCTION__, priv->is_online); 
    428437 
    429438                conic_emit_status (TNY_DEVICE (self), priv->is_online); 
     
    484493tny_maemo_conic_device_get_iap (TnyMaemoConicDevice *self, const gchar *iap_id) 
    485494{ 
     495        #ifdef MAEMO_CONIC_DUMMY 
     496        ConIcIap *iap = NULL; 
     497        #else 
     498        TnyMaemoConicDevicePriv *priv = NULL; 
     499        #endif 
     500         
    486501        g_return_val_if_fail (TNY_IS_MAEMO_CONIC_DEVICE(self), NULL); 
    487502        g_return_val_if_fail (iap_id, NULL); 
     
    490505        /* Note that we have re-declared the private struct so that we  
    491506         * can do this, which is very bad and fragile: */ 
    492         ConIcIap *iap = g_object_new (CON_IC_TYPE_IAP, NULL); 
     507        iap = g_object_new (CON_IC_TYPE_IAP, NULL); 
    493508        iap->id = g_strdup(iap_id); 
    494509        iap->name = g_strdup_printf("%s name", iap->id); 
     
    521536tny_maemo_conic_device_get_iap_list (TnyMaemoConicDevice *self) 
    522537{ 
     538#ifdef MAEMO_CONIC_DUMMY 
     539        GSList* result = NULL; 
     540        int i = 0; 
     541#else 
     542        TnyMaemoConicDevicePriv *priv = NULL; 
     543#endif /* MAEMO_CONIC_DUMMY */ 
     544         
    523545        g_return_val_if_fail (TNY_IS_MAEMO_CONIC_DEVICE(self), NULL); 
    524546 
     
    529551         * So we return a fake list so we can exercise funcationality  
    530552         * that uses connections: */ 
    531          GSList* result = NULL; 
    532           
    533          int i = 0; 
    534553         for (i = 0; i < 10; ++i) { 
    535554                /* con_ic_iap_new (id) would checkc for a gconf dir and  
     
    586605        TnyMaemoConicDevice *self; 
    587606        TnyMaemoConicDevicePriv *priv; 
     607        gboolean already_online = FALSE; 
    588608        g_return_if_fail (TNY_IS_DEVICE(device)); 
    589609        self = TNY_MAEMO_CONIC_DEVICE (device); 
    590610        priv = TNY_MAEMO_CONIC_DEVICE_GET_PRIVATE (self); 
    591611         
    592         const gboolean already_online = tny_maemo_conic_device_is_online (device); 
     612        already_online = tny_maemo_conic_device_is_online (device); 
    593613 
    594614        g_message (__FUNCTION__); 
     
    609629        TnyMaemoConicDevice *self; 
    610630        TnyMaemoConicDevicePriv *priv; 
     631        gboolean already_offline = FALSE; 
    611632        g_return_if_fail (TNY_IS_DEVICE(device)); 
    612633        self = TNY_MAEMO_CONIC_DEVICE (device); 
     
    617638#endif /*MAEMO_CONIC_DUMMY*/ 
    618639 
    619         const gboolean already_offline = !tny_maemo_conic_device_is_online (device); 
     640        already_offline = !tny_maemo_conic_device_is_online (device); 
    620641 
    621642        priv->forced = TRUE; 
     
    632653 
    633654        #ifdef MAEMO_CONIC_DUMMY 
    634         on_dummy_connection_check(self); 
     655        on_dummy_connection_check (self); 
    635656        #endif /* MAEMO_CONIC_DUMMY */ 
    636657         
     
    703724tny_maemo_conic_device_finalize (GObject *obj) 
    704725{ 
    705         printf ("DEBUG: %s\n", __FUNCTION__); 
    706  
    707         TnyMaemoConicDevicePriv *priv; 
    708         priv   = TNY_MAEMO_CONIC_DEVICE_GET_PRIVATE (obj); 
     726        TnyMaemoConicDevicePriv *priv = TNY_MAEMO_CONIC_DEVICE_GET_PRIVATE (obj); 
    709727         
    710728        #ifndef MAEMO_CONIC_DUMMY