Fixes issue 24868 Clean code: fix unlikely argument and remove some

unused variables. ManagedBeanManagerImpl and
JdbcConnPoolAppStatsProvider need specific review.
(Remove unused imports)

appserver/connectors/admin/src/main/java/org/glassfish/connectors/admin/cli/ConnectorSecurityMap.java

@@ -16,7 +16,6 @@
 
 package org.glassfish.connectors.admin.cli;
 
-import com.sun.enterprise.util.LocalStringManagerImpl;
 import org.glassfish.api.I18n;
 import org.glassfish.connectors.config.ConnectorConnectionPool;
 import org.glassfish.connectors.config.SecurityMap;

appserver/deployment/dol/src/main/java/com/sun/enterprise/deployment/annotation/handlers/ManagedScheduledExecutorDefinitionListHandler.java

@@ -21,7 +21,6 @@
 import jakarta.enterprise.concurrent.ManagedScheduledExecutorDefinition;
 import jakarta.inject.Inject;
 
-import java.lang.System.Logger;
 import java.util.Set;
 
 import org.glassfish.apf.AnnotationHandlerFor;

appserver/deployment/jakartaee-core/src/main/java/org/glassfish/javaee/core/deployment/GetContextRootCommand.java

@@ -19,7 +19,6 @@
 import org.glassfish.api.Param;
 import org.glassfish.internal.data.ApplicationInfo;
 import org.glassfish.internal.data.ApplicationRegistry;
-import com.sun.enterprise.util.LocalStringManagerImpl;
 import com.sun.enterprise.deployment.Application;
 import com.sun.enterprise.deployment.BundleDescriptor;
 import org.glassfish.deployment.common.DeploymentProperties;

appserver/persistence/cmp/generator-database/src/main/java/com/sun/jdo/spi/persistence/generator/database/DatabaseOutputStream.java

@@ -23,10 +23,11 @@
 
 package com.sun.jdo.spi.persistence.generator.database;
 
-import java.io.*;
-import java.sql.*;
-import java.util.ResourceBundle;
-import org.glassfish.persistence.common.I18NHelper;
+import java.io.OutputStream;
+import java.sql.Connection;
+import java.sql.PreparedStatement;
+import java.sql.SQLException;
+
 import com.sun.jdo.spi.persistence.utility.logging.Logger;
 
 

appserver/persistence/cmp/support-sqlstore/src/main/java/com/sun/jdo/spi/persistence/support/sqlstore/ejb/EJBHelper.java

@@ -23,12 +23,17 @@
 
 import java.util.ResourceBundle;
 
-import jakarta.transaction.*;
 import org.glassfish.persistence.common.I18NHelper;
-import com.sun.jdo.api.persistence.support.PersistenceManagerFactory;
+
 import com.sun.jdo.api.persistence.support.JDOFatalInternalException;
-import com.sun.jdo.spi.persistence.utility.logging.Logger;
-import com.sun.jdo.spi.persistence.support.sqlstore.LogHelperPersistenceManager;
+import com.sun.jdo.api.persistence.support.PersistenceManagerFactory;
+
+import jakarta.transaction.RollbackException;
+import jakarta.transaction.Synchronization;
+import jakarta.transaction.SystemException;
+import jakarta.transaction.Transaction;
+import jakarta.transaction.TransactionManager;
+import jakarta.transaction.UserTransaction;
 
   /** Provides helper methods for a Forte for Java implementation with the
    * application server specific information in the distributed transaction

nucleus/admin/config-api/src/main/java/org/glassfish/config/support/DomainXml.java

@@ -21,7 +21,6 @@
 import com.sun.enterprise.config.serverbeans.Server;
 import com.sun.enterprise.module.ModulesRegistry;
 import com.sun.enterprise.module.bootstrap.StartupContext;
-import com.sun.enterprise.universal.i18n.LocalStringsImpl;
 import com.sun.enterprise.util.LocalStringManagerImpl;
 import com.sun.enterprise.util.io.FileUtils;
 

nucleus/deployment/admin/src/main/java/org/glassfish/deployment/admin/GetHostAndPortCommand.java

@@ -28,7 +28,6 @@
 import com.sun.enterprise.config.serverbeans.VirtualServer;
 import com.sun.enterprise.config.serverbeans.Cluster;
 import com.sun.enterprise.util.HostAndPort;
-import com.sun.enterprise.util.LocalStringManagerImpl;
 import com.sun.enterprise.util.StringUtils;
 import java.util.Collection;
 import org.glassfish.grizzly.config.dom.Http;

nucleus/deployment/admin/src/main/java/org/glassfish/deployment/admin/InstanceLifecycleModuleCommand.java

@@ -32,7 +32,6 @@
 import org.glassfish.hk2.api.PerLookup;
 import org.jvnet.hk2.config.Transaction;
 
-import com.sun.enterprise.util.LocalStringManagerImpl;
 import com.sun.enterprise.config.serverbeans.Applications;
 import com.sun.enterprise.config.serverbeans.Application;
 import com.sun.enterprise.config.serverbeans.Domain;

nucleus/deployment/admin/src/main/java/org/glassfish/deployment/admin/IsSnifferUserVisibleCommand.java

@@ -17,7 +17,6 @@
 package org.glassfish.deployment.admin;
 
 import com.sun.enterprise.config.serverbeans.Application;
-import com.sun.enterprise.util.LocalStringManagerImpl;
 import org.glassfish.api.ActionReport;
 import org.glassfish.api.Param;
 import org.glassfish.api.admin.AdminCommand;

nucleus/deployment/admin/src/main/java/org/glassfish/deployment/admin/ListAppRefsCommand.java

@@ -31,7 +31,6 @@
 import com.sun.enterprise.config.serverbeans.Module;
 import org.glassfish.api.admin.ExecuteOn;
 import org.glassfish.api.admin.RuntimeType;
-import com.sun.enterprise.util.LocalStringManagerImpl;
 import java.util.Collection;
 
 import jakarta.inject.Inject;

nucleus/deployment/admin/src/main/java/org/glassfish/deployment/admin/ListLibrariesCommand.java

@@ -30,7 +30,6 @@
 
 import org.glassfish.hk2.api.PerLookup;
 import jakarta.inject.Inject;
-import com.sun.enterprise.util.LocalStringManagerImpl;
 import com.sun.enterprise.util.io.FileUtils;
 import com.sun.enterprise.config.serverbeans.Domain;
 

nucleus/security/services/src/main/java/org/glassfish/security/services/impl/AuthenticationServiceImpl.java

@@ -21,7 +21,6 @@
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Map;
-import java.util.logging.Logger;
 
 import jakarta.inject.Inject;
 import jakarta.inject.Singleton;