Ignore:
Timestamp:
2016-08-06T19:21:33+02:00 (8 years ago)
Author:
Don-vip
Message:

sonar - squid:S00100 - Method names should comply with a naming convention

Location:
trunk/src/org/openstreetmap/josm/data
Files:
8 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/data/osm/DataSet.java

    r10716 r10748  
    712712        synchronized (selectionLock) {
    713713            for (PrimitiveId o : osm) {
    714                 changed = changed | this.__toggleSelected(o);
     714                changed = changed | this.dotoggleSelected(o);
    715715            }
    716716            if (changed) {
    … …  
    731731    }
    732732
    733     private boolean __toggleSelected(PrimitiveId primitiveId) {
     733    private boolean dotoggleSelected(PrimitiveId primitiveId) {
    734734        OsmPrimitive primitive = getPrimitiveByIdChecked(primitiveId);
    735735        if (primitive == null)

  • trunk/src/org/openstreetmap/josm/data/osm/QuadBuckets.java

    r10001 r10748  
    134134        }
    135135
    136         boolean remove_content(T o) {
     136        boolean removeContent(T o) {
    137137            // If two threads try to remove item at the same time from different buckets of this QBLevel,
    138138            // it might happen that one thread removes bucket but don't remove parent because it still sees
    … …  
    147147            }
    148148            if (this.canRemove()) {
    149                 this.remove_from_parent();
     149                this.removeFromParent();
    150150            }
    151151            return ret;
    … …  
    158158         * is a dead end.
    159159         */
    160         void __split() {
     160        void doSplit() {
    161161            List<T> tmpcontent = content;
    162162            content = null;
    … …  
    165165                int idx = o.getBBox().getIndex(level);
    166166                if (idx == -1) {
    167                     __add_content(o);
     167                    doAddContent(o);
    168168                } else {
    169169                    getChild(idx).doAdd(o);
    … …  
    173173        }
    174174
    175         boolean __add_content(T o) {
     175        boolean doAddContent(T o) {
    176176            // The split_lock will keep two concurrent calls from overwriting content
    177177            if (content == null) {
    … …  
    190190        }
    191191
    192         private void search_contents(BBox searchBbox, List<T> result) {
     192        private void searchContents(BBox searchBbox, List<T> result) {
    193193            /*
    194194             * It is possible that this was created in a split
    … …  
    219219        }
    220220
    221         QBLevel<T> next_sibling() {
     221        QBLevel<T> findNextSibling() {
    222222            return (parent == null) ? null : parent.firstSiblingOf(this);
    223223        }
    … …  
    229229        QBLevel<T> nextSibling() {
    230230            QBLevel<T> next = this;
    231             QBLevel<T> sibling = next.next_sibling();
     231            QBLevel<T> sibling = next.findNextSibling();
    232232            // Walk back up the tree to find the next sibling node.
    233233            // It may be either a leaf or branch.
    … …  
    237237                    break;
    238238                }
    239                 sibling = next.next_sibling();
     239                sibling = next.findNextSibling();
    240240            }
    241241            return sibling;
    … …  
    289289                }
    290290            }
    291             __add_content(o);
     291            doAddContent(o);
    292292            if (isLeaf() && content.size() > MAX_OBJECTS_PER_LEVEL && level < QuadTiling.NR_LEVELS) {
    293                 __split();
     293                doSplit();
    294294            }
    295295        }
    … …  
    307307
    308308            if (this.hasContent()) {
    309                 search_contents(searchBbox, result);
     309                searchContents(searchBbox, result);
    310310            }
    311311
    … …  
    330330        }
    331331
    332         int index_of(QBLevel<T> findThis) {
     332        int indexOf(QBLevel<T> findThis) {
    333333            QBLevel<T>[] children = getChildren();
    334334            for (int i = 0; i < QuadTiling.TILES_PER_LEVEL; i++) {
    … …  
    359359        }
    360360
    361         void remove_from_parent() {
     361        void removeFromParent() {
    362362            if (parent == null)
    363363                return;
    … …  
    378378
    379379            if (parent.canRemove()) {
    380                 parent.remove_from_parent();
     380                parent.removeFromParent();
    381381            }
    382382        }
    … …  
    461461        searchCache = null; // Search cache might point to one of removed buckets
    462462        QBLevel<T> bucket = root.findBucket(t.getBBox());
    463         if (bucket.remove_content(t)) {
     463        if (bucket.removeContent(t)) {
    464464            size--;
    465465            return true;
    … …  
    564564            contentIndex--;
    565565            T object = peek();
    566             currentNode.remove_content(object);
     566            currentNode.removeContent(object);
    567567        }
    568568    }
    … …  
    607607        // of the nodes which is a parent of the search cache
    608608        while (tmp != null) {
    609             tmp.search_contents(searchBbox, ret);
     609            tmp.searchContents(searchBbox, ret);
    610610            tmp = tmp.parent;
    611611        }

  • trunk/src/org/openstreetmap/josm/data/projection/CustomProjection.java

    r10469 r10748  
    416416            if (parameters.get(Param.es.key) != null) {
    417417                double es = parseDouble(parameters, Param.es.key);
    418                 return Ellipsoid.create_a_es(a, es);
     418                return Ellipsoid.createAes(a, es);
    419419            }
    420420            if (parameters.get(Param.rf.key) != null) {
    421421                double rf = parseDouble(parameters, Param.rf.key);
    422                 return Ellipsoid.create_a_rf(a, rf);
     422                return Ellipsoid.createArf(a, rf);
    423423            }
    424424            if (parameters.get(Param.f.key) != null) {
    425425                double f = parseDouble(parameters, Param.f.key);
    426                 return Ellipsoid.create_a_f(a, f);
     426                return Ellipsoid.createAf(a, f);
    427427            }
    428428            if (parameters.get(Param.b.key) != null) {
    429429                double b = parseDouble(parameters, Param.b.key);
    430                 return Ellipsoid.create_a_b(a, b);
     430                return Ellipsoid.createAb(a, b);
    431431            }
    432432        }

  • trunk/src/org/openstreetmap/josm/data/projection/Ellipsoid.java

    r10680 r10748  
    1717     * Airy 1830
    1818     */
    19     public static final Ellipsoid Airy = Ellipsoid.create_a_b(6377563.396, 6356256.910);
     19    public static final Ellipsoid Airy = Ellipsoid.createAb(6377563.396, 6356256.910);
    2020
    2121    /**
    2222     * Modified Airy 1849
    2323     */
    24     public static final Ellipsoid AiryMod = Ellipsoid.create_a_b(6377340.189, 6356034.446);
     24    public static final Ellipsoid AiryMod = Ellipsoid.createAb(6377340.189, 6356034.446);
    2525
    2626    /**
    … …  
    2828     * same as GRS67 Modified
    2929     */
    30     public static final Ellipsoid AustSA = Ellipsoid.create_a_rf(6378160.0, 298.25);
     30    public static final Ellipsoid AustSA = Ellipsoid.createArf(6378160.0, 298.25);
    3131
    3232    /**
    3333     * Bessel 1841 ellipsoid
    3434     */
    35     public static final Ellipsoid Bessel1841 = Ellipsoid.create_a_rf(6377397.155, 299.1528128);
     35    public static final Ellipsoid Bessel1841 = Ellipsoid.createArf(6377397.155, 299.1528128);
    3636
    3737    /**
    3838     * Bessel 1841 (Namibia)
    3939     */
    40     public static final Ellipsoid BesselNamibia = Ellipsoid.create_a_rf(6377483.865, 299.1528128);
     40    public static final Ellipsoid BesselNamibia = Ellipsoid.createArf(6377483.865, 299.1528128);
    4141
    4242    /**
    4343     * Clarke 1866 ellipsoid
    4444     */
    45     public static final Ellipsoid Clarke1866 = Ellipsoid.create_a_b(6378206.4, 6356583.8);
     45    public static final Ellipsoid Clarke1866 = Ellipsoid.createAb(6378206.4, 6356583.8);
    4646
    4747    /**
    4848     * Clarke 1880 (modified)
    4949     */
    50     public static final Ellipsoid Clarke1880 = Ellipsoid.create_a_rf(6378249.145, 293.4663);
     50    public static final Ellipsoid Clarke1880 = Ellipsoid.createArf(6378249.145, 293.4663);
    5151
    5252    /**
    5353     * Clarke 1880 IGN (French national geographic institute)
    5454     */
    55     public static final Ellipsoid ClarkeIGN = Ellipsoid.create_a_b(6378249.2, 6356515.0);
     55    public static final Ellipsoid ClarkeIGN = Ellipsoid.createAb(6378249.2, 6356515.0);
    5656
    5757    /**
    5858     * Everest (Sabah &amp; Sarawak)
    5959     */
    60     public static final Ellipsoid EverestSabahSarawak = Ellipsoid.create_a_rf(6377298.556, 300.8017);
     60    public static final Ellipsoid EverestSabahSarawak = Ellipsoid.createArf(6377298.556, 300.8017);
    6161
    6262    /**
    6363     * GRS67 ellipsoid
    6464     */
    65     public static final Ellipsoid GRS67 = Ellipsoid.create_a_rf(6378160.0, 298.247167427);
     65    public static final Ellipsoid GRS67 = Ellipsoid.createArf(6378160.0, 298.247167427);
    6666
    6767    /**
    6868     * GRS80 ellipsoid
    6969     */
    70     public static final Ellipsoid GRS80 = Ellipsoid.create_a_rf(6378137.0, 298.257222101);
     70    public static final Ellipsoid GRS80 = Ellipsoid.createArf(6378137.0, 298.257222101);
    7171
    7272    /**
    … …  
    7575     * Proj.4 code: intl
    7676     */
    77     public static final Ellipsoid Hayford = Ellipsoid.create_a_rf(6378388.0, 297.0);
     77    public static final Ellipsoid Hayford = Ellipsoid.createArf(6378388.0, 297.0);
    7878
    7979    /**
    8080     * Helmert 1906
    8181     */
    82     public static final Ellipsoid Helmert = Ellipsoid.create_a_rf(6378200.0, 298.3);
     82    public static final Ellipsoid Helmert = Ellipsoid.createArf(6378200.0, 298.3);
    8383
    8484    /**
    8585     * Krassowsky 1940 ellipsoid
    8686     */
    87     public static final Ellipsoid Krassowsky = Ellipsoid.create_a_rf(6378245.0, 298.3);
     87    public static final Ellipsoid Krassowsky = Ellipsoid.createArf(6378245.0, 298.3);
    8888
    8989    /**
    9090     * WGS66 ellipsoid
    9191     */
    92     public static final Ellipsoid WGS66 = Ellipsoid.create_a_rf(6378145.0, 298.25);
     92    public static final Ellipsoid WGS66 = Ellipsoid.createArf(6378145.0, 298.25);
    9393
    9494    /**
    9595     * WGS72 ellipsoid
    9696     */
    97     public static final Ellipsoid WGS72 = Ellipsoid.create_a_rf(6378135.0, 298.26);
     97    public static final Ellipsoid WGS72 = Ellipsoid.createArf(6378135.0, 298.26);
    9898
    9999    /**
    100100     * WGS84 ellipsoid
    101101     */
    102     public static final Ellipsoid WGS84 = Ellipsoid.create_a_rf(6378137.0, 298.257223563);
     102    public static final Ellipsoid WGS84 = Ellipsoid.createArf(6378137.0, 298.257223563);
    103103
    104104    /**
    … …  
    162162     * @return the new ellipsoid
    163163     */
    164     public static Ellipsoid create_a_b(double a, double b) {
     164    public static Ellipsoid createAb(double a, double b) {
    165165        double e2 = (a*a - b*b) / (a*a);
    166166        double e = Math.sqrt(e2);
    … …  
    176176     * @return the new ellipsoid
    177177     */
    178     public static Ellipsoid create_a_es(double a, double es) {
     178    public static Ellipsoid createAes(double a, double es) {
    179179        double b = a * Math.sqrt(1.0 - es);
    180180        double e = Math.sqrt(es);
    … …  
    190190     * @return the new ellipsoid
    191191     */
    192     public static Ellipsoid create_a_f(double a, double f) {
     192    public static Ellipsoid createAf(double a, double f) {
    193193        double b = a * (1.0 - f);
    194194        double e2 = f * (2 - f);
    … …  
    205205     * @return the new ellipsoid
    206206     */
    207     public static Ellipsoid create_a_rf(double a, double rf) {
    208         return create_a_f(a, 1.0 / rf);
     207    public static Ellipsoid createArf(double a, double rf) {
     208        return createAf(a, 1.0 / rf);
    209209    }
    210210

  • trunk/src/org/openstreetmap/josm/data/projection/proj/AbstractProj.java

    r10378 r10748  
    137137     * @throws RuntimeException if the itteration does not converge.
    138138     */
    139     protected final double inv_mlfn(double arg) {
     139    protected final double invMlfn(double arg) {
    140140        double s, t, phi, k = 1.0/(1.0 - e2);
    141141        int i;

  • trunk/src/org/openstreetmap/josm/data/projection/proj/CassiniSoldner.java

    r9555 r10748  
    7979    @Override
    8080    public double[] invproject(double x, double y) {
    81         double ph1 = inv_mlfn(ml0 + y);
     81        double ph1 = invMlfn(ml0 + y);
    8282        double tn = Math.tan(ph1);
    8383        double t = tn * tn;

  • trunk/src/org/openstreetmap/josm/data/projection/proj/Sinusoidal.java

    r9955 r10748  
    4343            return new double[]{north, east / cos(north)};
    4444        } else {
    45             final double phi = inv_mlfn(north);
     45            final double phi = invMlfn(north);
    4646            double s = abs(phi);
    4747            final double lambda;

  • trunk/src/org/openstreetmap/josm/data/projection/proj/TransverseMercator.java

    r10250 r10748  
    156156    @Override
    157157    public double[] invproject(double x, double y) {
    158         double phi = inv_mlfn(ml0 + y);
     158        double phi = invMlfn(ml0 + y);
    159159
    160160        if (Math.abs(phi) >= Math.PI/2) {
Note: See TracChangeset for help on using the changeset viewer.