Adds javadocs, fixes warnings.

resources/languages/resources.properties

@@ -658,7 +658,7 @@ plugin.jabberaccregwizz.CSERVER=Server
 plugin.jabberaccregwizz.SERVER=Connect Server
 plugin.jabberaccregwizz.PORT=Port
 plugin.jabberaccregwizz.ENABLE_KEEP_ALIVE=Enable keep alive
-plugin.jabberaccregwizz.ENABLE_GMAIL_NOTIFICATIONS=Enable GMail notifications for new messages
+plugin.jabberaccregwizz.ENABLE_GMAIL_NOTIFICATIONS=Enable Gmail notifications for new messages
 plugin.jabberaccregwizz.NEW_ACCOUNT_TITLE=Jabber new account registration
 plugin.jabberaccregwizz.REGISTER_NEW_ACCOUNT_TEXT=In case you don't have a Jabber account, click on this button to create a new one.
 plugin.jabberaccregwizz.CHOOSE_SERVER_TITLE=Jabber new account server chooser

src/net/java/sip/communicator/impl/protocol/jabber/CallPeerMediaHandlerJabberImpl.java

@@ -30,8 +30,8 @@ public class CallPeerMediaHandlerJabberImpl
      * The <tt>Logger</tt> used by the <tt>CallPeerMediaHandlerJabberImpl</tt>
      * class and its instances for logging output.
      */
-    private static final Logger logger = Logger
-                    .getLogger(CallPeerMediaHandlerJabberImpl.class.getName());
+    private static final Logger logger
+        = Logger.getLogger(CallPeerMediaHandlerJabberImpl.class);
 
     /**
      * A temporarily single transport manager that we use for generating
@@ -111,9 +111,7 @@ public ContentPacketExtension getRemoteContent(String contentType)
                 = JingleUtils.getRtpDescription(content);
 
             if(description.getMedia().equals(contentType))
-            {
                 return content;
-            }
         }
         return null;
     }
@@ -133,9 +131,7 @@ public ContentPacketExtension getLocalContent(String contentType)
                 = JingleUtils.getRtpDescription(content);
 
             if(description.getMedia().equals(contentType))
-            {
                 return content;
-            }
         }
         return null;
     }
@@ -296,7 +292,7 @@ public void processOffer(List<ContentPacketExtension> offer)
                 if(helloHash != null && helloHash[1].length() > 0)
                 {
                     ZrtpHashPacketExtension hash
-                    = new ZrtpHashPacketExtension();
+                        = new ZrtpHashPacketExtension();
                     hash.setVersion(helloHash[0]);
                     hash.setValue(helloHash[1]);
 
@@ -407,7 +403,7 @@ protected List<ContentPacketExtension> generateSessionAccept()
      * Creates a {@link ContentPacketExtension}s of the streams for a
      * specific <tt>MediaDevice</tt>.
      *
-     * @param type <tt>MediaDevice</tt>
+     * @param dev <tt>MediaDevice</tt>
      * @return the {@link ContentPacketExtension}s of stream that this
      * handler is prepared to initiate.
      * @throws OperationFailedException if we fail to create the descriptions
@@ -464,7 +460,7 @@ private ContentPacketExtension createContent(MediaDevice dev)
      * prepared to initiate depending on available <tt>MediaDevice</tt>s and
      * local on-hold and video transmission preferences.
      *
-     * @param type <tt>MediaType</tt> of the content
+     * @param mediaType <tt>MediaType</tt> of the content
      * @return a {@link List} containing the {@link ContentPacketExtension}s of
      * streams that this handler is prepared to initiate.
      *
@@ -642,9 +638,7 @@ public void reinitAllContents()
             ContentPacketExtension ext = remoteContentMap.get(key);
 
             if(ext != null)
-            {
                 processContent(ext);
-            }
         }
     }
 
@@ -687,9 +681,7 @@ public void removeLocalContent(String name)
         ContentPacketExtension content = localContentMap.remove(name);
 
         if(content == null)
-        {
             return;
-        }
 
         RtpDescriptionPacketExtension description
             = JingleUtils.getRtpDescription(content);
@@ -714,9 +706,7 @@ public void removeRemoteContent(String name)
         ContentPacketExtension content = remoteContentMap.remove(name);
 
         if(content == null)
-        {
             return;
-        }
 
         RtpDescriptionPacketExtension description
             = JingleUtils.getRtpDescription(content);
@@ -853,7 +843,6 @@ public void processAnswer(List<ContentPacketExtension> answer)
      *
      * @return the transport manager that is handling our address management.
      */
