Changeset 1f92232 for apps


Ignore:
Timestamp:
Aug 26, 2018 1:50:40 PM (23 months ago)
Author:
zzz <zzz@…>
Branches:
master
Children:
e64ad7c
Parents:
db54edc
Message:

Findbugs all over

Location:
apps
Files:
9 edited

Legend:

Unmodified
Added
Removed
  • apps/addressbook/java/src/net/i2p/addressbook/Daemon.java

    rdb54edc r1f92232  
    190190                if (knownNames != null) {
    191191                    oldDest = null;
    192                     isKnown = key != null ? knownNames.contains(key) : null;
     192                    isKnown = key != null ? knownNames.contains(key) : false;
    193193                } else {
    194194                    oldDest = key != null ? router.lookup(key) : null;
     
    804804    public static void main(String[] args) {
    805805        Daemon daemon = new Daemon();
    806         if (args != null && args.length > 0 && args[0].equals("test"))
     806        if (args.length > 0 && args[0].equals("test"))
    807807            daemon.test(args);
    808808        else
     
    824824    }
    825825   
     826    /**
     827     *  @param args may be null
     828     */
    826829    public void run(String[] args) {
    827830        _running = true;
    828831        String settingsLocation = "config.txt";
    829832        File homeFile;
    830         if (args.length > 0) {
     833        if (args != null && args.length > 0) {
    831834            homeFile = new SecureDirectory(args[0]);
    832835            if (!homeFile.isAbsolute())
  • apps/addressbook/java/src/net/metanotion/io/block/BlockFile.java

    rdb54edc r1f92232  
    591591                metaIndex.close();
    592592
    593                 Set<String> oi = openIndices.keySet();
    594                 Iterator<String> i = oi.iterator();
    595                 Object k;
    596                 while(i.hasNext()) {
    597                         k = i.next();
    598                         BSkipList bsl = openIndices.get(k);
     593                for (BSkipList bsl : openIndices.values()) {
    599594                        bsl.close();
    600595                }
  • apps/i2psnark/java/src/org/klomp/snark/SnarkManager.java

    rdb54edc r1f92232  
    202202
    203203    /** host names for config form */
    204     public static final Set<String> KNOWN_OPENTRACKERS = new HashSet<String>(Arrays.asList(new String[] {
     204    static final Set<String> KNOWN_OPENTRACKERS = new HashSet<String>(Arrays.asList(new String[] {
    205205        "tracker.welterde.i2p", "cfmqlafjfmgkzbt4r3jsfyhgsr5abgxryl6fnz3d3y5a365di5aa.b32.i2p",
    206206        "opentracker.dg2.i2p", "w7tpbzncbcocrqtwwm3nezhnnsw4ozadvi2hmvzdhrqzfxfum7wa.b32.i2p",
  • apps/i2ptunnel/java/src/net/i2p/i2ptunnel/ui/TunnelConfig.java

    rdb54edc r1f92232  
    812812        };
    813813
    814     public static final Set<String> _noShowSet = new HashSet<String>(128);
    815     public static final Set<String> _nonProxyNoShowSet = new HashSet<String>(4);
     814    static final Set<String> _noShowSet = new HashSet<String>(128);
     815    static final Set<String> _nonProxyNoShowSet = new HashSet<String>(4);
    816816    static {
    817817        _noShowSet.addAll(Arrays.asList(_noShowOpts));
  • apps/routerconsole/java/src/net/i2p/router/web/helpers/ConfigSummaryHandler.java

    rdb54edc r1f92232  
    103103                    }
    104104                }
    105                 for (Iterator<Integer> iter = sections.keySet().iterator(); iter.hasNext(); ) {
    106                     int i = iter.next();
     105                for (Iterator<Map.Entry<Integer, String>> iter = sections.entrySet().iterator(); iter.hasNext(); ) {
     106                    Map.Entry<Integer, String> e = iter.next();
     107                    Integer i = e.getKey();
    107108                    if (toDelete.contains(i)) {
    108                         String removedName = sections.get(i);
     109                        String removedName = e.getValue();
    109110                        iter.remove();
    110111                        addFormNotice(_t("Removed") + ": " + removedName);
  • apps/sam/java/src/net/i2p/sam/SAMStreamSession.java

    rdb54edc r1f92232  
    2222import java.util.Iterator;
    2323import java.util.List;
     24import java.util.Map;
    2425import java.util.Properties;
    2526import java.util.Set;
     
    397398     */
    398399    protected int createSocketHandler ( I2PSocket s, int id ) {
    399         SAMStreamSessionSocketReader reader = null;
    400         StreamSender sender = null;
     400        SAMStreamSessionSocketReader reader;
     401        StreamSender sender;
    401402        if (id == 0) {
    402403            id = createUniqueId();
     
    469470
    470471        synchronized (handlersMap) {
    471             reader = handlersMap.remove(Integer.valueOf(id));
    472             sender = sendersMap.remove(Integer.valueOf(id));
     472            Integer iid = Integer.valueOf(id);
     473            reader = handlersMap.remove(iid);
     474            sender = sendersMap.remove(iid);
    473475        }
    474476
     
    487489     */
    488490    private void removeAllSocketHandlers() {
    489         Integer id;
    490         Set<Integer> keySet;
    491         Iterator<Integer> iter;
    492 
    493491        synchronized (handlersMap) {
    494             keySet = handlersMap.keySet();
    495             iter = keySet.iterator();
    496            
    497             while (iter.hasNext()) {
    498                  id = iter.next();
    499                  handlersMap.get(id).stopRunning();
     492            for (Map.Entry<Integer, SAMStreamSessionSocketReader> e : handlersMap.entrySet()) {
     493                 Integer id = e.getKey();
     494                 e.getValue().stopRunning();
    500495                 sendersMap.get(id).shutDownGracefully();
    501496            }
  • apps/streaming/java/src/net/i2p/client/streaming/impl/MessageInputStream.java

    rdb54edc r1f92232  
    289289                buf.append(" not ready blocks: [");
    290290                long notAvailable = 0;
    291                 for (Long id : _notYetReadyBlocks.keySet()) {
    292                     ByteArray ba = _notYetReadyBlocks.get(id);
     291                for (Map.Entry<Long, ByteArray> e : _notYetReadyBlocks.entrySet()) {
     292                    Long id = e.getKey();
     293                    ByteArray ba = e.getValue();
    293294                    buf.append(id).append(" ");
    294                    
    295                     if (ba != null)
    296                         notAvailable += ba.getValid();
     295                    notAvailable += ba.getValid();
    297296                }
    298297               
     
    590589                buf.append(" not ready blocks: [");
    591590                long notAvailable = 0;
    592                 for (Long id : _notYetReadyBlocks.keySet()) {
    593                     ByteArray ba = _notYetReadyBlocks.get(id);
     591                for (Map.Entry<Long, ByteArray> e : _notYetReadyBlocks.entrySet()) {
     592                    Long id = e.getKey();
     593                    ByteArray ba = e.getValue();
    594594                    buf.append(id).append(" ");
    595                     if (ba != null)
    596                         notAvailable += ba.getValid();
     595                    notAvailable += ba.getValid();
    597596                }
    598597                buf.append("] not ready bytes: ").append(notAvailable);
  • apps/susidns/src/java/src/i2p/susi/dns/TranslateSVGServlet.java

    rdb54edc r1f92232  
    3232    private static final long serialVersionUID = 18638760L;
    3333    private final I2PAppContext _context;
    34     private final String DIR = "/svg";
     34    private static final String DIR = "/svg";
    3535
    3636    public TranslateSVGServlet() {
  • apps/susimail/src/src/i2p/susi/webmail/WebMail.java

    rdb54edc r1f92232  
    899899                                synchronized(_so) {
    900900                                        mc = _so.caches.get(DIR_FOLDER);
     901                                        if (mc == null) {
     902                                                _so.error += "Internal error, no folder\n";
     903                                                return;
     904                                        }
    901905                                        while (mc.isLoading()) {
    902906                                                try {
     
    911915                                // fetch the mail outside the lock
    912916                                // TODO, would be better to add each email as we get it
    913                                 if (mc != null) {
    914                                         found = mc.getMail(MailCache.FetchMode.HEADER);
    915                                 }
     917                                found = mc.getMail(MailCache.FetchMode.HEADER);
    916918                        }
    917919                        if (log.shouldDebug()) log.debug("CW.FNM connected? " + connected + " found? " + found);
     
    28992901                                        sessionObject.error += relay.error;
    29002902                                }
    2901                                 sessionObject.info.replace(_t("Sending mail.") + '\n', "");
     2903                                sessionObject.info = sessionObject.info.replace(_t("Sending mail.") + '\n', "");
    29022904                        }
    29032905                }
     
    33783380                }
    33793381                out.println(button(LOGOUT, _t("Logout") ));
    3380                 if (mail.hasBody() && !mc.getFolderName().equals(DIR_DRAFTS)) {
     3382                if (hasHeader && mail.hasBody() && !mc.getFolderName().equals(DIR_DRAFTS)) {
    33813383                        // can't move unless has body
    33823384                        // can't move from drafts
Note: See TracChangeset for help on using the changeset viewer.