Fix all javadoc warnings

This commit is contained in:
Florian Schmaus 2014-03-10 18:31:45 +01:00
parent 0058631ed6
commit b8a5437b28
42 changed files with 137 additions and 205 deletions

View File

@ -493,16 +493,6 @@ public class SASLAuthentication {
}
}
/**
* Notification message saying that SASL authentication has failed. The server may have
* closed the connection depending on the number of possible retries.
*
* @deprecated replaced by {@see #authenticationFailed(String)}.
*/
void authenticationFailed() {
authenticationFailed(null);
}
/**
* Notification message saying that SASL authentication has failed. The server may have
* closed the connection depending on the number of possible retries.

View File

@ -176,7 +176,7 @@ public final class SmackConfiguration {
* Sets the default max size of a packet collector before it will delete
* the older packets.
*
* @param The number of packets to queue before deleting older packets.
* @param collectorSize the number of packets to queue before deleting older packets.
*/
public static void setPacketCollectorSize(int collectorSize) {
packetCollectorSize = collectorSize;

View File

@ -844,7 +844,7 @@ public abstract class XMPPConnection {
* XMPPConnection holds this information in order to avoid a dependency to
* smackx where EntityCapsManager lives from smack.
*
* @return
* @return the servers entity caps node
*/
public String getServiceCapsNode() {
return serviceCapsNode;

View File

@ -124,7 +124,7 @@ public abstract class IQ extends Packet {
* <li>No child element of the IQ element.
* </ul>
*
* @param iq the {@link Type#GET IQ.Type.GET} or {@link Type#SET IQ.Type.SET} IQ packet.
* @param request the {@link Type#GET IQ.Type.GET} or {@link Type#SET IQ.Type.SET} IQ packet.
* @throws IllegalArgumentException if the IQ packet does not have a type of
* {@link Type#GET IQ.Type.GET} or {@link Type#SET IQ.Type.SET}.
* @return a new {@link Type#RESULT IQ.Type.RESULT} IQ based on the originating IQ.
@ -158,7 +158,7 @@ public abstract class IQ extends Packet {
* <li>The provided {@link XMPPError XMPPError}.
* </ul>
*
* @param iq the {@link Type#GET IQ.Type.GET} or {@link Type#SET IQ.Type.SET} IQ packet.
* @param request the {@link Type#GET IQ.Type.GET} or {@link Type#SET IQ.Type.SET} IQ packet.
* @param error the error to associate with the created IQ packet.
* @throws IllegalArgumentException if the IQ packet does not have a type of
* {@link Type#GET IQ.Type.GET} or {@link Type#SET IQ.Type.SET}.

View File

@ -14,7 +14,6 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package org.jivesoftware.smack.parsing;
/**
@ -25,7 +24,7 @@ package org.jivesoftware.smack.parsing;
*
* Subclasses may or may not override certain methods of this class. Each of these methods will receive the exception
* that caused the parsing error and an instance of an Unparsed Packet type. The latter can be used to inspect the
* stanza that caused the parsing error by using the getContent() (for example {@link UnparsedIQ#getContent()})
* stanza that caused the parsing error by using the getContent() (for example {@link UnparsablePacket#getContent()})
* method.
*
* Smack provides 2 predefined ParsingExceptionCallback's: {@link ExceptionLoggingCallback} and {@link ExceptionThrowingCallback}.

View File

@ -56,8 +56,8 @@ import org.xmlpull.v1.XmlPullParser;
* &lt;/message&gt;
*
* I would have a classes
* {@link ItemsProvider} extends {@link EmbeddedExtensionProvider}
* {@link ItemProvider} extends {@link EmbeddedExtensionProvider}
* <tt>ItemsProvider</tt> extends {@link EmbeddedExtensionProvider}
* <tt>ItemProvider</tt> extends {@link EmbeddedExtensionProvider}
* and
* AtomProvider extends {@link PacketExtensionProvider}
*

View File

@ -41,9 +41,9 @@ import javax.security.sasl.SaslException;
* <li>{@link #getName()} -- returns the common name of the SASL mechanism.</li>
* </ul>
* Subclasses will likely want to implement their own versions of these methods:
* <li>{@link #authenticate(String, String, String)} -- Initiate authentication stanza using the
* <li>{@link #authenticate(String, String, String, String)} -- Initiate authentication stanza using the
* deprecated method.</li>
* <li>{@link #authenticate(String, String, CallbackHandler)} -- Initiate authentication stanza
* <li>{@link #authenticate(String, CallbackHandler)} -- Initiate authentication stanza
* using the CallbackHandler method.</li>
* <li>{@link #challengeReceived(String)} -- Handle a challenge from the server.</li>
* </ul>
@ -87,7 +87,7 @@ public abstract class SASLMechanism implements CallbackHandler {
/**
* Builds and sends the <tt>auth</tt> stanza to the server. Note that this method of
* authentication is not recommended, since it is very inflexable. Use
* {@link #authenticate(String, String, CallbackHandler)} whenever possible.
* {@link #authenticate(String, CallbackHandler)} whenever possible.
*
* Explanation of auth stanza:
*

View File

@ -55,7 +55,7 @@ public class DNSUtil {
/**
* Returns the current DNS resolved used to perform DNS lookups.
*
* @return
* @return the active DNSResolver
*/
public static DNSResolver getDNSResolver() {
return dnsResolver;
@ -155,7 +155,7 @@ public class DNSUtil {
* is calculated by random. The others are ore simply ordered by their priority.
*
* @param records
* @return
* @return the list of resolved HostAddresses
*/
protected static List<HostAddress> sortSRVRecords(List<SRVRecord> records) {
// RFC 2782, Usage rules: "If there is precisely one SRV RR, and its Target is "."

View File

@ -24,7 +24,7 @@ public interface StringEncoder {
* Encodes an string to another representation
*
* @param string
* @return
* @return the encoded String
*/
String encode(String string);
@ -32,7 +32,7 @@ public interface StringEncoder {
* Decodes an string back to it's initial representation
*
* @param string
* @return
* @return the decoded String
*/
String decode(String string);
}

View File

@ -208,7 +208,7 @@ public class XmppDateTime {
* does not.
*
* @param dateString
* @return
* @return the String with converted timezone
*/
public static String convertXep82TimezoneToRfc822(String dateString) {
if (dateString.charAt(dateString.length() - 1) == 'Z') {
@ -235,7 +235,7 @@ public class XmppDateTime {
* Converts a time zone to the String format as specified in XEP-0082
*
* @param timeZone
* @return
* @return the String representation of the TimeZone
*/
public static String asString(TimeZone timeZone) {
int rawOffset = timeZone.getRawOffset();

View File

@ -1,6 +1,6 @@
/**
*
* Copyright 2013 Florian Schmaus
* Copyright 2013-2014 Florian Schmaus
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@ -17,8 +17,8 @@
package org.jivesoftware.smack.util.dns;
/**
* @see <a href="http://tools.ietf.org/html/rfc2782>RFC 2782: A DNS RR for specifying the location of services (DNS
* SRV)<a>
* @see <a href="http://tools.ietf.org/html/rfc2782">RFC 2782: A DNS RR for specifying the location of services (DNS
* SRV)</a>
* @author Florian Schmaus
*
*/

View File

@ -139,7 +139,6 @@ public class CarbonManager extends Manager {
*
* @param new_state whether carbons should be enabled or disabled
*
* @return true if the operation was successful
* @throws XMPPException
*/
public void setCarbonsEnabled(final boolean new_state) throws XMPPException {
@ -154,7 +153,6 @@ public class CarbonManager extends Manager {
/**
* Helper method to enable carbons.
*
* @return true if the operation was successful
* @throws XMPPException
*/
public void enableCarbons() throws XMPPException {
@ -164,7 +162,6 @@ public class CarbonManager extends Manager {
/**
* Helper method to disable carbons.
*
* @return true if the operation was successful
* @throws XMPPException
*/
public void disableCarbons() throws XMPPException {

View File

@ -35,20 +35,15 @@ import java.util.concurrent.ConcurrentHashMap;
import java.util.logging.Level;
import java.util.logging.Logger;
import org.jivesoftware.smack.SmackConfiguration;
import org.jivesoftware.smack.XMPPException;
/**
* The Socks5Proxy class represents a local SOCKS5 proxy server. It can be enabled/disabled by
* setting the <code>localSocks5ProxyEnabled</code> flag in the <code>smack-config.xml</code> or by
* invoking {@link SmackConfiguration#setLocalSocks5ProxyEnabled(boolean)}. The proxy is enabled by
* default.
* invoking {@link #setLocalSocks5ProxyEnabled(boolean)}. The proxy is enabled by default.
* <p>
* The port of the local SOCKS5 proxy can be configured by setting <code>localSocks5ProxyPort</code>
* in the <code>smack-config.xml</code> or by invoking
* {@link SmackConfiguration#setLocalSocks5ProxyPort(int)}. Default port is 7777. If you set the
* port to a negative value Smack tries to the absolute value and all following until it finds an
* open port.
* The port of the local SOCKS5 proxy can be configured by invoking
* {@link #setLocalSocks5ProxyPort(int)}. Default port is 7777. If you set the port to a negative
* value Smack tries to the absolute value and all following until it finds an open port.
* <p>
* If your application is running on a machine with multiple network interfaces or if you want to
* provide your public address in case you are behind a NAT router, invoke

View File

@ -136,7 +136,7 @@ public class EntityCapsManager extends Manager {
* Get the Node version (node#ver) of a JID. Returns a String or null if
* EntiyCapsManager does not have any information.
*
* @param user
* @param jid
* the user (Full JID)
* @return the node version (node#ver) or null
*/
@ -371,7 +371,7 @@ public class EntityCapsManager extends Manager {
* "http://www.igniterealtime.org/projects/smack/#66/0NaeaBKkwk85efJTGmU47vXI=
* )
*
* @return
* @return the local NodeVer
*/
public String getLocalNodeVer() {
return ENTITY_NODE + '#' + getCapsVersion();
@ -381,7 +381,7 @@ public class EntityCapsManager extends Manager {
* Returns true if Entity Caps are supported by a given JID
*
* @param jid
* @return
* @return true if the entity supports Entity Capabilities.
*/
public boolean areEntityCapsSupported(String jid) throws XMPPException {
return sdm.supportsFeature(jid, NAMESPACE);
@ -390,7 +390,7 @@ public class EntityCapsManager extends Manager {
/**
* Returns true if Entity Caps are supported by the local service/server
*
* @return
* @return true if the user's server supports Entity Capabilities.
*/
public boolean areEntityCapsSupportedByServer() throws XMPPException {
return areEntityCapsSupported(connection().getServiceName());
@ -398,19 +398,10 @@ public class EntityCapsManager extends Manager {
/**
* Updates the local user Entity Caps information with the data provided
*
*
* If we are connected and there was already a presence send, another
* presence is send to inform others about your new Entity Caps node string.
*
* @param discoverInfo
* the local users discover info (mostly the service discovery
* features)
* @param identityType
* the local users identity type
* @param identityName
* the local users identity name
* @param extendedInfo
* the local users extended info
*
*/
public void updateLocalEntityCaps() {
XMPPConnection connection = connection();
@ -478,8 +469,8 @@ public class EntityCapsManager extends Manager {
* @see <a href="http://xmpp.org/extensions/xep-0115.html#ver-proc">XEP-0115
* 5.4 Processing Method</a>
*
* @param capsNode
* the caps node (i.e. node#ver)
* @param ver
* @param hash
* @param info
* @return true if it's valid and should be cache, false if not
*/

View File

@ -138,7 +138,7 @@ public class SimpleDirectoryPersistentCache implements EntityCapsPersistentCache
* Tries to restore an DiscoverInfo packet from a file.
*
* @param file
* @return
* @return the restored DiscoverInfo
* @throws IOException
*/
private static DiscoverInfo restoreInfoFromFile(File file) throws IOException {

View File

@ -14,7 +14,6 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package org.jivesoftware.smackx.commands;
import org.jivesoftware.smack.SmackConfiguration;
@ -156,26 +155,4 @@ public class RemoteCommand extends AdHocCommand {
public String getOwnerJID() {
return jid;
}
/**
* Returns the number of milliseconds to wait for a respone. The
* {@link SmackConfiguration#getPacketReplyTimeout default} value
* should be adjusted for commands that can take a long time to execute.
*
* @return the number of milliseconds to wait for responses.
*/
public long getPacketReplyTimeout() {
return packetReplyTimeout;
}
/**
* Returns the number of milliseconds to wait for a respone. The
* {@link SmackConfiguration#getPacketReplyTimeout default} value
* should be adjusted for commands that can take a long time to execute.
*
* @param packetReplyTimeout the number of milliseconds to wait for responses.
*/
public void setPacketReplyTimeout(long packetReplyTimeout) {
this.packetReplyTimeout = packetReplyTimeout;
}
}

View File

@ -271,7 +271,7 @@ public class ServiceDiscoveryManager extends Manager {
/**
* Returns all identities of this client as unmodifiable Collection
*
* @return
* @return all identies as set
*/
public Set<DiscoverInfo.Identity> getIdentities() {
Set<Identity> res = new HashSet<Identity>(identities);
@ -454,7 +454,7 @@ public class ServiceDiscoveryManager extends Manager {
* Returns the data form that is set as extended information for this Service Discovery instance (XEP-0128)
*
* @see <a href="http://xmpp.org/extensions/xep-0128.html">XEP-128: Service Discovery Extensions</a>
* @return
* @return the data form
*/
public DataForm getExtendedInfo() {
return extendedInfo;
@ -464,7 +464,7 @@ public class ServiceDiscoveryManager extends Manager {
* Returns the data form as List of PacketExtensions, or null if no data form is set.
* This representation is needed by some classes (e.g. EntityCapsManager, NodeInformationProvider)
*
* @return
* @return the data form as List of PacketExtensions
*/
public List<PacketExtension> getExtendedInfoAsList() {
List<PacketExtension> res = null;
@ -603,7 +603,7 @@ public class ServiceDiscoveryManager extends Manager {
* be returned by the server whenever the server receives a disco request targeted to the bare
* address of the client (i.e. user@host.com).
*
* @param DiscoverInfo the discover info packet to check.
* @param info the discover info packet to check.
* @return true if the server supports publishing of items.
*/
public static boolean canPublishItems(DiscoverInfo info) {
@ -646,11 +646,11 @@ public class ServiceDiscoveryManager extends Manager {
}
/**
* Queries the remote jid for it's features and returns true if the given feature is found.
* Queries the remote entity for it's features and returns true if the given feature is found.
*
* @param jid
* @param jid the JID of the remote entity
* @param feature
* @return
* @return true if the entity supports the feature, false otherwise
* @throws XMPPException
*/
public boolean supportsFeature(String jid, String feature) throws XMPPException {
@ -663,8 +663,8 @@ public class ServiceDiscoveryManager extends Manager {
*/
/**
* Loads the ServiceDiscoveryManager with an EntityCapsManger
* that speeds up certain lookups
* Loads the ServiceDiscoveryManager with an EntityCapsManger that speeds up certain lookups.
*
* @param manager
*/
public void setEntityCapsManager(EntityCapsManager manager) {
@ -672,8 +672,7 @@ public class ServiceDiscoveryManager extends Manager {
}
/**
* Updates the Entity Capabilities Verification String
* if EntityCaps is enabled
* Updates the Entity Capabilities Verification String if EntityCaps is enabled.
*/
private void renewEntityCapsVersion() {
if (capsManager != null && capsManager.entityCapsEnabled())

View File

@ -424,12 +424,13 @@ public class DiscoverInfo extends IQ {
}
/**
* Compares this identity with another one. The comparison order is:
* Category, Type, Lang. If all three are identical the other Identity is considered equal.
* Name is not used for comparision, as defined by XEP-0115
* Compares this identity with another one. The comparison order is: Category, Type, Lang.
* If all three are identical the other Identity is considered equal. Name is not used for
* comparison, as defined by XEP-0115
*
* @param obj
* @return
* @param other
* @return a negative integer, zero, or a positive integer as this object is less than,
* equal to, or greater than the specified object.
*/
public int compareTo(DiscoverInfo.Identity other) {
String otherLang = other.lang == null ? "" : other.lang;

View File

@ -46,7 +46,6 @@ public class Forwarded implements PacketExtension {
/**
* Creates a new Forwarded packet extension.
*
* @param delay an optional {@link DelayInfo} timestamp of the packet.
* @param fwdPacket the packet that is forwarded (required).
*/
public Forwarded(Packet fwdPacket) {

View File

@ -218,7 +218,7 @@ public class PingManager extends Manager {
* {@link #isPingSupported(String)} is false.
*
* @param notifyListeners Notify the PingFailedListener in case of error if true
* @return
* @return true if the user's server could be pinged.
*/
public boolean pingMyServer(boolean notifyListeners) {
boolean res = ping(connection().getServiceName());
@ -273,7 +273,7 @@ public class PingManager extends Manager {
* users server. If there was no successful Ping (e.g. because this
* feature is disabled) -1 will be returned.
*
* @return
* @return the timestamp of the last successful ping.
*/
public long getLastSuccessfulPing() {
return Math.max(lastSuccessfulAutomaticPing, lastSuccessfulManualPing);

View File

@ -42,7 +42,7 @@ public class PrivacyList {
private final boolean isDefaultList;
/** Holds the list name used to print **/
private final String listName;
/** Holds the list of {@see PrivacyItem} **/
/** Holds the list of {@link PrivacyItem} */
private final List<PrivacyItem> items;
protected PrivacyList(boolean isActiveList, boolean isDefaultList,

View File

@ -37,7 +37,7 @@ import org.jivesoftware.smackx.pubsub.provider.ItemProvider;
* <li>Will require payload if the node configuration has {@link ConfigureForm#isDeliverPayloads()} set
* to true.
*
* <p>To customise the payload object being returned from the {@link #getPayload()} method, you can
* <p>To customise the payload object being returned from the {@link PayloadItem#getPayload()} method, you can
* add a custom parser as explained in {@link ItemProvider}.
*
* @author Robin Collier

View File

@ -82,7 +82,6 @@ public class ItemsExtension extends NodeExtension implements EmbeddedPacketExten
* @param itemsType Type of representation
* @param nodeId The node to which the items are being sent or deleted
* @param items The list of {@link Item} or {@link RetractItem}
* @param attributeValue The value of the <b>max_items</b>
*/
public ItemsExtension(ItemsElementType itemsType, String nodeId, List<? extends PacketExtension> items)
{
@ -104,10 +103,8 @@ public class ItemsExtension extends NodeExtension implements EmbeddedPacketExten
* <li>Items deleted event - itemsType = {@link ItemsElementType#items}, items = list of {@link RetractItem} and
* attributeValue = <code>null</code>
*
* @param itemsType Type of representation
* @param nodeId The node to which the items are being sent or deleted
* @param items The list of {@link Item} or {@link RetractItem}
* @param attributeValue The value of the <b>max_items</b>
*/
public ItemsExtension(String nodeId, List<? extends PacketExtension> items, boolean notify)
{

View File

@ -19,7 +19,7 @@ package org.jivesoftware.smackx.pubsub;
import org.jivesoftware.smack.packet.PacketExtension;
/**
* The default payload representation for {@link Item#getPayload()}. It simply
* The default payload representation for {@link PayloadItem#getPayload()}. It simply
* stores the XML payload as a string.
*
* @author Robin Collier

View File

@ -27,10 +27,10 @@ import java.util.logging.Logger;
/**
* A Time IQ packet, which is used by XMPP clients to exchange their respective local
* times. Clients that wish to fully support the entitity time protocol should register
* times. Clients that wish to fully support the entity time protocol should register
* a PacketListener for incoming time requests that then respond with the local time.
*
* @see http://www.xmpp.org/extensions/xep-0202.html
* @see <a href="http://www.xmpp.org/extensions/xep-0202.html">XEP-202</a>
* @author Florian Schmaus
*/
public class Time extends IQ {

View File

@ -197,7 +197,7 @@ public class DataForm implements PacketExtension {
* Returns true if this DataForm has at least one FORM_TYPE field which is
* hidden. This method is used for sanity checks.
*
* @return
* @return true if there is at least one field which is hidden.
*/
public boolean hasHiddenFormTypeField() {
boolean found = false;

View File

@ -124,7 +124,7 @@ public class ContentNegotiator extends JingleNegotiator {
}
/**
* @return
* @return the JingleTransportManager
*/
public JingleTransportManager getTransportManager() {
return jingleTransportManager;
@ -134,7 +134,7 @@ public class ContentNegotiator extends JingleNegotiator {
* Called from above when starting a new session.
*/
protected void doStart() {
JingleContent result = new JingleContent(creator, name);
// JingleContent result = new JingleContent(creator, name);
// result.setDescription(mediaNeg.start());
// result.addJingleTransport(transNeg.start());

View File

@ -204,7 +204,7 @@ public class JingleManager implements JingleSessionListener {
* If a fully implemented JingleMediaSession is entered, JingleManager manage Jingle signalling and jmf
*
* @param connection XMPP XMPPConnection to be used
* @param jingleMediaManager an implemeted JingleMediaManager to be used.
* @param jingleMediaManagers an implemeted JingleMediaManager to be used.
*/
public JingleManager(XMPPConnection connection, List<JingleMediaManager> jingleMediaManagers) {
this.connection = connection;
@ -321,7 +321,7 @@ public class JingleManager implements JingleSessionListener {
/**
* Get the Media Managers of this Jingle Manager
*
* @return
* @return the list of JingleMediaManagers
*/
public List<JingleMediaManager> getMediaManagers() {
return jingleMediaManagers;
@ -330,7 +330,7 @@ public class JingleManager implements JingleSessionListener {
/**
* Set the Media Managers of this Jingle Manager
*
* @param jingleMediaManager JingleMediaManager to be used for open, close, start and stop jmf streamings
* @param jingleMediaManagers JingleMediaManager to be used for open, close, start and stop jmf streamings
*/
public void setMediaManagers(List<JingleMediaManager> jingleMediaManagers) {
this.jingleMediaManagers = jingleMediaManagers;
@ -516,7 +516,6 @@ public class JingleManager implements JingleSessionListener {
*
* @param responder the fully qualified jabber ID with resource of the other
* user.
* @param payloadTypes list of supported payload types
* @return The session on which the negotiation can be run.
*/
public JingleSession createOutgoingJingleSession(String responder) throws XMPPException {
@ -549,7 +548,6 @@ public class JingleManager implements JingleSessionListener {
* will create an JingleSession which allows the negotiation to procede.
*
* @param request the remote request that is being accepted.
* @param payloadTypes the list of supported Payload types that can be accepted
* @return the session which manages the rest of the negotiation.
*/
public JingleSession createIncomingJingleSession(JingleSessionRequest request) throws XMPPException {
@ -585,7 +583,7 @@ public class JingleManager implements JingleSessionListener {
* Get a session with the informed JID. If no session is found, return null.
*
* @param jid
* @return
* @return the JingleSession
*/
public JingleSession getSession(String jid) {
for (JingleSession jingleSession : jingleSessions) {

View File

@ -65,7 +65,7 @@ public abstract class JingleNegotiator {
/**
* Default constructor with a Connection
*
* @param connection the connection associated
* @param session the jingle session
*/
public JingleNegotiator(JingleSession session) {
this.session = session;
@ -119,7 +119,7 @@ public abstract class JingleNegotiator {
/**
* Set the XMPP connection associated.
*
* @param connection the connection to set
* @param session the jingle session
*/
public void setSession(JingleSession session) {
this.session = session;
@ -140,7 +140,7 @@ public abstract class JingleNegotiator {
* Check if the passed ID is the expected ID
*
* @param id
* @return
* @return true if is expected id
*/
public boolean isExpectedId(String id) {
if (id != null) {

View File

@ -102,7 +102,7 @@ public class JingleSession extends JingleNegotiator implements MediaReceivedList
* the responder JID
* @param sessionid
* the session ID
* @param jingleMediaManager
* @param jingleMediaManagers
* the jingleMediaManager
*/
public JingleSession(XMPPConnection conn, String initiator, String responder, String sessionid,
@ -136,7 +136,7 @@ public class JingleSession extends JingleNegotiator implements MediaReceivedList
* the initiator JID
* @param responder
* the responder JID
* @param jingleMediaManager
* @param jingleMediaManagers
* the jingleMediaManager
*/
public JingleSession(XMPPConnection conn, JingleSessionRequest request, String initiator, String responder,
@ -171,7 +171,7 @@ public class JingleSession extends JingleNegotiator implements MediaReceivedList
/**
* Get the Media Manager of this Jingle Session
*
* @return
* @return the JingleMediaManagers
*/
public List<JingleMediaManager> getMediaManagers() {
return jingleMediaManagers;
@ -180,7 +180,7 @@ public class JingleSession extends JingleNegotiator implements MediaReceivedList
/**
* Set the Media Manager of this Jingle Session
*
* @param jingleMediaManager
* @param jingleMediaManagers
*/
public void setMediaManagers(List<JingleMediaManager> jingleMediaManagers) {
this.jingleMediaManagers = jingleMediaManagers;
@ -272,7 +272,6 @@ public class JingleSession extends JingleNegotiator implements MediaReceivedList
*
* @param iq
* the packet received
* @return the new Jingle packet to send.
* @throws XMPPException
*/
public synchronized void receivePacketAndRespond(IQ iq) throws XMPPException {
@ -1055,8 +1054,8 @@ public class JingleSession extends JingleNegotiator implements MediaReceivedList
* From whom the packet is sent.
* @param errCode
* The error code.
* @param errStr
* The error string.
* @param error
* The XMPPError string.
* @return The created IQ packet.
*/
public static IQ createError(String ID, String to, String from, int errCode, XMPPError error) {
@ -1076,7 +1075,7 @@ public class JingleSession extends JingleNegotiator implements MediaReceivedList
*
* @param iq
* The Jingle packet we are responing to
* @param error
* @param jingleError
* the IQ packet we want to complete and send
*/
public IQ createJingleError(IQ iq, JingleError jingleError) {

View File

@ -72,7 +72,7 @@ public abstract class JingleMediaManager {
* @param payloadType
* @param remote
* @param local
* @return
* @return the media session
*/
public abstract JingleMediaSession createMediaSession(PayloadType payloadType, final TransportCandidate remote,
final TransportCandidate local, JingleSession jingleSession);

View File

@ -69,7 +69,7 @@ public abstract class JingleMediaSession {
/**
* Returns the PayloadType of the Media Session
*
* @return
* @return the PayloadType
*/
public PayloadType getPayloadType() {
return payloadType;
@ -78,7 +78,7 @@ public abstract class JingleMediaSession {
/**
* Returns the Media Session local Candidate
*
* @return
* @return the TransportCandidate
*/
public TransportCandidate getLocal() {
return local;
@ -87,7 +87,7 @@ public abstract class JingleMediaSession {
/**
* Returns the Media Session remote Candidate
*
* @return
* @return the TransportCandidate
*/
public TransportCandidate getRemote() {
return remote;

View File

@ -66,7 +66,7 @@ public class MediaNegotiator extends JingleNegotiator {
* but it does not start the negotiation. For starting the negotiation, call
* startNegotiation.
*
* @param js
* @param session
* The jingle session.
*/
public MediaNegotiator(JingleSession session, JingleMediaManager mediaManager, List<PayloadType> pts,
@ -198,7 +198,7 @@ public class MediaNegotiator extends JingleNegotiator {
* will be the best payload type to use.
*
* @param jingle
* @return
* @return the iq
*/
private IQ receiveContentAcceptAction(Jingle jingle, JingleDescription description) throws XMPPException {
IQ response = null;
@ -228,7 +228,7 @@ public class MediaNegotiator extends JingleNegotiator {
* Receive a session-initiate packet.
* @param jingle
* @param description
* @return
* @return the iq
*/
private IQ receiveSessionInitiateAction(Jingle jingle, JingleDescription description) {
IQ response = null;
@ -416,27 +416,27 @@ public class MediaNegotiator extends JingleNegotiator {
}
}
/**
* Create an offer for the list of audio payload types.
*
* @return a new Jingle packet with the list of audio Payload Types
*/
private Jingle createAudioPayloadTypesOffer() {
JingleContent jingleContent = new JingleContent(parentNegotiator.getCreator(), parentNegotiator.getName());
JingleDescription audioDescr = new JingleDescription.Audio();
// Add the list of payloads for audio and create a
// JingleDescription
// where we announce our payloads...
audioDescr.addAudioPayloadTypes(localAudioPts);
jingleContent.setDescription(audioDescr);
Jingle jingle = new Jingle(JingleActionEnum.CONTENT_ACCEPT);
jingle.addContent(jingleContent);
return jingle;
}
// /**
// * Create an offer for the list of audio payload types.
// *
// * @return a new Jingle packet with the list of audio Payload Types
// */
// private Jingle createAudioPayloadTypesOffer() {
//
// JingleContent jingleContent = new JingleContent(parentNegotiator.getCreator(), parentNegotiator.getName());
// JingleDescription audioDescr = new JingleDescription.Audio();
//
// // Add the list of payloads for audio and create a
// // JingleDescription
// // where we announce our payloads...
// audioDescr.addAudioPayloadTypes(localAudioPts);
// jingleContent.setDescription(audioDescr);
//
// Jingle jingle = new Jingle(JingleActionEnum.CONTENT_ACCEPT);
// jingle.addContent(jingleContent);
//
// return jingle;
// }
// Predefined messages and Errors
@ -502,7 +502,6 @@ public class MediaNegotiator extends JingleNegotiator {
/**
* Called from above when starting a new session.
* @return
*/
protected void doStart() {

View File

@ -311,7 +311,7 @@ public class AudioChannel {
*
* @param codecFormat
* @param milliseconds
* @return
* @return the best packet size
* @throws IllegalArgumentException
*/
private int getPacketSize(Format codecFormat, int milliseconds) throws IllegalArgumentException {

View File

@ -49,7 +49,7 @@ public abstract class JingleTransportManager {
/**
* Get a new Transport Resolver to be used in a Jingle Session
*
* @return
* @return the TransportResolver to be used
*/
public TransportResolver getResolver(JingleSession session) throws XMPPException {
TransportResolver resolver = createResolver(session);
@ -64,7 +64,7 @@ public abstract class JingleTransportManager {
/**
* Create a Transport Resolver instance according to the implementation.
*
* @return
* @return the TransportResolver
*/
protected abstract TransportResolver createResolver(JingleSession session);

View File

@ -148,7 +148,7 @@ public class RTPBridge extends IQ {
/**
* Get the Session ID of the Packet (usually same as Jingle Session ID)
*
* @return
* @return the session ID
*/
public String getSid() {
return sid;
@ -166,7 +166,7 @@ public class RTPBridge extends IQ {
/**
* Get the Host A IP Address
*
* @return
* @return the Host A IP Address
*/
public String getHostA() {
return hostA;
@ -184,7 +184,7 @@ public class RTPBridge extends IQ {
/**
* Get the Host B IP Address
*
* @return
* @return the Host B IP Address
*/
public String getHostB() {
return hostB;
@ -202,7 +202,7 @@ public class RTPBridge extends IQ {
/**
* Get Side A receive port
*
* @return
* @return the side A receive prot
*/
public int getPortA() {
return portA;
@ -220,7 +220,7 @@ public class RTPBridge extends IQ {
/**
* Get Side B receive port
*
* @return
* @return the side B receive port
*/
public int getPortB() {
return portB;
@ -238,7 +238,7 @@ public class RTPBridge extends IQ {
/**
* Get the RTP Bridge IP
*
* @return
* @return the RTP Bridge IP
*/
public String getIp() {
return ip;
@ -256,7 +256,7 @@ public class RTPBridge extends IQ {
/**
* Get the RTP Agent Pass
*
* @return
* @return the RTP Agent Pass
*/
public String getPass() {
return pass;
@ -274,7 +274,7 @@ public class RTPBridge extends IQ {
/**
* Get the name of the Candidate
*
* @return
* @return the name of the Candidate
*/
public String getName() {
return name;
@ -292,7 +292,7 @@ public class RTPBridge extends IQ {
/**
* Get the Child Element XML of the Packet
*
* @return
* @return the Child Element XML of the Packet
*/
public String getChildElementXML() {
StringBuilder str = new StringBuilder();
@ -327,7 +327,6 @@ public class RTPBridge extends IQ {
int eventType;
String elementName;
String namespace;
if (!parser.getNamespace().equals(RTPBridge.NAMESPACE))
throw new Exception("Not a RTP Bridge packet");
@ -343,7 +342,6 @@ public class RTPBridge extends IQ {
while (!done) {
eventType = parser.next();
elementName = parser.getName();
namespace = parser.getNamespace();
if (eventType == XmlPullParser.START_TAG) {
if (elementName.equals("candidate")) {
@ -361,8 +359,6 @@ public class RTPBridge extends IQ {
}
}
else if (elementName.equals("publicip")) {
//String p = parser.getAttributeName(0);
int x = parser.getAttributeCount();
for (int i = 0; i < parser.getAttributeCount(); i++) {
if (parser.getAttributeName(i).equals("ip"))
iq.setIp(parser.getAttributeValue(i));
@ -385,7 +381,7 @@ public class RTPBridge extends IQ {
*
* @param connection
* @param sessionID
* @return
* @return the new RTPBridge
*/
public static RTPBridge getRTPBridge(XMPPConnection connection, String sessionID) {
@ -410,7 +406,7 @@ public class RTPBridge extends IQ {
* Check if the server support RTPBridge Service.
*
* @param connection
* @return
* @return true if the server supports the RTPBridge service
*/
public static boolean serviceAvailable(XMPPConnection connection) {
@ -452,7 +448,7 @@ public class RTPBridge extends IQ {
* Check if the server support RTPBridge Service.
*
* @param connection
* @return
* @return the RTPBridge
*/
public static RTPBridge relaySession(XMPPConnection connection, String sessionID, String pass, TransportCandidate proxyCandidate, TransportCandidate localCandidate) {

View File

@ -19,6 +19,7 @@ package org.jivesoftware.smackx.jingle.nat;
import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;
import java.util.logging.Level;
import java.util.logging.Logger;
import org.jivesoftware.smack.XMPPConnection;
@ -77,7 +78,7 @@ public class STUN extends IQ {
/**
* Get a list of STUN Servers recommended by the Server
*
* @return
* @return the list of STUN servers
*/
public List<StunServerAddress> getServers() {
return servers;
@ -86,7 +87,7 @@ public class STUN extends IQ {
/**
* Get Public Ip returned from the XMPP server
*
* @return
* @return the public IP
*/
public String getPublicIp() {
return publicIp;
@ -104,7 +105,7 @@ public class STUN extends IQ {
/**
* Get the Child Element XML of the Packet
*
* @return
* @return the child element XML
*/
public String getChildElementXML() {
StringBuilder str = new StringBuilder();
@ -130,7 +131,6 @@ public class STUN extends IQ {
int eventType;
String elementName;
String namespace;
if (!parser.getNamespace().equals(NAMESPACE))
throw new Exception("Not a STUN packet");
@ -141,7 +141,6 @@ public class STUN extends IQ {
while (!done) {
eventType = parser.next();
elementName = parser.getName();
namespace = parser.getNamespace();
if (eventType == XmlPullParser.START_TAG) {
if (elementName.equals("server")) {
@ -181,7 +180,7 @@ public class STUN extends IQ {
* If a error occurs or the server don't support STUN Service, null is returned.
*
* @param connection
* @return
* @return the STUN server address
*/
public static STUN getSTUNServer(XMPPConnection connection) {
@ -205,8 +204,8 @@ public class STUN extends IQ {
/**
* Check if the server support STUN Service.
*
* @param xmppConnection
* @return
* @param connection the connection
* @return true if the server support STUN
*/
public static boolean serviceAvailable(XMPPConnection connection) {
@ -239,7 +238,7 @@ public class STUN extends IQ {
}
}
catch (XMPPException e) {
e.printStackTrace();
LOGGER.log(Level.SEVERE, "serviceAvailable", e);
}
return false;
}
@ -260,7 +259,7 @@ public class STUN extends IQ {
/**
* Get the Host Address
*
* @return
* @return the host address
*/
public String getServer() {
return server;
@ -269,7 +268,7 @@ public class STUN extends IQ {
/**
* Get the Server Port
*
* @return
* @return the server port
*/
public String getPort() {
return port;

View File

@ -103,7 +103,7 @@ public abstract class TransportCandidate {
/**
* Get local IP to bind to this candidate
*
* @return
* @return the local IP
*/
public String getLocalIp() {
return localIp == null ? ip : localIp;
@ -119,9 +119,9 @@ public abstract class TransportCandidate {
}
/**
* Get the symetric candidate for this candidate if it exists.
* Get the symmetric candidate for this candidate if it exists.
*
* @return
* @return the symmetric candidate
*/
public TransportCandidate getSymmetric() {
return symmetric;
@ -157,7 +157,7 @@ public abstract class TransportCandidate {
/**
* Get the XMPPConnection use to send or receive this candidate
*
* @return
* @return the connection
*/
public XMPPConnection getConnection() {
return connection;
@ -175,7 +175,7 @@ public abstract class TransportCandidate {
/**
* Get the jingle's sessionId that is using this candidate
*
* @return
* @return the session ID
*/
public String getSessionId() {
return sessionId;

View File

@ -87,7 +87,7 @@ public abstract class TransportNegotiator extends JingleNegotiator {
/**
* Default constructor.
*
* @param js The Jingle session
* @param session The Jingle session
* @param transResolver The JingleTransportManager to use
*/
public TransportNegotiator(JingleSession session, TransportResolver transResolver, ContentNegotiator parentNegotiator) {
@ -503,10 +503,6 @@ public abstract class TransportNegotiator extends JingleNegotiator {
return result;
}
private boolean isOfferStarted() {
return resolver.isResolving() || resolver.isResolved();
}
/**
* Send an offer for a transport candidate
*
@ -658,7 +654,7 @@ public abstract class TransportNegotiator extends JingleNegotiator {
/**
* @param jingle
* @param jingleTransport
* @return
* @return the iq
*/
private IQ receiveSessionInitiateAction(Jingle jingle) throws XMPPException {
IQ response = null;
@ -682,7 +678,7 @@ public abstract class TransportNegotiator extends JingleNegotiator {
/**
* @param jingle
* @param jingleTransport
* @return
* @return the iq
*/
private IQ receiveTransportInfoAction(Jingle jingle) throws XMPPException {
IQ response = null;
@ -744,7 +740,7 @@ public abstract class TransportNegotiator extends JingleNegotiator {
/**
* @param jingle
* @return
* @return the iq
*/
private IQ receiveSessionAcceptAction(Jingle jingle) {
IQ response = null;

View File

@ -140,7 +140,7 @@ public abstract class TransportResolver {
/**
* Chack if the Transport Resolver is initialized
*
* @return
* @return true if initialized
*/
public synchronized boolean isInitialized() {
return initialized;

View File

@ -70,7 +70,7 @@ public class RosterExchangeManager {
/**
* Creates a new roster exchange manager.
*
* @param con a XMPPConnection which is used to send and receive messages.
* @param connection a XMPPConnection which is used to send and receive messages.
*/
public RosterExchangeManager(XMPPConnection connection) {
weakRefConnection = new WeakReference<XMPPConnection>(connection);

View File

@ -56,7 +56,7 @@ import java.util.List;
* Creates a socket connection to a XMPP server. This is the default connection
* to a Jabber server and is specified in the XMPP Core (RFC 3920).
*
* @see Connection
* @see XMPPConnection
* @author Matt Tucker
*/
public class TCPConnection extends XMPPConnection {