Add descriptive comment

qgis · Dec 27, 2019 · 1b5bc76 · 1b5bc76
1 parent 57b41e4
commit 1b5bc76
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/src/core/pal/costcalculator.cpp b/src/core/pal/costcalculator.cpp
@@ -188,6 +188,9 @@ void CostCalculator::finalizeCandidatesCosts( Feats *feat, PalRtree<FeaturePart>
   // "try to exclude all conflitual labels (good ones have cost < 1 by pruning)"
   // my interpretation: it appears this scans through the candidates and chooses some threshold
   // based on the candidate cost, after which all remaining candidates are simply discarded
+  // my suspicion: I don't think this is needed (or wanted) here, and is reflective of the fact
+  // that obstacle costs are too low vs inferior candidate placement costs (i.e. without this,
+  // an "around point" label would rather be placed over an obstacle then be placed in a position > 6 o'clock
   double discrim = 0.0;
   std::size_t stop = 0;
   do