User: ejort   
  Date: 02/01/05 04:08:53

  Modified:    src/main/org/jboss/jms/jndi JBossMQProvider.java
                        JMSProviderLoader.java
  Log:
  Guarded debug logging
  
  Revision  Changes    Path
  1.8       +7 -4      jboss/src/main/org/jboss/jms/jndi/JBossMQProvider.java
  
  Index: JBossMQProvider.java
  ===================================================================
  RCS file: /cvsroot/jboss/jboss/src/main/org/jboss/jms/jndi/JBossMQProvider.java,v
  retrieving revision 1.7
  retrieving revision 1.8
  diff -u -r1.7 -r1.8
  --- JBossMQProvider.java      2001/07/07 00:59:18     1.7
  +++ JBossMQProvider.java      2002/01/05 12:08:53     1.8
  @@ -31,7 +31,7 @@
    * Created: Fri Dec 22 09:34:04 2000
    * 6/22/01 - hchirino - The queue/topic jndi references are now configed via JMX
    *
  - * @version <pre>$Revision: 1.7 $</pre>
  + * @version <pre>$Revision: 1.8 $</pre>
    * @author  <a href="mailto:[EMAIL PROTECTED]";>Peter Antman</a>
    * @author  <a href="mailto:[EMAIL PROTECTED]";>Hiram Chirino</a>
    * @author  <a href="mailto:[EMAIL PROTECTED]";>Jason Dillon</a>
  @@ -82,9 +82,11 @@
       */
      public Context getInitialContext() throws NamingException {
         Context ctx = null;
  +      boolean debug = log.isDebugEnabled();
         if (providerURL == null) {
            // Use default
  -         log.debug("no provider url; connecting to local JNDI");
  +         if (debug)
  +            log.debug("no provider url; connecting to local JNDI");
            ctx = new InitialContext(); // Only for JBoss embedded now
         } else {
            // Try another location
  @@ -95,11 +97,12 @@
            props.put(SECURITY_MANAGER, hasJndiSecurityManager);
            props.put(Context.URL_PKG_PREFIXES, URL_PKG_PREFIXES);
   
  -         log.debug("connecting to remote JNDI with props: " + props);
  +         if (debug)
  +            log.debug("connecting to remote JNDI with props: " + props);
            ctx = new InitialContext(props);
         }
   
  -      if (log.isDebugEnabled()) {
  +      if (debug) {
            log.debug("created context: " + ctx);
         }
         return ctx;
  
  
  
  1.8       +4 -3      jboss/src/main/org/jboss/jms/jndi/JMSProviderLoader.java
  
  Index: JMSProviderLoader.java
  ===================================================================
  RCS file: /cvsroot/jboss/jboss/src/main/org/jboss/jms/jndi/JMSProviderLoader.java,v
  retrieving revision 1.7
  retrieving revision 1.8
  diff -u -r1.7 -r1.8
  --- JMSProviderLoader.java    2001/11/12 06:52:17     1.7
  +++ JMSProviderLoader.java    2002/01/05 12:08:53     1.8
  @@ -40,7 +40,7 @@
    *
    * @author  <a href="mailto:[EMAIL PROTECTED]";>Hiram Chirino</a>
    * @author  <a href="mailto:[EMAIL PROTECTED]";>Jason Dillon</a>
  - * @version $Revision: 1.7 $
  + * @version $Revision: 1.8 $
    */
   public class JMSProviderLoader 
      extends ServiceMBeanSupport
  @@ -173,8 +173,9 @@
      private void bind(Context ctx, String name, Object val)
         throws NamingException
      {
  -      log.debug("attempting to bind " + val + " to " + name);
  -      
  +      if (log.isDebugEnabled())
  +         log.debug("attempting to bind " + val + " to " + name);
  +
         // Bind val to name in ctx, and make sure that all
         // intermediate contexts exist
         Name n = ctx.getNameParser("").parse(name);
  
  
  

_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development

Reply via email to