Use 10NM as the major/minor light threshold

src/GUI/mapview.cpp

@@ -1254,7 +1254,7 @@ void MapView::drawHillShading(bool draw)
 	setMap(_map);
 }
 
-void MapView::selectLayers(Layers layers)
+void MapView::selectLayers(MapView::Layers layers)
 {
 	_layers = layers;
 

src/GUI/mapview.h

@@ -136,7 +136,7 @@ public slots:
 	void showMotionInfo(bool show);
 	void useStyles(bool use);
 	void drawHillShading(bool draw);
-	void selectLayers(Layers layers);
+	void selectLayers(MapView::Layers layers);
 
 private slots:
 	void updatePOI();

src/map/ENC/rastertile.cpp

@@ -14,6 +14,7 @@ using namespace ENC;
 #define TEXT_EXTENT 160
 #define TSSLPT_SIZE 24
 #define RANGE_FACTOR 4
+#define MAJOR_RANGE 10
 
 static const float C1 = 0.866025f; /* sqrt(3)/2 */
 static const QColor tsslptPen = QColor(0xeb, 0x49, 0xeb);
@@ -277,7 +278,7 @@ void RasterTile::processPoints(QList<MapData::Point> &points,
 			double range = attr.value(VALNMR).toDouble();
 
 			if (attr.contains(SECTR1)
-			  || (range > 6 && !(point.type() & 0xFFFF))) {
+			  || (range >= MAJOR_RANGE && !(point.type() & 0xFFFF))) {
 				sectorLights.append(SectorLight(point.pos(), color,
 				  attr.value(LITVIS).toUInt(), range,
 				  attr.value(SECTR1).toDouble(), attr.value(SECTR2).toDouble()));

src/map/IMG/rastertile.cpp

@@ -19,13 +19,38 @@ using namespace IMG;
 #define TEXT_EXTENT 160
 #define ICON_PADDING 2
 #define RANGE_FACTOR 4
-#define RANGE_MIN    6
+#define MAJOR_RANGE  10
 #define ROAD  0
 #define WATER 1
 
 #define AREA(rect) \
 	(rect.size().width() * rect.size().height())
 
+struct Sector
+{
+	Sector(Light::Color color, quint32 start, quint32 end)
+	  : color(color), start(start), end(end) {}
+
+	bool operator==(const Sector &other) const
+	{
+		return (color == other.color && start == other.start && end == other.end);
+	}
+	bool operator<(const Sector &other) const
+	{
+		if (color == other.color) {
+			if (start == other.start)
+				return end < other.end;
+			else
+				return start < other.start;
+		} else
+			return color < other.color;
+	}
+
+	Light::Color color;
+	quint32 start;
+	quint32 end;
+};
+
 static const QColor textColor(Qt::black);
 static const QColor haloColor(Qt::white);
 static const QColor shieldColor(Qt::white);
@@ -239,6 +264,7 @@ void RasterTile::drawSectorLights(QPainter *painter,
 	for (int i = 0; i < lights.size(); i++) {
 		const MapData::Point *p = lights.at(i);
 		QPoint pos(p->coordinates.lon(), p->coordinates.lat());
+		QMap<Sector, quint32> rangeMap;
 
 		for (int j = 0; j < p->lights.size(); j++) {
 			const Light &l = p->lights.at(j);
@@ -248,8 +274,16 @@ void RasterTile::drawSectorLights(QPainter *painter,
 					const Light::Sector &start = l.sectors().at(k);
 					const Light::Sector &end = (k == l.sectors().size() - 1)
 					  ? l.sectors().at(0) : l.sectors().at(k+1);
+					quint32 angle = end.angle() - start.angle();
+
+					if (start.color() && (angle || start.range() >= MAJOR_RANGE)) {
+						quint32 range = start.range() ? start.range() : 6;
+						Sector s(start.color(), start.angle(), end.angle());
+						if (rangeMap.value(s) >= range)
+							continue;
+						else
+							rangeMap.insert(s, range);
 
-					if (start.color()) {
 						double a1 = -(end.angle() / 10.0 + 90.0);
 						double a2 = -(start.angle() / 10.0 + 90.0);
 						if (a1 > a2)
@@ -258,9 +292,7 @@ void RasterTile::drawSectorLights(QPainter *painter,
 						if (as == 0)
 							as = 360;
 
-						QRect rect(lightRect(pos, start.range()
-						  ? start.range() : RANGE_MIN));
-
+						QRect rect(lightRect(pos, range));
 						painter->setPen(QPen(Qt::black, 6, Qt::SolidLine,
 						  Qt::FlatCap));
 						painter->drawArc(rect, a1 * 16, as * 16);
@@ -268,7 +300,7 @@ void RasterTile::drawSectorLights(QPainter *painter,
 						  Qt::SolidLine, Qt::FlatCap));
 						painter->drawArc(rect, a1 * 16, as * 16);
 
-						if (a2 - a1 != 0) {
+						if (angle) {
 							QLineF ln(pos, QPointF(pos.x() + rect.width(),
 							  pos.y()));
 							ln.setAngle(a1);
@@ -279,9 +311,14 @@ void RasterTile::drawSectorLights(QPainter *painter,
 						}
 					}
 				}
-			} else if (l.range() > RANGE_MIN) {
-				QRect rect(lightRect(pos, l.range()));
+			} else if (l.color() && l.range() >= MAJOR_RANGE) {
+				Sector s(l.color(), 0, 3600);
+				if (rangeMap.value(s) >= l.range())
+					continue;
+				else
+					rangeMap.insert(s, l.range());
 
+				QRect rect(lightRect(pos, l.range()));
 				painter->setPen(QPen(Qt::black, 6, Qt::SolidLine, Qt::FlatCap));
 				painter->drawArc(rect, 0, 360 * 16);
 				painter->setPen(QPen(Style::color(l.color()), 4, Qt::SolidLine,
@@ -463,19 +500,47 @@ void RasterTile::processShields(const QList<MapData::Poly> &lines,
 	}
 }
 
-static bool showAsSector(const QVector<Light> &lights)
+static bool sectorLight(const QVector<Light> &lights)
 {
 	for (int i = 0; i < lights.size(); i++) {
 		const Light &l = lights.at(i);
-		if ((l.color() && l.range() > RANGE_MIN) || !l.sectors().isEmpty())
+		if (l.color() && l.range() >= MAJOR_RANGE)
 			return true;
+		for (int j = 0; j < l.sectors().size(); j++) {
+			const Light::Sector &start = l.sectors().at(j);
+			const Light::Sector &end = (j == l.sectors().size() - 1)
+			  ? l.sectors().at(0) : l.sectors().at(j+1);
+			quint32 angle = end.angle() - start.angle();
+			if (start.color() && (angle || start.range() >= MAJOR_RANGE))
+				return true;
+		}
 	}
 
 	return false;
 }
 
+static Light::Color ordinaryLight(const QVector<Light> &lights)
+{
+	for (int i = 0; i < lights.size(); i++) {
+		const Light &l = lights.at(i);
+		if (l.color() && l.range() < MAJOR_RANGE)
+			return l.color();
+		for (int j = 0; j < l.sectors().size(); j++) {
+			const Light::Sector &start = l.sectors().at(j);
+			const Light::Sector &end = (j == l.sectors().size() - 1)
+			  ? l.sectors().at(0) : l.sectors().at(j+1);
+			quint32 angle = end.angle() - start.angle();
+			if (start.color() && !angle && start.range() < MAJOR_RANGE)
+				return start.color();
+		}
+	}
+
+	return Light::None;
+}
+
 void RasterTile::processPoints(QList<MapData::Point> &points,
-  QList<TextItem*> &textItems, QList<const MapData::Point*> &lights)
+  QList<TextItem*> &textItems, QList<TextItem *> &lights,
+  QList<const MapData::Point*> &sectorLights)
 {
 	std::sort(points.begin(), points.end());
 
@@ -484,10 +549,10 @@ void RasterTile::processPoints(QList<MapData::Point> &points,
 		const Style *style = _data->style();
 		const Style::Point &ps = style->point(point.type);
 		bool poi = Style::isPOI(point.type);
-		bool sl = showAsSector(point.lights);
+		bool sl = sectorLight(point.lights);
 
 		if (sl)
-			lights.append(&point);
+			sectorLights.append(&point);
 
 		const QString *label = point.label.text().isEmpty()
 		  ? 0 : &(point.label.text());
@@ -511,17 +576,15 @@ void RasterTile::processPoints(QList<MapData::Point> &points,
 		  color, hcolor, 0, ICON_PADDING);
 		if (item->isValid() && (sl || !item->collides(textItems))) {
 			textItems.append(item);
-			for (int j = 0; j < point.lights.size(); j++) {
-				const Light &l = point.lights.at(j);
-				if (l.color() && l.range() <= RANGE_MIN) {
-					textItems.append(new TextPointItem(pos + style->lightOffset(),
-					  0, 0, style->light(l.color()), 0, 0, 0, 0));
-					break;
-				}
-			}
+			Light::Color color = ordinaryLight(point.lights);
+			if (color)
+				lights.append(new TextPointItem(pos + style->lightOffset(),
+				  0, 0, style->light(color), 0, 0, 0, 0));
 		} else
 			delete item;
 	}
+
+
 }
 
 void RasterTile::fetchData(QList<MapData::Poly> &polygons,
@@ -606,8 +669,8 @@ void RasterTile::render()
 	QList<MapData::Poly> polygons;
 	QList<MapData::Poly> lines;
 	QList<MapData::Point> points;
-	QList<TextItem*> textItems;
-	QList<const MapData::Point*> lights;
+	QList<TextItem*> textItems, lights;
+	QList<const MapData::Point*> sectorLights;
 	QImage arrows[2];
 
 	arrows[ROAD] = Util::svg2img(":/symbols/oneway.svg", _ratio);
@@ -618,7 +681,7 @@ void RasterTile::render()
 	ll2xy(lines);
 	ll2xy(points);
 
-	processPoints(points, textItems, lights);
+	processPoints(points, textItems, lights, sectorLights);
 	processPolygons(polygons, textItems);
 	processLines(lines, textItems, arrows);
 
@@ -633,9 +696,11 @@ void RasterTile::render()
 	drawPolygons(&painter, polygons);
 	drawHillShading(&painter);
 	drawLines(&painter, lines);
-	drawSectorLights(&painter, lights);
+	drawTextItems(&painter, lights);
+	drawSectorLights(&painter, sectorLights);
 	drawTextItems(&painter, textItems);
 
+	qDeleteAll(lights);
 	qDeleteAll(textItems);
 
 	//painter.setPen(Qt::red);

src/map/IMG/rastertile.h

@@ -60,18 +60,22 @@ private:
 	void ll2xy(QList<MapData::Poly> &polys) const;
 	void ll2xy(QList<MapData::Point> &points) const;
 
-	void drawPolygons(QPainter *painter, const QList<MapData::Poly> &polygons) const;
+	void drawPolygons(QPainter *painter,
+	  const QList<MapData::Poly> &polygons) const;
 	void drawLines(QPainter *painter, const QList<MapData::Poly> &lines) const;
-	void drawTextItems(QPainter *painter, const QList<TextItem*> &textItems) const;
+	void drawTextItems(QPainter *painter,
+	  const QList<TextItem*> &textItems) const;
 	void drawHillShading(QPainter *painter) const;
-	void drawSectorLights(QPainter *painter, const QList<const MapData::Point*> &lights) const;
+	void drawSectorLights(QPainter *painter,
+	  const QList<const MapData::Point*> &lights) const;
 
 	void processPolygons(const QList<MapData::Poly> &polygons,
 	  QList<TextItem *> &textItems);
 	void processLines(QList<MapData::Poly> &lines, QList<TextItem*> &textItems,
 	  const QImage (&arrows)[2]);
-	void processPoints(QList<MapData::Point> &points, QList<TextItem*> &textItems,
-	  QList<const MapData::Point *> &lights);
+	void processPoints(QList<MapData::Point> &points,
+	  QList<TextItem*> &textItems, QList<TextItem*> &lights,
+	  QList<const MapData::Point*> &sectorLights);
 	void processShields(const QList<MapData::Poly> &lines,
 	  QList<TextItem*> &textItems);
 	void processStreetNames(const QList<MapData::Poly> &lines,