-    @Override
     public TransportManagerJabberImpl getTransportManager()
     {
         return transportManager;
@@ -889,13 +878,9 @@ public void setRemotelyOnHold(boolean onHold)
         {
             //off hold - make sure that we re-enable sending if that's
             if(audioStream != null)
-            {
                 calculatePostHoldDirection(audioStream);
-            }
             if(videoStream != null)
-            {
                 calculatePostHoldDirection(videoStream);
-            }
         }
     }
 

src/net/java/sip/communicator/impl/protocol/jabber/OperationSetBasicInstantMessagingJabberImpl.java

@@ -509,7 +509,7 @@ public void registrationStateChanged(RegistrationStateChangeEvent evt)
             }
             else if (evt.getNewState() == RegistrationState.REGISTERED)
             {
-                //subscribe for Google (GMail or Google Apps) notifications
+                //subscribe for Google (Gmail or Google Apps) notifications
                 //for new mail messages.
                 boolean enableGmailNotifications
                    = jabberProvider
@@ -893,7 +893,7 @@ public boolean accept(Packet packet)
 
     /**
      * Subscribes this provider as interested in receiving notifications for
-     * new mail messages from Google mail services such as GMail or Google Apps.
+     * new mail messages from Google mail services such as Gmail or Google Apps.
      */
     private void subscribeForGmailNotifications()
     {
@@ -908,14 +908,14 @@ private void subscribeForGmailNotifications()
         {
             if (logger.isDebugEnabled())
                 logger.debug(accountIDService
-                        +" does not seem to provide a GMail notification "
+                        +" does not seem to provide a Gmail notification "
                         +" service so we won't be trying to subscribe for it");
             return;
         }
 
         if (logger.isDebugEnabled())
             logger.debug(accountIDService
-                        +" seems to provide a GMail notification "
+                        +" seems to provide a Gmail notification "
                         +" service so we will try to subscribe for it");
 
         ProviderManager providerManager = ProviderManager.getInstance();

src/net/java/sip/communicator/impl/protocol/jabber/extensions/mailnotification/MailThreadInfo.java

@@ -601,7 +601,7 @@ private void parseSenders(XmlPullParser parser)
 
     /**
      * Creates an html description of all participant names in the thread.
-     * We try to do this in a GMail-like (although quite simplified) way:<br/>
+     * We try to do this in a Gmail-like (although quite simplified) way:<br/>
      * We print the whole name for a sole participant. <br/>
      * We print only the first names for more than one participant. <br/>
      * We print up to three names max. <br/>

src/net/java/sip/communicator/impl/protocol/jabber/extensions/mailnotification/MailboxQueryIQ.java

@@ -11,7 +11,7 @@
 
 /**
  * A straightforward <tt>IQ</tt> extension. The <tt>QueryNotify</tt> object is
- * used to create queries for the GMail mail server. It creates a simple
+ * used to create queries for the Gmail mail server. It creates a simple
  * <tt>IQ</tt> packet which represents the query.
  *
  * @author Matthieu Helleringer

src/net/java/sip/communicator/launcher/ChangeJVMFrame.java

@@ -76,19 +76,14 @@ public ChangeJVMFrame(String osName)
         this.javaLinkPane.setContentType("text/html");
         this.javaLinkPane.setEditable(false);
 
-        if (osName.equals(MAC_OSX))
-            this.javaLinkPane.setText(macLink);
-        else
-            this.javaLinkPane.setText(defaultLink);
+        this.javaLinkPane.setText(osName.equals(MAC_OSX) ? macLink : defaultLink);
 
         this.javaLinkPane.addHyperlinkListener(new HyperlinkListener()
         {
             public void hyperlinkUpdate(HyperlinkEvent e)
             {
                 if (e.getEventType() == HyperlinkEvent.EventType.ACTIVATED)
-                {
                     new BrowserLauncher().openURL(e.getDescription());
-                }
             }
         });
 

src/net/java/sip/communicator/plugin/jabberaccregwizz/JabberAccountRegistration.java

@@ -175,10 +175,10 @@ public boolean isSendKeepAlive()
     }
 
     /**
-     * Determines whether SIP Communicator should be querying GMail servers
+     * Determines whether SIP Communicator should be querying Gmail servers
      * for unread mail messages.
      *
-     * @return <tt>true</tt> if we are to enable GMail notifications and
+     * @return <tt>true</tt> if we are to enable Gmail notifications and
      * <tt>false</tt> otherwise.
      */
     public boolean isGmailNotificationEnabled()
@@ -229,10 +229,10 @@ public void setSendKeepAlive(boolean sendKeepAlive)
     }
 
     /**
-     * Specifies whether SIP Communicator should be querying GMail servers
+     * Specifies whether SIP Communicator should be querying Gmail servers
      * for unread mail messages.
      *
-     * @param enabled <tt>true</tt> if we are to enable GMail notification and
+     * @param enabled <tt>true</tt> if we are to enable Gmail notification and
      * <tt>false</tt> otherwise.
      */
     public void setGmailNotificationEnabled(boolean enabled)

src/net/java/sip/communicator/plugin/jabberaccregwizz/JabberAccountRegistrationWizard.java

@@ -471,7 +471,7 @@ protected String getServerFromUserName(String userName)
     }
 
     /**
-     * Opens the GMail signup URI in the OS's default browser.
+     * Opens the Gmail signup URI in the OS's default browser.
      */
     public void webSignup()
     {

src/net/java/sip/communicator/util/Base64.java

@@ -32,7 +32,6 @@
 
 import java.io.*;
 
-
 public class Base64
 {
     private static final Base64Encoder encoder = new Base64Encoder();

src/net/java/sip/communicator/util/GenericBuffer.java

@@ -3,16 +3,15 @@
  * 
  * Distributable under LGPL license. See terms of license at gnu.org.
  */
-
 package net.java.sip.communicator.util;
 
 import java.util.*;
 
 /**
  * The GenericBuffer class provides a way to minimize the effort needed to
- * bufferize any kind of information. This class is particularly suited to
+ * buffer any kind of information. This class is particularly suited to
  * optimizations based on reusing already computed data.
- * 
+ *
  * @author Benoit Pradelle
  */
 public class GenericBuffer<T>

src/net/java/sip/communicator/util/Logger.java

@@ -6,7 +6,6 @@
  */
 package net.java.sip.communicator.util;
 
-import java.io.*;
 import java.util.logging.*;
 
 /**

src/net/java/sip/communicator/util/ScStdOut.java

@@ -272,6 +272,4 @@ public void println()
         if(stdOutPrintingEnabled)
             super.println();
     }
-
-
 }

src/net/java/sip/communicator/util/Sha1Crypto.java

@@ -1,8 +1,13 @@
+/*
+ * SIP Communicator, the OpenSource Java VoIP and Instant Messaging client.
+ *
+ * Distributable under LGPL license.
+ * See terms of license at gnu.org.
+ */
 package net.java.sip.communicator.util;
 
-import java.io.UnsupportedEncodingException;
-import java.security.MessageDigest;
-import java.security.NoSuchAlgorithmException;
+import java.io.*;
+import java.security.*;
 
 public class Sha1Crypto
 {

src/net/java/sip/communicator/util/UtilActivator.java

@@ -29,6 +29,10 @@ public class UtilActivator
     implements BundleActivator, 
                Thread.UncaughtExceptionHandler
 {
+    /**
+     * The <tt>Logger</tt> used by the <tt>UtilActivator</tt> class and its
+     * instances for logging output.
+     */
     private static final Logger logger
         = Logger.getLogger(UtilActivator.class);
 
@@ -99,14 +103,10 @@ public static ConfigurationService getConfigurationService()
     {
         if (configurationService == null)
         {
-            ServiceReference serviceReference =
-                bundleContext.getServiceReference(ConfigurationService.class
-                    .getName());
-
-            if (serviceReference != null)
-                configurationService =
-                    (ConfigurationService) bundleContext
-                        .getService(serviceReference);
+            configurationService
+                = ServiceUtils.getService(
+                        bundleContext,
+                        ConfigurationService.class);
         }
         return configurationService;
     }
@@ -115,14 +115,10 @@ public static KeybindingsService getKeybindingsService()
     {
         if (keybindingsService == null)
         {
-            ServiceReference serviceReference =
-                bundleContext.getServiceReference(KeybindingsService.class
-                    .getName());
-
-            if (serviceReference != null)
-                keybindingsService =
-                    (KeybindingsService) bundleContext
-                        .getService(serviceReference);
+            keybindingsService
+                = ServiceUtils.getService(
+                        bundleContext,
+                        KeybindingsService.class);
         }
         return keybindingsService;
     }
@@ -135,9 +131,10 @@ public static KeybindingsService getKeybindingsService()
     public static ResourceManagementService getResources()
     {
         if (resourceService == null)
-            resourceService =
-                ResourceManagementServiceUtils.getService(bundleContext);
-
+        {
+            resourceService
+                = ResourceManagementServiceUtils.getService(bundleContext);
+        }
         return resourceService;
     }
 
@@ -154,9 +151,7 @@ public static BufferedImage getImage(String imageID)
         URL path = getResources().getImageURL(imageID);
 
         if (path == null)
-        {
             return null;
-        }
 
         try
         {

src/net/java/sip/communicator/util/launchutils/AEGetURLEventHandler.java

@@ -4,12 +4,11 @@
  * Distributable under LGPL license.
  * See terms of license at gnu.org.
  */
-
 package net.java.sip.communicator.util.launchutils;
 
 /**
- * Registers as listener for Apple Event GURL.
- * And will handle any url comming from the os by passing it to LaunchArgHandler.
+ * Registers as listener for kAEGetURL AppleScript events.
+ * And will handle any url coming from the OS by passing it to LaunchArgHandler.
  *
  * @author Lubomir Marinov
  * @author Damian Minkov

src/net/java/sip/communicator/util/launchutils/ArgDelegator.java

@@ -22,6 +22,10 @@
  */
 class ArgDelegator
 {
+    /**
+     * The <tt>Logger</tt> used by the <tt>ArgDelegator</tt> class and its
+     * instances for logging output.
+     */
     private static final Logger logger = Logger.getLogger(ArgDelegator.class);
 
     /**
@@ -96,9 +100,7 @@ public void handleConcurrentInvocationRequest()
         synchronized(recordedArgs)
         {
             if(uriDelegationPeer != null)
-            {
                 uriDelegationPeer.handleConcurrentInvocationRequest();
-            }
         }
     }
 }

src/net/java/sip/communicator/util/skin/Skinnable.java

@@ -4,7 +4,6 @@
  * Distributable under LGPL license.
  * See terms of license at gnu.org.
  */
-
 package net.java.sip.communicator.util.skin;
 
 /**