aboutsummaryrefslogtreecommitdiffstats
path: root/main/src/cgeo/geocaching/cgBase.java
diff options
context:
space:
mode:
Diffstat (limited to 'main/src/cgeo/geocaching/cgBase.java')
-rw-r--r--main/src/cgeo/geocaching/cgBase.java5
1 files changed, 5 insertions, 0 deletions
diff --git a/main/src/cgeo/geocaching/cgBase.java b/main/src/cgeo/geocaching/cgBase.java
index 0615c0f..ca82bb0 100644
--- a/main/src/cgeo/geocaching/cgBase.java
+++ b/main/src/cgeo/geocaching/cgBase.java
@@ -692,9 +692,14 @@ public class cgBase {
String inventoryPre = null;
// GC* code
+ // FIXME: most of the blocks below cannot raise any exception
try {
final Matcher matcherCode = patternCode.matcher(row);
+ // FIXME: what are those while loops that can erase the previous values? This should either be fixed
+ // or documented here.
while (matcherCode.find()) {
+ // FIXME: here, and probably below, a find necessarily has a group count > 0 as the first group
+ // is not optional.
if (matcherCode.groupCount() > 0) {
// The String constructor is necessary as long as the pattern matching doesn't use the
// methods from BaseUtil. Otherwise every geocode holds the complete page in memory