Changeset 28b6675


Ignore:
Timestamp:
Nov 20, 2013 12:02:44 PM (7 years ago)
Author:
dg2-new <dg2-new@…>
Branches:
master
Children:
eabcc96
Parents:
413ad6b0
Message:

findbugs BOB/SAM. more to do.

Location:
apps
Files:
6 edited

Legend:

Unmodified
Added
Removed
  • apps/BOB/src/net/i2p/BOB/DoCMDS.java

    r413ad6b0 r28b6675  
    818818                                                                                        database.add(Arg, nickinfo);
    819819                                                                                        nickinfo.add(P_NICKNAME, Arg);
    820                                                                                         nickinfo.add(P_STARTING, new Boolean(false));
    821                                                                                         nickinfo.add(P_RUNNING, new Boolean(false));
    822                                                                                         nickinfo.add(P_STOPPING, new Boolean(false));
    823                                                                                         nickinfo.add(P_QUIET, new Boolean(false));
     820                                                                                        nickinfo.add(P_STARTING, Boolean.valueOf(false));
     821                                                                                        nickinfo.add(P_RUNNING, Boolean.valueOf(false));
     822                                                                                        nickinfo.add(P_STOPPING, Boolean.valueOf(false));
     823                                                                                        nickinfo.add(P_QUIET, Boolean.valueOf(false));
    824824                                                                                        nickinfo.add(P_INHOST, "localhost");
    825825                                                                                        nickinfo.add(P_OUTHOST, "localhost");
     
    10771077                                                                                                        if (prt > 1 && prt < 65536) {
    10781078                                                                                                                try {
    1079                                                                                                                         nickinfo.add(P_OUTPORT, new Integer(prt));
     1079                                                                                                                        nickinfo.add(P_OUTPORT, Integer.valueOf(prt));
    10801080                                                                                                                } catch (Exception ex) {
    10811081                                                                                                                        try {
     
    13561356                                                                                                }
    13571357
    1358                                                                                                 nickinfo.add(P_STOPPING, new Boolean(true));
     1358                                                                                                nickinfo.add(P_STOPPING, Boolean.valueOf(true));
    13591359                                                                                                try {
    13601360                                                                                                        wunlock();
  • apps/BOB/src/net/i2p/BOB/MUXlisten.java

    r413ad6b0 r28b6675  
    105105                        this.database.getWriteLock();
    106106                        this.info.getWriteLock();
    107                         this.info.add("STARTING", new Boolean(false));
     107                        this.info.add("STARTING", Boolean.valueOf(false));
    108108                        this.info.releaseWriteLock();
    109109                        this.database.releaseWriteLock();
     
    113113                        this.database.getWriteLock();
    114114                        this.info.getWriteLock();
    115                         this.info.add("STARTING", new Boolean(false));
     115                        this.info.add("STARTING", Boolean.valueOf(false));
    116116                        this.info.releaseWriteLock();
    117117                        this.database.releaseWriteLock();
     
    121121                        this.database.getWriteLock();
    122122                        this.info.getWriteLock();
    123                         this.info.add("STARTING", new Boolean(false));
     123                        this.info.add("STARTING", Boolean.valueOf(false));
    124124                        this.info.releaseWriteLock();
    125125                        this.database.releaseWriteLock();
     
    205205                                                        wlock();
    206206                                                        try {
    207                                                                 info.add("STARTING", new Boolean(false));
     207                                                                info.add("STARTING", Boolean.valueOf(false));
    208208                                                        } catch (Exception e) {
    209209                                                                wunlock();
     
    259259                                wlock();
    260260                                try {
    261                                         info.add("STARTING", new Boolean(false));
    262                                         info.add("STOPPING", new Boolean(true));
    263                                         info.add("RUNNING", new Boolean(false));
     261                                        info.add("STARTING", Boolean.valueOf(false));
     262                                        info.add("STOPPING", Boolean.valueOf(true));
     263                                        info.add("RUNNING", Boolean.valueOf(false));
    264264                                } catch (Exception e) {
    265265                                        lock.set(false);
     
    310310                                wlock();
    311311                                try {
    312                                         info.add("STARTING", new Boolean(false));
    313                                         info.add("STOPPING", new Boolean(false));
    314                                         info.add("RUNNING", new Boolean(false));
     312                                        info.add("STARTING", Boolean.valueOf(false));
     313                                        info.add("STOPPING", Boolean.valueOf(false));
     314                                        info.add("RUNNING", Boolean.valueOf(false));
    315315                                } catch (Exception e) {
    316316                                        lock.set(false);
  • apps/BOB/src/net/i2p/BOB/TCPtoI2P.java

    r413ad6b0 r28b6675  
    7272        private static String lnRead(InputStream in) throws IOException {
    7373                String S = "";
     74        StringBuilder builder = new StringBuilder();
    7475                int b;
    7576                char c;
     
    8687                        }
    8788                        c = (char) (b & 0x7f); // We only care about ASCII
    88                         S = S + c;
     89            builder.append(c);
    8990                }
    90                 return S;
     91                return builder.toString();
    9192        }
    9293
  • apps/sam/java/src/net/i2p/sam/SAMStreamSession.java

    r413ad6b0 r28b6675  
    306306
    307307        synchronized (handlersMapLock) {
    308             handlersMap.put(new Integer(id), reader);
    309             sendersMap.put(new Integer(id), sender);
     308            handlersMap.put(Integer.valueOf(id), reader);
     309            sendersMap.put(Integer.valueOf(id), sender);
    310310        }
    311311
     
    333333    protected SAMStreamSessionSocketReader getSocketReader ( int id ) {
    334334        synchronized (handlersMapLock) {
    335             return handlersMap.get(new Integer(id));
     335            return handlersMap.get(Integer.valueOf(id));
    336336        }
    337337    }
    338338    private StreamSender getSender(int id) {
    339339        synchronized (handlersMapLock) {
    340             return sendersMap.get(new Integer(id));
     340            return sendersMap.get(Integer.valueOf(id));
    341341        }
    342342    }
     
    350350    protected boolean checkSocketHandlerId ( int id ) {
    351351        synchronized (handlersMapLock) {
    352             return (!(handlersMap.get(new Integer(id)) == null));
     352            return (!(handlersMap.get(Integer.valueOf(id)) == null));
    353353        }
    354354    }
     
    364364
    365365        synchronized (handlersMapLock) {
    366             reader = handlersMap.remove(new Integer(id));
    367             sender = sendersMap.remove(new Integer(id));
     366            reader = handlersMap.remove(Integer.valueOf(id));
     367            sender = sendersMap.remove(Integer.valueOf(id));
    368368        }
    369369
  • apps/sam/java/src/net/i2p/sam/SAMUtils.java

    r413ad6b0 r28b6675  
    190190    private static String dumpProperties(Properties props) {
    191191        Enumeration names = props.propertyNames();
     192        StringBuilder builder = new StringBuilder();
    192193        String msg = "";
    193194        String key, val;
     
    203204                firstIter = false;
    204205            }
    205             msg += " \"" + key + "\" -> \"" + val + "\"";
     206            builder.append(" \"" + key + "\" -> \"" + val + "\"");
    206207        }
    207208       
    208         return msg;
     209        return builder.toString();
    209210    }
    210211   
  • apps/sam/java/src/net/i2p/sam/SAMv3Handler.java

    r413ad6b0 r28b6675  
    219219                public SessionRecord( String dest, Properties props, SAMv3Handler handler )
    220220                {
    221                         m_dest = new String(dest) ;
     221                        m_dest = dest;
    222222                        m_props = new Properties() ;
    223223                        m_props.putAll(props);
     
    236236                synchronized public String getDest()
    237237                {
    238                         return new String(m_dest) ;
     238                        return m_dest;
    239239                }
    240240                synchronized public Properties getProps()
Note: See TracChangeset for help on using the changeset viewer.