Changes in / [881fd4e:1b5ab36]

Files:

• conf.c (modified) (1 diff)
• ipc.c (modified) (2 diffs)
• irc.c (modified) (1 diff)
• protocols/jabber/jabber.c (modified) (1 diff)
• protocols/jabber/xmlparse.c (modified) (4 diffs)
• protocols/nogaim.c (modified) (1 diff)
• protocols/oscar/oscar_util.c (modified) (2 diffs)
• protocols/oscar/service.c (modified) (2 diffs)
• set.c (modified) (1 diff)

conf.c

@@ -95 +95 @@
                         conf->port = i;
                 }
-                else if( opt == 'P' )
+                else if( opt == 'p' )
                 {
                         g_free( conf->pidfile );

ipc.c

@@ -507 +507 @@
         }
 
-        if (bind(serversock, (struct sockaddr *)&un_addr, sizeof(un_addr)) == -1) {
+        if (bind(serversock, &un_addr, sizeof(un_addr)) == -1) {
                 log_message( LOGLVL_WARNING, "Unable to bind UNIX socket to %s: %s", IPCSOCKET, strerror(errno) );
                 return 0;
@@ -571 +571 @@
         ipc_to_children_str( "OPERMSG :New BitlBee master process started (version " BITLBEE_VERSION ")\r\n" );
         
-        fclose( fp );
         return 1;
 }

irc.c

@@ -557 +557 @@
         
         u = user_find( irc, irc->mynick );
-        is_private = u->is_private;
+        if( u ) is_private = u->is_private;
         
         va_start( params, format );

protocols/jabber/jabber.c

@@ -1549 +1549 @@
         if(jd->gjc != NULL) {
                 gjab_delete(jd->gjc);
-                /* YAY for modules with their own memory pool managers!...
                 g_free(jd->gjc->sid);
-                And a less sarcastic yay for valgrind. :-) */
                 jd->gjc = NULL;
         }

protocols/jabber/xmlparse.c

@@ -1461 +1461 @@
 s = protocolEncodingName;
 #endif
-    if (ns ? XmlInitEncodingNS(&initEncoding, &encoding, s) : XmlInitEncoding(&initEncoding, &encoding, s))
+    if ((ns ? XmlInitEncodingNS : XmlInitEncoding)(&initEncoding, &encoding, s))
         return XML_ERROR_NONE;
     return handleUnknownEncoding(parser, protocolEncodingName);
@@ -1475 +1475 @@
     int standalone = -1;
     if (!(ns
-            ? XmlParseXmlDeclNS(isGeneralTextEntity,
+            ? XmlParseXmlDeclNS
+            : XmlParseXmlDecl)(isGeneralTextEntity,
                                encoding,
                                s,
@@ -1483 +1484 @@
                                &encodingName,
                                &newEncoding,
-                               &standalone)
-            : XmlParseXmlDecl(isGeneralTextEntity,
-                               encoding,
-                               s,
-                               next,
-                               &eventPtr,
-                               &version,
-                               &encodingName,
-                               &newEncoding,
-                               &standalone)))
+                               &standalone))
         return XML_ERROR_SYNTAX;
     if (!isGeneralTextEntity && standalone == 1)
@@ -1545 +1537 @@
             }
             enc = (ns
-                   ? XmlInitUnknownEncodingNS(unknownEncodingMem,
+                   ? XmlInitUnknownEncodingNS
+                   : XmlInitUnknownEncoding)(unknownEncodingMem,
                                              info.map,
                                              info.convert,
-                                             info.data)
-                   : XmlInitUnknownEncoding(unknownEncodingMem,
-                                             info.map,
-                                             info.convert,
-                                             info.data));
+                                             info.data);
             if (enc) {
                 unknownEncodingData = info.data;

protocols/nogaim.c

@@ -612 +612 @@
                         return;
                 }
-                /* Why did we have this here....
-                return; */
+                return;
         }
         

protocols/oscar/oscar_util.c

@@ -109 +109 @@
 
         curPtr = sn;
-        while ( (*curPtr) != (char) '\0') {
+        while ( (*curPtr) != (char) NULL) {
                 if ((*curPtr) != ' ')
                 i++;
@@ -140 +140 @@
         curPtr1 = sn1;
         curPtr2 = sn2;
-        while ( (*curPtr1 != (char) '\0') && (*curPtr2 != (char) '\0') ) {
+        while ( (*curPtr1 != (char) NULL) && (*curPtr2 != (char) NULL) ) {
                 if ( (*curPtr1 == ' ') || (*curPtr2 == ' ') ) {
                         if (*curPtr1 == ' ')

protocols/oscar/service.c

@@ -881 +881 @@
 #endif
 
-/* len can't be 0 here anyway...
                 } else if ((offset == 0x00001000) && (len == 0x00000000)) {
 
@@ -888 +887 @@
                         aimbs_put32(&fr->data, 0xe9800998);
                         aimbs_put32(&fr->data, 0xecf8427e);
-*/
+
                 } else
                         do_error_dialog(sess->aux_data, "WARNING: unknown hash request", "Gaim");

set.c

@@ -150 +150 @@
         if( s )
         {
-                if( t )
-                        t->next = s->next;
-                else
-                        irc->set = s->next;
-                
+                t->next = s->next;
                 g_free( s->key );
                 if( s->value ) g_free( s->value );