xiaoyan159/vtm
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

LineClipper: name and unify clip states (#519)

Browse Source
This commit is contained in:
Gustl22 2018-03-23 11:04:39 +01:00 committed by Emux
parent c3b62fbdf5
commit 27a3c5a926
No known key found for this signature in database
GPG Key ID: 89C6921D7AF2BDD0
4 changed files with 22 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
vtm/src/org/oscim/layers/PathLayer.java
View File

@ -396,11 +396,11 @@ public class PathLayer extends Layer implements GestureListener {
}
int clip = mClipper.clipNext(x, y);
if (clip < 1) {
if (clip != LineClipper.INSIDE) {
if (i > 2)
ll.addLine(projected, i, false);
if (clip < 0) {
if (clip == LineClipper.INTERSECTION) {
/* add line segment */
segment = mClipper.getLine(segment, 0);
ll.addLine(segment, 4, false);
@ -412,7 +412,7 @@ public class PathLayer extends Layer implements GestureListener {
}
i = 0;
// if the end point is inside, add it
if (mClipper.getPrevOutcode() == 0) {
if (mClipper.getPrevOutcode() == LineClipper.INSIDE) {
projected[i++] = prevX;
projected[i++] = prevY;
}

2
vtm/src/org/oscim/renderer/bucket/ExtrusionBucket.java
View File

@ -556,7 +556,7 @@ public class ExtrusionBucket extends RenderBucket {
}
/* check if face is within tile */
if (mClipper.clipNext((int) nx, (int) ny) == 0) {
if (mClipper.clipNext((int) nx, (int) ny) == LineClipper.OUTSIDE) {
even = ++even % 2;
continue;
}

17
vtm/src/org/oscim/renderer/bucket/PolygonBucket.java
View File

@ -333,28 +333,27 @@ public final class PolygonBucket extends RenderBucket {
/* project bbox of polygon to screen */
v.mvp.prj2D(pb.bbox, 0, box, 0, 4);
int out = 0;
int out = LineClipper.INSIDE;
for (int i = 0; i < 8; i += 2) {
int o = mScreenClip.outcode(box[i], box[i + 1]);
if (o == 0) {
if (o == LineClipper.INSIDE) {
/* at least one corner is inside */
out = 0;
out = LineClipper.INSIDE;
break;
}
out |= o;
}
/* Check if any polygon-bucket edge intersects the screen.
* Also check the very unlikely case where the view might
* be
* completly contained within box */
if ((out != 0) && (out != 0xF)) {
* be completely contained within box */
if ((out != LineClipper.INSIDE) && (out != LineClipper.OUTSIDE)) {
mScreenClip.clipStart(box[6], box[7]);
out = 0;
for (int i = 0; i < 8 && out == 0; i += 2)
out = LineClipper.OUTSIDE;
for (int i = 0; i < 8 && out == LineClipper.OUTSIDE; i += 2)
out = mScreenClip.clipNext(box[i], box[i + 1]);
if (out == 0) {
if (out == LineClipper.OUTSIDE) {
//log.debug("out {}\n {}\n {}", out, Arrays.toString(pb.bbox), Arrays.toString(box));
// log.debug("outside {} {} {}", out,

17
vtm/src/org/oscim/utils/geom/LineClipper.java
View File

@ -1,6 +1,7 @@
/*
* Copyright 2012, 2013 Hannes Janetzek
* Copyright 2016 Bezzu
* Copyright 2018 Gustl22
*
* This file is part of the OpenScienceMap project (http://www.opensciencemap.org).
*
@ -31,6 +32,8 @@ public class LineClipper {
public static final int RIGHT = 2; // 0010
public static final int BOTTOM = 4; // 0100
public static final int TOP = 8; // 1000
public static final int OUTSIDE = 0xF; // 1111
public static final int INTERSECTION = -1;
private float xmin, xmax, ymin, ymax;
@ -90,8 +93,8 @@ public class LineClipper {
}
/**
* @return 0 if not intersection, 1 fully within, -1 clipped (and 'out' set
* to new points)
* @return OUTSIDE (= 1111) if not intersection, INSIDE (= 0) if fully within,
* INTERSECTION (= -1) if clipped (and 'out' set to new points)
*/
public int clipNext(float x1, float y1) {
int accept;
@ -109,12 +112,12 @@ public class LineClipper {
if ((mPrevOutcode | outcode) == 0) {
// Bitwise OR is 0. Trivially accept
accept = 1;
accept = INSIDE;
} else if ((mPrevOutcode & outcode) != 0) {
// Bitwise AND is not 0. Trivially reject
accept = 0;
accept = OUTSIDE;
} else {
accept = clip(mPrevX, mPrevY, x1, y1, mPrevOutcode, outcode) ? -1 : 0;
accept = clip(mPrevX, mPrevY, x1, y1, mPrevOutcode, outcode) ? INTERSECTION : OUTSIDE;
}
mPrevOutcode = outcode;
mPrevX = x1;
@ -258,10 +261,10 @@ public class LineClipper {
y = in.points[inPos++];
int clip = clipNext(x, y);
if (clip == 0) {
if (clip == OUTSIDE) {
/* current segment is fully outside */
inside = false; // needed?
} else if (clip == 1) {
} else if (clip == INSIDE) {
/* current segment is fully within */
out.addPoint(x, y);
} else { /* clip == -1 */