Lint Report

Assert: Assertions

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/GLUtessellatorImpl.java:250: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 247         switch (which) {
 248             case GLU.GLU_TESS_TOLERANCE:
 249 /* tolerance should be in range [0..1] */
 250                 assert (0.0 <= relTolerance && relTolerance <= 1.0);
 251                 value[value_offset] = relTolerance;
 252                 break;

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/GLUtessellatorImpl.java:254: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 251                 value[value_offset] = relTolerance;
 252                 break;
 253             case GLU.GLU_TESS_WINDING_RULE:
 254                 assert (windingRule == GLU.GLU_TESS_WINDING_ODD ||
 255                         windingRule == GLU.GLU_TESS_WINDING_NONZERO ||
 256                         windingRule == GLU.GLU_TESS_WINDING_POSITIVE ||

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/GLUtessellatorImpl.java:262: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 259                 value[value_offset] = windingRule;
 260                 break;
 261             case GLU.GLU_TESS_BOUNDARY_ONLY:
 262                 assert (boundaryOnly == true || boundaryOnly == false);
 263                 value[value_offset] = boundaryOnly ? 1 : 0;
 264                 break;

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Geom.java:72: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

  69     static double EdgeEval(GLUvertex u, GLUvertex v, GLUvertex w) {
  70         double gapL, gapR;
  71 
  72         assert (VertLeq(u, v) && VertLeq(v, w));
  73 
  74         gapL = v.s - u.s;

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Geom.java:91: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

  88     static double EdgeSign(GLUvertex u, GLUvertex v, GLUvertex w) {
  89         double gapL, gapR;
  90 
  91         assert (VertLeq(u, v) && VertLeq(v, w));
  92 
  93         gapL = v.s - u.s;

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Geom.java:121: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 118          */
 119         double gapL, gapR;
 120 
 121         assert (TransLeq(u, v) && TransLeq(v, w));
 122 
 123         gapL = v.t - u.t;

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Geom.java:144: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 141          */
 142         double gapL, gapR;
 143 
 144         assert (TransLeq(u, v) && TransLeq(v, w));
 145 
 146         gapL = v.t - u.t;

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:654: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 651         while (fHead.next != fHead) {
 652             __gl_meshZapFace(fHead.next);
 653         }
 654         assert (mesh.vHead.next == mesh.vHead);
 655     }
 656

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:690: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 687 
 688         fPrev = fHead;
 689         for (fPrev = fHead; (f = fPrev.next) != fHead; fPrev = f) {
 690             assert (f.prev == fPrev);
 691             e = f.anEdge;
 692             do {

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:693: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 690             assert (f.prev == fPrev);
 691             e = f.anEdge;
 692             do {
 693                 assert (e.Sym != e);
 694                 assert (e.Sym.Sym == e);
 695                 assert (e.Lnext.Onext.Sym == e);

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:694: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 691             e = f.anEdge;
 692             do {
 693                 assert (e.Sym != e);
 694                 assert (e.Sym.Sym == e);
 695                 assert (e.Lnext.Onext.Sym == e);
 696                 assert (e.Onext.Sym.Lnext == e);

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:695: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 692             do {
 693                 assert (e.Sym != e);
 694                 assert (e.Sym.Sym == e);
 695                 assert (e.Lnext.Onext.Sym == e);
 696                 assert (e.Onext.Sym.Lnext == e);
 697                 assert (e.Lface == f);

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:696: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 693                 assert (e.Sym != e);
 694                 assert (e.Sym.Sym == e);
 695                 assert (e.Lnext.Onext.Sym == e);
 696                 assert (e.Onext.Sym.Lnext == e);
 697                 assert (e.Lface == f);
 698                 e = e.Lnext;

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:697: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 694                 assert (e.Sym.Sym == e);
 695                 assert (e.Lnext.Onext.Sym == e);
 696                 assert (e.Onext.Sym.Lnext == e);
 697                 assert (e.Lface == f);
 698                 e = e.Lnext;
 699             } while (e != f.anEdge);

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:701: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 698                 e = e.Lnext;
 699             } while (e != f.anEdge);
 700         }
 701         assert (f.prev == fPrev && f.anEdge == null && f.data == null);
 702 
 703         vPrev = vHead;

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:705: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 702 
 703         vPrev = vHead;
 704         for (vPrev = vHead; (v = vPrev.next) != vHead; vPrev = v) {
 705             assert (v.prev == vPrev);
 706             e = v.anEdge;
 707             do {

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:708: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 705             assert (v.prev == vPrev);
 706             e = v.anEdge;
 707             do {
 708                 assert (e.Sym != e);
 709                 assert (e.Sym.Sym == e);
 710                 assert (e.Lnext.Onext.Sym == e);

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:709: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 706             e = v.anEdge;
 707             do {
 708                 assert (e.Sym != e);
 709                 assert (e.Sym.Sym == e);
 710                 assert (e.Lnext.Onext.Sym == e);
 711                 assert (e.Onext.Sym.Lnext == e);

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:710: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 707             do {
 708                 assert (e.Sym != e);
 709                 assert (e.Sym.Sym == e);
 710                 assert (e.Lnext.Onext.Sym == e);
 711                 assert (e.Onext.Sym.Lnext == e);
 712                 assert (e.Org == v);

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:711: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 708                 assert (e.Sym != e);
 709                 assert (e.Sym.Sym == e);
 710                 assert (e.Lnext.Onext.Sym == e);
 711                 assert (e.Onext.Sym.Lnext == e);
 712                 assert (e.Org == v);
 713                 e = e.Onext;

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:712: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 709                 assert (e.Sym.Sym == e);
 710                 assert (e.Lnext.Onext.Sym == e);
 711                 assert (e.Onext.Sym.Lnext == e);
 712                 assert (e.Org == v);
 713                 e = e.Onext;
 714             } while (e != v.anEdge);

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:716: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 713                 e = e.Onext;
 714             } while (e != v.anEdge);
 715         }
 716         assert (v.prev == vPrev && v.anEdge == null && v.data == null);
 717 
 718         ePrev = eHead;

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:720: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 717 
 718         ePrev = eHead;
 719         for (ePrev = eHead; (e = ePrev.next) != eHead; ePrev = e) {
 720             assert (e.Sym.next == ePrev.Sym);
 721             assert (e.Sym != e);
 722             assert (e.Sym.Sym == e);

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:721: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 718         ePrev = eHead;
 719         for (ePrev = eHead; (e = ePrev.next) != eHead; ePrev = e) {
 720             assert (e.Sym.next == ePrev.Sym);
 721             assert (e.Sym != e);
 722             assert (e.Sym.Sym == e);
 723             assert (e.Org != null);

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:722: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 719         for (ePrev = eHead; (e = ePrev.next) != eHead; ePrev = e) {
 720             assert (e.Sym.next == ePrev.Sym);
 721             assert (e.Sym != e);
 722             assert (e.Sym.Sym == e);
 723             assert (e.Org != null);
 724             assert (e.Sym.Org != null);

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:725: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 722             assert (e.Sym.Sym == e);
 723             assert (e.Org != null);
 724             assert (e.Sym.Org != null);
 725             assert (e.Lnext.Onext.Sym == e);
 726             assert (e.Onext.Sym.Lnext == e);
 727         }

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:726: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 723             assert (e.Org != null);
 724             assert (e.Sym.Org != null);
 725             assert (e.Lnext.Onext.Sym == e);
 726             assert (e.Onext.Sym.Lnext == e);
 727         }
 728         assert (e.Sym.next == ePrev.Sym

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Mesh.java:728: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 725             assert (e.Lnext.Onext.Sym == e);
 726             assert (e.Onext.Sym.Lnext == e);
 727         }
 728         assert (e.Sym.next == ePrev.Sym
 729                 && e.Sym == mesh.eHeadSym
 730                 && e.Sym.Sym == e

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Normal.java:92: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

  89     static void Normalize(double[] v) {
  90         double len = v[0] * v[0] + v[1] * v[1] + v[2] * v[2];
  91 
  92         assert (len > 0);
  93         len = Math.sqrt(len);
  94         v[0] /= len;

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/PriorityQHeap.java:105: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 102                 ++child;
 103             }
 104 
 105             assert (child <= max);
 106 
 107             hChild = n[child].handle;

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/PriorityQHeap.java:207: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 204         if (initialized) {
 205             FloatUp(curr);
 206         }
 207         assert (free != Integer.MAX_VALUE);
 208         return free;
 209     }

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/PriorityQHeap.java:239: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 236         PriorityQ.PQhandleElem[] h = handles;
 237         int curr;
 238 
 239         assert (hCurr >= 1 && hCurr <= max && h[hCurr].key != null);
 240 
 241         curr = h[hCurr].node;

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/PriorityQSort.java:217: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 214                 return Integer.MAX_VALUE;
 215             }
 216         }
 217         assert curr != Integer.MAX_VALUE;
 218         keys[curr] = keyNew;
 219

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/PriorityQSort.java:273: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 270             return;
 271         }
 272         curr = -(curr + 1);
 273         assert curr < max && keys[curr] != null;
 274 
 275         keys[curr] = null;

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Render.java:118: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 115              */
 116             if (f.inside && !f.marked) {
 117                 RenderMaximumFaceGroup(tess, f);
 118                 assert (f.marked);
 119             }
 120         }

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Render.java:287: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 284             /* Just add the triangle to a triangle list, so we can render all
 285              * the separate triangles at once.
 286              */
 287             assert (size == 1);
 288             tess.lonelyTriList = AddToTrail(e.Lface, tess.lonelyTriList);
 289         }

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Render.java:343: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 340                 tess.callVertexOrVertexData( e.Sym.Org.data);
 341             }
 342 
 343             assert (size == 0);
 344             tess.callEndOrEndData();
 345         }

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Render.java:371: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 368                 tess.callVertexOrVertexData( e.Sym.Org.data);
 369             }
 370 
 371             assert (size == 0);
 372             tess.callEndOrEndData();
 373         }

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:164: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 161              * deleted with zero winding number (ie. it better not get merged
 162              * with a real edge).
 163              */
 164             assert (reg.eUp.winding == 0);
 165         }
 166         reg.eUp.activeRegion = null;

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:175: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 172 /*
 173  * Replace an upper edge which needs fixing (see ConnectRightVertex).
 174  */ {
 175         assert (reg.fixUpperEdge);
 176         if (!Mesh.__gl_meshDelete(reg.eUp)) return false;
 177         reg.fixUpperEdge = false;

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:357: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 354         /* Insert the new right-going edges in the dictionary */
 355         e = eFirst;
 356         do {
 357             assert (Geom.VertLeq(e.Org, e.Sym.Org));
 358             AddRegionBelow(tess, regUp, e.Sym);
 359             e = e.Onext;

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:399: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 396             ePrev = e;
 397         }
 398         regPrev.dirty = true;
 399         assert (regPrev.windingNumber - e.winding == reg.windingNumber);
 400 
 401         if (cleanUp) {

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:576: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 573         GLUhalfEdge eLo = regLo.eUp;
 574         GLUhalfEdge e;
 575 
 576         assert (!Geom.VertEq(eUp.Sym.Org, eLo.Sym.Org));
 577 
 578         if (Geom.VertLeq(eUp.Sym.Org, eLo.Sym.Org)) {

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:623: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 620         GLUvertex orgMin;
 621         GLUhalfEdge e;
 622 
 623         assert (!Geom.VertEq(dstLo, dstUp));
 624         assert (Geom.EdgeSign(dstUp, tess.event, orgUp) <= 0);
 625         assert (Geom.EdgeSign(dstLo, tess.event, orgLo) >= 0);

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:624: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 621         GLUhalfEdge e;
 622 
 623         assert (!Geom.VertEq(dstLo, dstUp));
 624         assert (Geom.EdgeSign(dstUp, tess.event, orgUp) <= 0);
 625         assert (Geom.EdgeSign(dstLo, tess.event, orgLo) >= 0);
 626         assert (orgUp != tess.event && orgLo != tess.event);

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:625: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 622 
 623         assert (!Geom.VertEq(dstLo, dstUp));
 624         assert (Geom.EdgeSign(dstUp, tess.event, orgUp) <= 0);
 625         assert (Geom.EdgeSign(dstLo, tess.event, orgLo) >= 0);
 626         assert (orgUp != tess.event && orgLo != tess.event);
 627         assert (!regUp.fixUpperEdge && !regLo.fixUpperEdge);

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:626: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 623         assert (!Geom.VertEq(dstLo, dstUp));
 624         assert (Geom.EdgeSign(dstUp, tess.event, orgUp) <= 0);
 625         assert (Geom.EdgeSign(dstLo, tess.event, orgLo) >= 0);
 626         assert (orgUp != tess.event && orgLo != tess.event);
 627         assert (!regUp.fixUpperEdge && !regLo.fixUpperEdge);
 628

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:627: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 624         assert (Geom.EdgeSign(dstUp, tess.event, orgUp) <= 0);
 625         assert (Geom.EdgeSign(dstLo, tess.event, orgLo) >= 0);
 626         assert (orgUp != tess.event && orgLo != tess.event);
 627         assert (!regUp.fixUpperEdge && !regLo.fixUpperEdge);
 628 
 629         if (orgUp == orgLo) return false;	/* right endpoints are the same */

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:646: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 643 
 644         Geom.EdgeIntersect(dstUp, orgUp, dstLo, orgLo, isect);
 645         /* The following properties are guaranteed: */
 646         assert (Math.min(orgUp.t, dstUp.t) <= isect.t);
 647         assert (isect.t <= Math.max(orgLo.t, dstLo.t));
 648         assert (Math.min(dstLo.s, dstUp.s) <= isect.s);

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:647: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 644         Geom.EdgeIntersect(dstUp, orgUp, dstLo, orgLo, isect);
 645         /* The following properties are guaranteed: */
 646         assert (Math.min(orgUp.t, dstUp.t) <= isect.t);
 647         assert (isect.t <= Math.max(orgLo.t, dstLo.t));
 648         assert (Math.min(dstLo.s, dstUp.s) <= isect.s);
 649         assert (isect.s <= Math.max(orgLo.s, orgUp.s));

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:648: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 645         /* The following properties are guaranteed: */
 646         assert (Math.min(orgUp.t, dstUp.t) <= isect.t);
 647         assert (isect.t <= Math.max(orgLo.t, dstLo.t));
 648         assert (Math.min(dstLo.s, dstUp.s) <= isect.s);
 649         assert (isect.s <= Math.max(orgLo.s, orgUp.s));
 650

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:649: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 646         assert (Math.min(orgUp.t, dstUp.t) <= isect.t);
 647         assert (isect.t <= Math.max(orgLo.t, dstLo.t));
 648         assert (Math.min(dstLo.s, dstUp.s) <= isect.s);
 649         assert (isect.s <= Math.max(orgLo.s, orgUp.s));
 650 
 651         if (Geom.VertLeq(isect, tess.event)) {

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:947: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 944             /* e.Org is an unprocessed vertex - just combine them, and wait
 945              * for e.Org to be pulled from the queue
 946              */
 947             assert (TOLERANCE_NONZERO);
 948             SpliceMergeVertices(tess, e, vEvent.anEdge);
 949             return;

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:968: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 965         /* vEvent coincides with e.Sym.Org, which has already been processed.
 966          * Splice in the additional right-going edges.
 967          */
 968         assert (TOLERANCE_NONZERO);
 969         regUp = TopRightRegion(regUp);
 970         reg = RegionBelow(regUp);

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:977: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 974             /* Here e.Sym.Org has only a single fixable edge going right.
 975              * We can delete it since now we have some real right-going edges.
 976              */
 977             assert (eTopLeft != eTopRight);   /* there are some left edges too */
 978             DeleteRegion(tess, reg);
 979             if (!Mesh.__gl_meshDelete(eTopRight)) throw new RuntimeException();

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:1175: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

1172              * created by ConnectRightVertex().
1173              */
1174             if (!reg.sentinel) {
1175                 assert (reg.fixUpperEdge);
1176                 assert (++fixedEdges == 1);
1177             }

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:1176: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

1173              */
1174             if (!reg.sentinel) {
1175                 assert (reg.fixUpperEdge);
1176                 assert (++fixedEdges == 1);
1177             }
1178             assert (reg.windingNumber == 0);

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:1178: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

1175                 assert (reg.fixUpperEdge);
1176                 assert (++fixedEdges == 1);
1177             }
1178             assert (reg.windingNumber == 0);
1179             DeleteRegion(tess, reg);
1180 /*    __gl_meshDelete( reg.eUp );*/

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/Sweep.java:1281: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

1278         for (f = mesh.fHead.next; f != mesh.fHead; f = fNext) {
1279             fNext = f.next;
1280             e = f.anEdge;
1281             assert (e.Lnext != e);
1282 
1283             if (e.Lnext.Lnext == e) {

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/TessMono.java:92: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

  89          * be close to the edge we want.
  90          */
  91         up = face.anEdge;
  92         assert (up.Lnext != up && up.Lnext.Lnext != up);
  93 
  94         for (; Geom.VertLeq(up.Sym.Org, up.Org); up = up.Onext.Sym)

../../src/main/java/gov/nasa/worldwind/util/glu/tessellator/TessMono.java:128: Assertions are unreliable. Use BuildConfig.DEBUG conditional checks instead.

 125         /* Now lo.Org == up.Sym.Org == the leftmost vertex.  The remaining region
 126          * can be tessellated in a fan from this leftmost vertex.
 127          */
 128         assert (lo.Lnext != up);
 129         while (lo.Lnext.Lnext != up) {
 130             GLUhalfEdge tempHalfEdge = Mesh.__gl_meshConnect(lo.Lnext, lo);

Priority: 6 / 10
Category: Correctness

Severity: Warning

Explanation: Assertions.

Assertions are not checked at runtime. There are ways to request that they be used by Dalvik (adb shell setprop debug.assert 1), but the property is ignored in many places and can not be relied upon. Instead, perform conditional checking inside if (BuildConfig.DEBUG) { } blocks. That constant is a static final boolean which is true in debug builds and false in release builds, and the Java compiler completely removes all code inside the if-body from the app.

For example, you can replace assert speed > 0 with if (BuildConfig.DEBUG && !(speed > 0)) { throw new AssertionError() }.

(Note: This lint check does not flag assertions purely asserting nullness or non-nullness; these are typically more intended for tools usage than runtime checks.)

More info: https://code.google.com/p/android/issues/detail?id=65183

To suppress this error, use the issue id "Assert" as explained in the Suppressing Warnings and Errors section.

NewApi: Calling new methods on older versions

../../../worldwind: The SDK platform-tools version (23.0.1) is too old to check APIs compiled with API 24; please update

Note: This issue has an associated quickfix operation in Android Studio/IntelliJ & Eclipse/ADT
Priority: 6 / 10
Category: Correctness

Severity: Error

Explanation: Calling new methods on older versions.

This check scans through all the Android API calls in the application and warns about any calls that are not available on all versions targeted by this application (according to its minimum SDK attribute in the manifest).

If you really want to use this API and don't need to support older devices just set the minSdkVersion in your build.gradle or AndroidManifest.xml files.

If your code is deliberately accessing newer APIs, and you have ensured (e.g. with conditional execution) that this code will only ever be called on a supported platform, then you can annotate your class or method with the @TargetApi annotation specifying the local minimum SDK to apply, such as @TargetApi(11), such that this check considers 11 rather than your manifest file's minimum SDK as the required API level.

If you are deliberately setting android: attributes in style definitions, make sure you place this in a values-vNN folder in order to avoid running into runtime conflicts on certain devices where manufacturers have added custom attributes whose ids conflict with the new ones on later platforms.

Similarly, you can use tools:targetApi="11" in an XML file to indicate that the element will only be inflated in an adequate context.

More info:

To suppress this error, use the issue id "NewApi" as explained in the Suppressing Warnings and Errors section.

IconLocation: Image defined in density-independent drawable folder

../../src/main/res/drawable/gov_nasa_worldwind_worldtopobathy2004053.png:

Found bitmap drawable res/drawable/gov_nasa_worldwind_worldtopobathy2004053.png in densityless folder

Priority: 5 / 10
Category: Usability:Icons

Severity: Warning

Explanation: Image defined in density-independent drawable folder.

The res/drawable folder is intended for density-independent graphics such as shapes defined in XML. For bitmaps, move it to drawable-mdpi and consider providing higher and lower resolution versions in drawable-ldpi, drawable-hdpi and drawable-xhdpi. If the icon really is density independent (for example a solid color) you can place it in drawable-nodpi.

More info: http://developer.android.com/guide/practices/screens_support.html

To suppress this error, use the issue id "IconLocation" as explained in the Suppressing Warnings and Errors section.

AppLinksAutoVerifyError

Disabled By: Default
Priority: 5 / 10
Category: Correctness

Severity: Error

Explanation: App Links Auto Verification Failure.

Ensures that app links are correctly set and associated with website.

More info: https://g.co/appindexing/applinks

To suppress this error, use the issue id "AppLinksAutoVerifyError" as explained in the Suppressing Warnings and Errors section.

AppLinksAutoVerifyWarning

Disabled By: Default
Priority: 5 / 10
Category: Correctness

Severity: Warning

Explanation: Potential App Links Auto Verification Failure.

Ensures that app links are correctly set and associated with website.

More info: https://g.co/appindexing/applinks

To suppress this error, use the issue id "AppLinksAutoVerifyWarning" as explained in the Suppressing Warnings and Errors section.

BackButton

Disabled By: Default
Priority: 6 / 10
Category: Usability

Severity: Warning

Explanation: Back button.

According to the Android Design Guide,

"Other platforms use an explicit back button with label to allow the user to navigate up the application's hierarchy. Instead, Android uses the main action bar's app icon for hierarchical navigation and the navigation bar's back button for temporal navigation."
This check is not very sophisticated (it just looks for buttons with the label "Back"), so it is disabled by default to not trigger on common scenarios like pairs of Back/Next buttons to paginate through screens.

More info: http://developer.android.com/design/patterns/pure-android.html

To suppress this error, use the issue id "BackButton" as explained in the Suppressing Warnings and Errors section.

EasterEgg

Disabled By: Default
Priority: 6 / 10
Category: Security

Severity: Warning

Explanation: Code contains easter egg.

An "easter egg" is code deliberately hidden in the code, both from potential users and even from other developers. This lint check looks for code which looks like it may be hidden from sight.

More info:

To suppress this error, use the issue id "EasterEgg" as explained in the Suppressing Warnings and Errors section.

FieldGetter

Disabled By: Default
Priority: 4 / 10
Category: Performance

Severity: Warning

Explanation: Using getter instead of field.

Accessing a field within the class that defines a getter for that field is at least 3 times faster than calling the getter. For simple getters that do nothing other than return the field, you might want to just reference the local field directly instead.

NOTE: As of Android 2.3 (Gingerbread), this optimization is performed automatically by Dalvik, so there is no need to change your code; this is only relevant if you are targeting older versions of Android.

More info: http://developer.android.com/guide/practices/design/performance.html#internal_get_set

To suppress this error, use the issue id "FieldGetter" as explained in the Suppressing Warnings and Errors section.

GoogleAppIndexingApiWarning

Note: This issue has an associated quickfix operation in Android Studio/IntelliJ
Disabled By: Default
Priority: 5 / 10
Category: Usability

Severity: Warning

Explanation: Missing support for Google App Indexing Api.

Adds URLs to get your app into the Google index, to get installs and traffic to your app from Google Search.

More info: https://g.co/AppIndexing/AndroidStudio

To suppress this error, use the issue id "GoogleAppIndexingApiWarning" as explained in the Suppressing Warnings and Errors section.

GradleDependency

Note: This issue has an associated quickfix operation in Android Studio/IntelliJ
Disabled By: Command line flag
Priority: 4 / 10
Category: Correctness

Severity: Warning

Explanation: Obsolete Gradle Dependency.

This detector looks for usages of libraries where the version you are using is not the current stable release. Using older versions is fine, and there are cases where you deliberately want to stick with an older version. However, you may simply not be aware that a more recent version is available, and that is what this lint check helps find.

More info:

To suppress this error, use the issue id "GradleDependency" as explained in the Suppressing Warnings and Errors section.

IconExpectedSize

Disabled By: Default
Priority: 5 / 10
Category: Usability:Icons

Severity: Warning

Explanation: Icon has incorrect size.

There are predefined sizes (for each density) for launcher icons. You should follow these conventions to make sure your icons fit in with the overall look of the platform.

More info: http://developer.android.com/design/style/iconography.html

To suppress this error, use the issue id "IconExpectedSize" as explained in the Suppressing Warnings and Errors section.

LogConditional

Disabled By: Default
Priority: 5 / 10
Category: Performance

Severity: Warning

Explanation: Unconditional Logging Calls.

The BuildConfig class (available in Tools 17) provides a constant, "DEBUG", which indicates whether the code is being built in release mode or in debug mode. In release mode, you typically want to strip out all the logging calls. Since the compiler will automatically remove all code which is inside a "if (false)" check, surrounding your logging calls with a check for BuildConfig.DEBUG is a good idea.

If you really intend for the logging to be present in release mode, you can suppress this warning with a @SuppressLint annotation for the intentional logging calls.

More info:

To suppress this error, use the issue id "LogConditional" as explained in the Suppressing Warnings and Errors section.

MangledCRLF

Note: This issue has an associated quickfix operation in Eclipse/ADT
Disabled By: Default
Priority: 2 / 10
Category: Correctness

Severity: Error

Explanation: Mangled file line endings.

On Windows, line endings are typically recorded as carriage return plus newline: \r\n.

This detector looks for invalid line endings with repeated carriage return characters (without newlines). Previous versions of the ADT plugin could accidentally introduce these into the file, and when editing the file, the editor could produce confusing visual artifacts.

More info: https://bugs.eclipse.org/bugs/show_bug.cgi?id=375421

To suppress this error, use the issue id "MangledCRLF" as explained in the Suppressing Warnings and Errors section.

MissingRegistered

Disabled By: Default
Priority: 8 / 10
Category: Correctness

Severity: Error

Explanation: Missing registered class.

If a class is referenced in the manifest or in a layout file, it must also exist in the project (or in one of the libraries included by the project. This check helps uncover typos in registration names, or attempts to rename or move classes without updating the manifest file properly.

More info: http://developer.android.com/guide/topics/manifest/manifest-intro.html

To suppress this error, use the issue id "MissingRegistered" as explained in the Suppressing Warnings and Errors section.

NegativeMargin

Disabled By: Default
Priority: 4 / 10
Category: Usability

Severity: Warning

Explanation: Negative Margins.

Margin values should be positive. Negative values are generally a sign that you are making assumptions about views surrounding the current one, or may be tempted to turn off child clipping to allow a view to escape its parent. Turning off child clipping to do this not only leads to poor graphical performance, it also results in wrong touch event handling since touch events are based strictly on a chain of parent-rect hit tests. Finally, making assumptions about the size of strings can lead to localization problems.

More info:

To suppress this error, use the issue id "NegativeMargin" as explained in the Suppressing Warnings and Errors section.

NewerVersionAvailable

Note: This issue has an associated quickfix operation in Android Studio/IntelliJ
Disabled By: Default
Priority: 4 / 10
Category: Correctness

Severity: Warning

Explanation: Newer Library Versions Available.

This detector checks with a central repository to see if there are newer versions available for the dependencies used by this project. This is similar to the GradleDependency check, which checks for newer versions available in the Android SDK tools and libraries, but this works with any MavenCentral dependency, and connects to the library every time, which makes it more flexible but also much slower.

More info:

To suppress this error, use the issue id "NewerVersionAvailable" as explained in the Suppressing Warnings and Errors section.

SelectableText

Note: This issue has an associated quickfix operation in Android Studio/IntelliJ
Disabled By: Default
Priority: 7 / 10
Category: Usability

Severity: Warning

Explanation: Dynamic text should probably be selectable.

If a <TextView> is used to display data, the user might want to copy that data and paste it elsewhere. To allow this, the <TextView> should specify android:textIsSelectable="true".

This lint check looks for TextViews which are likely to be displaying data: views whose text is set dynamically. This value will be ignored on platforms older than API 11, so it is okay to set it regardless of your minSdkVersion.

More info:

To suppress this error, use the issue id "SelectableText" as explained in the Suppressing Warnings and Errors section.

StopShip

Note: This issue has an associated quickfix operation in Android Studio/IntelliJ
Disabled By: Default
Priority: 10 / 10
Category: Correctness

Severity: Warning

Explanation: Code contains STOPSHIP marker.

Using the comment // STOPSHIP can be used to flag code that is incomplete but checked in. This comment marker can be used to indicate that the code should not be shipped until the issue is addressed, and lint will look for these.

More info:

To suppress this error, use the issue id "StopShip" as explained in the Suppressing Warnings and Errors section.

TypographyQuotes

Note: This issue has an associated quickfix operation in Android Studio/IntelliJ & Eclipse/ADT
Disabled By: Default
Priority: 5 / 10
Category: Usability:Typography

Severity: Warning

Explanation: Straight quotes can be replaced with curvy quotes.

Straight single quotes and double quotes, when used as a pair, can be replaced by "curvy quotes" (or directional quotes). This can make the text more readable.

Note that you should never use grave accents and apostrophes to quote, `like this'.

(Also note that you should not use curvy quotes for code fragments.)

More info: http://en.wikipedia.org/wiki/Quotation_mark

To suppress this error, use the issue id "TypographyQuotes" as explained in the Suppressing Warnings and Errors section.

UnusedIds

Disabled By: Default
Priority: 1 / 10
Category: Performance

Severity: Warning

Explanation: Unused id.

This resource id definition appears not to be needed since it is not referenced from anywhere. Having id definitions, even if unused, is not necessarily a bad idea since they make working on layouts and menus easier, so there is not a strong reason to delete these.

More info:

To suppress this error, use the issue id "UnusedIds" as explained in the Suppressing Warnings and Errors section.

	Correctness
61	Assert: Assertions
1	NewApi: Calling new methods on older versions
	Usability:Icons
1	IconLocation: Image defined in density-independent drawable folder
	Disabled Checks (17)