From 28410cfb398b510b4f748f3a1134dd13dae9f7c4 Mon Sep 17 00:00:00 2001
From: Harvey Harrison <harvey.harrison@gmail.com>
Date: Tue, 16 Oct 2012 00:12:56 -0700
Subject: gluegen: remove two fast-path tests that can never trigger
ignores and unimplemented are both Set<Pattern>, they cannot contain a String,
immediately after these 'fast paths' we loop over the patterns calling .matches()
which makes sense, so remove the misleading tests.
Signed-off-by: Harvey Harrison <harvey.harrison@gmail.com>
---
src/java/com/jogamp/gluegen/JavaConfiguration.java | 14 --------------
1 file changed, 14 deletions(-)
(limited to 'src/java/com/jogamp/gluegen/JavaConfiguration.java')
diff --git a/src/java/com/jogamp/gluegen/JavaConfiguration.java b/src/java/com/jogamp/gluegen/JavaConfiguration.java
index d1cb46a..fa77a17 100644
--- a/src/java/com/jogamp/gluegen/JavaConfiguration.java
+++ b/src/java/com/jogamp/gluegen/JavaConfiguration.java
@@ -717,14 +717,6 @@ public class JavaConfiguration {
return true;
}
}
-
- // Simple case; the entire symbol is in the ignore table.
- if (ignores.contains(symbol)) {
- if(DEBUG_IGNORES) {
- System.err.println("Ignore Impl ignores: "+symbol);
- }
- return true;
- }
// Ok, the slow case. We need to check the entire table, in case the table
// contains an regular expression that matches the symbol.
@@ -779,12 +771,6 @@ public class JavaConfiguration {
throws a run-time exception with an "unimplemented" message
during glue code generation. */
public boolean isUnimplemented(String symbol) {
-
- // Simple case; the entire symbol is in the ignore table.
- if (unimplemented.contains(symbol)) {
- return true;
- }
-
// Ok, the slow case. We need to check the entire table, in case the table
// contains an regular expression that matches the symbol.
for (Pattern regexp : unimplemented) {
--
cgit v1.2.3