Ignore:
Timestamp:
2015-05-11T13:34:53+02:00 (5 years ago)
Author:
Don-vip
Message:

squid:S00116 - Field names should comply with a naming convention

Location:
trunk/src/org/openstreetmap/josm/io
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/io/DiffResultProcessor.java

    r8287 r8346  
    3535
    3636    private static class DiffResultEntry {
    37         public long new_id;
    38         public int new_version;
     37        private long newId;
     38        private int newVersion;
    3939    }
    4040
     
    128128                processed.add(p);
    129129                if (!p.isDeleted()) {
    130                     p.setOsmId(entry.new_id, entry.new_version);
     130                    p.setOsmId(entry.newId, entry.newVersion);
    131131                    p.setVisible(true);
    132132                } else {
     
    174174                    DiffResultEntry entry = new DiffResultEntry();
    175175                    if (atts.getValue("new_id") != null) {
    176                         entry.new_id = Long.parseLong(atts.getValue("new_id"));
     176                        entry.newId = Long.parseLong(atts.getValue("new_id"));
    177177                    }
    178178                    if (atts.getValue("new_version") != null) {
    179                         entry.new_version = Integer.parseInt(atts.getValue("new_version"));
     179                        entry.newVersion = Integer.parseInt(atts.getValue("new_version"));
    180180                    }
    181181                    diffResults.put(id, entry);
  • trunk/src/org/openstreetmap/josm/io/NmeaReader.java

    r8342 r8346  
    154154    }
    155155    public int getParserZeroCoordinates() {
    156         return ps.zero_coord;
     156        return ps.zeroCoord;
    157157    }
    158158    public int getParserChecksumErrors() {
    159         return ps.checksum_errors+ps.no_checksum;
     159        return ps.checksumErrors+ps.noChecksum;
    160160    }
    161161    public int getParserMalformed() {
     
    180180                return;
    181181            sb.append((char)loopstart_char);
    182             ps.p_Date="010100"; // TODO date problem
     182            ps.pDate="010100"; // TODO date problem
    183183            while(true) {
    184184                // don't load unparsable files completely to memory
     
    209209    private static class NMEAParserState {
    210210        protected Collection<WayPoint> waypoints = new ArrayList<>();
    211         protected String p_Time;
    212         protected String p_Date;
    213         protected WayPoint p_Wp;
    214 
    215         protected int success = 0; // number of successfully parsend sentences
     211        protected String pTime;
     212        protected String pDate;
     213        protected WayPoint pWp;
     214
     215        protected int success = 0; // number of successfully parsed sentences
    216216        protected int malformed = 0;
    217         protected int checksum_errors = 0;
    218         protected int no_checksum = 0;
     217        protected int checksumErrors = 0;
     218        protected int noChecksum = 0;
    219219        protected int unknown = 0;
    220         protected int zero_coord = 0;
     220        protected int zeroCoord = 0;
    221221    }
    222222
     
    243243                }
    244244                if (Integer.parseInt(chkstrings[1].substring(0,2),16) != chk) {
    245                     ps.checksum_errors++;
    246                     ps.p_Wp=null;
     245                    ps.checksumErrors++;
     246                    ps.pWp=null;
    247247                    return false;
    248248                }
    249249            } else {
    250                 ps.no_checksum++;
     250                ps.noChecksum++;
    251251            }
    252252            // now for the content
     
    254254            String accu;
    255255
    256             WayPoint currentwp = ps.p_Wp;
    257             String currentDate = ps.p_Date;
     256            WayPoint currentwp = ps.pWp;
     257            String currentDate = ps.pDate;
    258258
    259259            // handle the packet content
     
    271271
    272272                if ((latLon.lat()==0.0) && (latLon.lon()==0.0)) {
    273                     ps.zero_coord++;
     273                    ps.zeroCoord++;
    274274                    return false;
    275275                }
     
    279279                Date d = readTime(currentDate+accu);
    280280
    281                 if((ps.p_Time==null) || (currentwp==null) || !ps.p_Time.equals(accu)) {
     281                if((ps.pTime==null) || (currentwp==null) || !ps.pTime.equals(accu)) {
    282282                    // this node is newer than the previous, create a new waypoint.
    283283                    // no matter if previous WayPoint was null, we got something
    284284                    // better now.
    285                     ps.p_Time=accu;
     285                    ps.pTime=accu;
    286286                    currentwp = new WayPoint(latLon);
    287287                }
     
    384384                        e[GPRMC.LENGTH_EAST.position]
    385385                );
    386                 if((latLon.lat()==0.0) && (latLon.lon()==0.0)) {
    387                     ps.zero_coord++;
     386                if(latLon.lat()==0.0 && latLon.lon()==0.0) {
     387                    ps.zeroCoord++;
    388388                    return false;
    389389                }
     
    394394                Date d = readTime(currentDate+time);
    395395
    396                 if((ps.p_Time==null) || (currentwp==null) || !ps.p_Time.equals(time)) {
     396                if(ps.pTime==null || currentwp==null || !ps.pTime.equals(time)) {
    397397                    // this node is newer than the previous, create a new waypoint.
    398                     ps.p_Time=time;
     398                    ps.pTime=time;
    399399                    currentwp = new WayPoint(latLon);
    400400                }
     
    426426                return false;
    427427            }
    428             ps.p_Date = currentDate;
    429             if(ps.p_Wp != currentwp) {
    430                 if(ps.p_Wp!=null) {
    431                     ps.p_Wp.setTime();
    432                 }
    433                 ps.p_Wp = currentwp;
     428            ps.pDate = currentDate;
     429            if(ps.pWp != currentwp) {
     430                if(ps.pWp!=null) {
     431                    ps.pWp.setTime();
     432                }
     433                ps.pWp = currentwp;
    434434                ps.waypoints.add(currentwp);
    435435                ps.success++;
     
    441441            // out of bounds and such
    442442            ps.malformed++;
    443             ps.p_Wp=null;
     443            ps.pWp=null;
    444444            return false;
    445445        }
  • trunk/src/org/openstreetmap/josm/io/NoteWriter.java

    r8225 r8346  
    2323public class NoteWriter extends XmlWriter {
    2424
    25     private final DateFormat ISO8601_FORMAT = DateUtils.newIsoDateTimeFormat();
     25    private final DateFormat iso8601Format = DateUtils.newIsoDateTimeFormat();
    2626
    2727    /**
     
    5353            out.print("lat=\"" + note.getLatLon().lat() + "\" ");
    5454            out.print("lon=\"" + note.getLatLon().lon() + "\" ");
    55             out.print("created_at=\"" + ISO8601_FORMAT.format(note.getCreatedAt()) + "\" ");
     55            out.print("created_at=\"" + iso8601Format.format(note.getCreatedAt()) + "\" ");
    5656            if (note.getClosedAt() != null) {
    57                 out.print("closed_at=\"" + ISO8601_FORMAT.format(note.getClosedAt()) + "\" ");
     57                out.print("closed_at=\"" + iso8601Format.format(note.getClosedAt()) + "\" ");
    5858            }
    5959
     
    7272        out.print("    <comment");
    7373        out.print(" action=\"" + comment.getNoteAction() + "\" ");
    74         out.print("timestamp=\"" + ISO8601_FORMAT.format(comment.getCommentTimestamp()) + "\" ");
     74        out.print("timestamp=\"" + iso8601Format.format(comment.getCommentTimestamp()) + "\" ");
    7575        if (comment.getUser() != null && !comment.getUser().equals(User.getAnonymous())) {
    7676            out.print("uid=\"" + comment.getUser().getId() + "\" ");
Note: See TracChangeset for help on using the changeset viewer.