Changeset 8619fd2


Ignore:
Timestamp:
Nov 28, 2011 9:52:49 PM (8 years ago)
Author:
zzz <zzz@…>
Branches:
master
Children:
6b811b3
Parents:
d9dcb1e
Message:

dont use equalsIgnoreCase() for booleans

Files:
14 edited

Legend:

Unmodified
Added
Removed
  • apps/i2ptunnel/java/src/net/i2p/i2ptunnel/I2PTunnel.java

    rd9dcb1e r8619fd2  
    739739            boolean isShared = true;
    740740            if (args.length > 1) {
    741                 if ("true".equalsIgnoreCase(args[1].trim())) {
     741                if (Boolean.valueOf(args[1].trim()).booleanValue()) {
    742742                    isShared = true;
    743743                    if (args.length == 3)
     
    808808            boolean isShared = true;
    809809            if (args.length > 1) {
    810                 if ("true".equalsIgnoreCase(args[1].trim())) {
     810                if (Boolean.valueOf(args[1].trim()).booleanValue()) {
    811811                    isShared = true;
    812812                    if (args.length == 3)
     
    879879            boolean isShared = true;
    880880            if (args.length > 2) {
    881                 if ("true".equalsIgnoreCase(args[2].trim())) {
     881                if (Boolean.valueOf(args[2].trim()).booleanValue()) {
    882882                    isShared = true;
    883883                } else if ("false".equalsIgnoreCase(args[2].trim())) {
     
    946946            boolean isShared = false;
    947947            if (args.length > 1)
    948                 isShared = "true".equalsIgnoreCase(args[1].trim());
     948                isShared = Boolean.valueOf(args[1].trim()).booleanValue();
    949949
    950950            ownDest = !isShared;
     
    990990            boolean isShared = false;
    991991            if (args.length == 2)
    992                 isShared = "true".equalsIgnoreCase(args[1].trim());
     992                isShared = Boolean.valueOf(args[1].trim()).booleanValue();
    993993
    994994            ownDest = !isShared;
  • apps/i2ptunnel/java/src/net/i2p/i2ptunnel/I2PTunnelHTTPClientBase.java

    rd9dcb1e r8619fd2  
    102102        // If the socket is an InternalSocket, no auth required.
    103103        String authRequired = getTunnel().getClientOptions().getProperty(PROP_AUTH);
    104         if (authRequired != null && (authRequired.equalsIgnoreCase("true") || authRequired.equalsIgnoreCase("basic"))) {
     104        if (Boolean.valueOf(authRequired).booleanValue() || "basic".equalsIgnoreCase(authRequired)) {
    105105            if (s instanceof InternalSocket) {
    106106                if (_log.shouldLog(Log.INFO))
  • apps/i2ptunnel/java/src/net/i2p/i2ptunnel/I2PTunnelServer.java

    rd9dcb1e r8619fd2  
    178178            String usePool = getTunnel().getClientOptions().getProperty(PROP_USE_POOL);
    179179            if (usePool != null)
    180                 _usePool = "true".equalsIgnoreCase(usePool);
     180                _usePool = Boolean.valueOf(usePool).booleanValue();
    181181            else
    182182                _usePool = DEFAULT_USE_POOL;
  • apps/i2ptunnel/java/src/net/i2p/i2ptunnel/TunnelController.java

    rd9dcb1e r8619fd2  
    483483    public String getSharedClient() { return _config.getProperty("sharedClient", "true"); }
    484484    /** default true */
    485     public boolean getStartOnLoad() { return "true".equalsIgnoreCase(_config.getProperty("startOnLoad", "true")); }
     485    public boolean getStartOnLoad() { return Boolean.valueOf(_config.getProperty("startOnLoad", "true")).booleanValue(); }
    486486    public boolean getPersistentClientKey() { return Boolean.valueOf(_config.getProperty("option.persistentClientKey")).booleanValue(); }
    487487    public String getMyDestination() {
  • apps/i2ptunnel/java/src/net/i2p/i2ptunnel/web/EditBean.java

    rd9dcb1e r8619fd2  
    115115        TunnelController tun = getController(tunnel);
    116116        if (tun != null)
    117             return "true".equalsIgnoreCase(tun.getSharedClient());
     117            return Boolean.valueOf(tun.getSharedClient()).booleanValue();
    118118        else
    119119            return false;
  • apps/i2ptunnel/java/src/net/i2p/i2ptunnel/web/IndexBean.java

    rd9dcb1e r8619fd2  
    267267        // Only modify other shared tunnels
    268268        // if the current tunnel is shared, and of supported type
    269         if ("true".equalsIgnoreCase(cur.getSharedClient()) && isClient(cur.getType())) {
     269        if (Boolean.valueOf(cur.getSharedClient()).booleanValue() && isClient(cur.getType())) {
    270270            // all clients use the same I2CP session, and as such, use the same I2CP options
    271271            List controllers = _group.getControllers();
     
    279279                // Only modify this non-current tunnel
    280280                // if it belongs to a shared destination, and is of supported type
    281                 if ("true".equalsIgnoreCase(c.getSharedClient()) && isClient(c.getType())) {
     281                if (Boolean.valueOf(c.getSharedClient()).booleanValue() && isClient(c.getType())) {
    282282                    Properties cOpt = c.getConfig("");
    283283                    if (_tunnelQuantity != null) {
  • apps/routerconsole/java/src/net/i2p/router/web/ConfigNetHandler.java

    rd9dcb1e r8619fd2  
    170170            String oldAutoHost = _context.getProperty(ConfigNetHelper.PROP_I2NP_NTCP_AUTO_IP, "true");
    171171            String sAutoPort = _context.getProperty(ConfigNetHelper.PROP_I2NP_NTCP_AUTO_PORT, "true");
    172             boolean oldAutoPort = "true".equalsIgnoreCase(sAutoPort);
     172            boolean oldAutoPort = Boolean.valueOf(sAutoPort).booleanValue();
    173173            if (_ntcpHostname == null) _ntcpHostname = "";
    174174            if (_ntcpPort == null) _ntcpPort = "";
  • apps/routerconsole/java/src/net/i2p/router/web/ConfigNetHelper.java

    rd9dcb1e r8619fd2  
    6969
    7070    public String getEnableTimeSyncChecked() {
    71         String disabled = _context.getProperty(Timestamper.PROP_DISABLED, "false");
    72         if ( (disabled != null) && ("true".equalsIgnoreCase(disabled)) )
     71        boolean disabled = _context.getBooleanProperty(Timestamper.PROP_DISABLED);
     72        if (disabled)
    7373            return "";
    7474        else
     
    7878    /** @param prop must default to false */
    7979    public String getChecked(String prop) {
    80         if (Boolean.valueOf(_context.getProperty(prop)).booleanValue())
     80        if (_context.getBooleanProperty(prop))
    8181            return CHECKED;
    8282        return "";
     
    131131    /** default true */
    132132    public String getUpnpChecked() {
    133         if (Boolean.valueOf(_context.getProperty(TransportManager.PROP_ENABLE_UPNP, "true")).booleanValue())
     133        if (_context.getBooleanPropertyDefaultTrue(TransportManager.PROP_ENABLE_UPNP))
    134134            return CHECKED;
    135135        return "";
  • core/java/src/freenet/support/CPUInformation/CPUID.java

    rd9dcb1e r8619fd2  
    261261        try{
    262262        String wantedProp = System.getProperty("jcpuid.enable", "true");
    263         boolean wantNative = "true".equalsIgnoreCase(wantedProp);
     263        boolean wantNative = Boolean.valueOf(wantedProp).booleanValue();
    264264        if (wantNative) {
    265265            boolean loaded = loadGeneric();
  • core/java/src/net/i2p/util/NativeBigInteger.java

    rd9dcb1e r8619fd2  
    461461        try{
    462462            String wantedProp = System.getProperty("jbigi.enable", "true");
    463             boolean wantNative = "true".equalsIgnoreCase(wantedProp);
     463            boolean wantNative = Boolean.valueOf(wantedProp).booleanValue();
    464464            if (wantNative) {
    465465                debug("trying loadGeneric");
  • router/java/src/net/i2p/router/Router.java

    rd9dcb1e r8619fd2  
    11171117        try { _context.logManager().shutdown(); } catch (Throwable t) { }
    11181118        if (ALLOW_DYNAMIC_KEYS) {
    1119             if (Boolean.valueOf(_context.getProperty(PROP_DYNAMIC_KEYS)).booleanValue())
     1119            if (_context.getBooleanProperty(PROP_DYNAMIC_KEYS))
    11201120                killKeys();
    11211121        }
  • router/java/src/net/i2p/router/TunnelPoolSettings.java

    rd9dcb1e r8619fd2  
    237237    private static final boolean getBoolean(String str, boolean defaultValue) {
    238238        if (str == null) return defaultValue;
    239         boolean v = "TRUE".equalsIgnoreCase(str) || "YES".equalsIgnoreCase(str);
     239        boolean v = Boolean.valueOf(str).booleanValue() || "YES".equalsIgnoreCase(str);
    240240        return v;
    241241    }
  • router/java/src/net/i2p/router/transport/CommSystemFacadeImpl.java

    rd9dcb1e r8619fd2  
    330330            _log.info("old: " + ohost + " config: " + name + " auto: " + enabled + " status: " + status);
    331331        if (enabled.equalsIgnoreCase("always") ||
    332             (enabled.equalsIgnoreCase("true") && status == STATUS_OK)) {
     332            (Boolean.valueOf(enabled).booleanValue() && status == STATUS_OK)) {
    333333            String nhost = UDPProps.getProperty(UDPAddress.PROP_HOST);
    334334            if (_log.shouldLog(Log.INFO))
     
    353353        } else if (ohost == null || ohost.length() <= 0) {
    354354            return;
    355         } else if (enabled.equalsIgnoreCase("true") && status != STATUS_OK) {
     355        } else if (Boolean.valueOf(enabled).booleanValue() && status != STATUS_OK) {
    356356            // UDP transitioned to not-OK, turn off NTCP address
    357357            // This will commonly happen at startup if we were initially OK
  • router/java/src/net/i2p/router/transport/udp/UDPTransport.java

    rd9dcb1e r8619fd2  
    585585                if (oldIP != null &&
    586586                    System.getProperty("wrapper.version") != null &&
    587                     Boolean.valueOf(_context.getProperty(PROP_LAPTOP_MODE)).booleanValue() &&
     587                    _context.getBooleanProperty(PROP_LAPTOP_MODE) &&
    588588                    now - lastChanged > 10*60*1000 &&
    589589                    _context.router().getUptime() < 10*60*1000) {
     
    614614        if (isPubliclyRoutable(addr))
    615615            return true;
    616         return Boolean.valueOf(_context.getProperty("i2np.udp.allowLocal")).booleanValue();
     616        return _context.getBooleanProperty("i2np.udp.allowLocal");
    617617    }
    618618   
     
    13651365                    // substantial data has changed, so if we are in 'dynamic keys' mode, restart the
    13661366                    // router hard and regenerate a new identity
    1367                     if ("true".equalsIgnoreCase(_context.getProperty(Router.PROP_DYNAMIC_KEYS, "false"))) {
     1367                    if (_context.getBooleanProperty(Router.PROP_DYNAMIC_KEYS)) {
    13681368                        if (_log.shouldLog(Log.ERROR))
    13691369                            _log.error("SSU address updated. new address: "
Note: See TracChangeset for help on using the changeset viewer.