Skip to content

Commit

Permalink
Merge pull request #27 from jamezp/issue26
Browse files Browse the repository at this point in the history
[26] Remove the debug assertion as it could break on valid signatures.
  • Loading branch information
scottmarlow committed Feb 21, 2024
2 parents 5a8047d + fe19482 commit 424108d
Showing 1 changed file with 0 additions and 16 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -234,22 +234,6 @@ private MemberDescription processMember(MemberDescription mr, boolean dontClone)
if (!MemberDescription.EMPTY_THROW_LIST.equals(ths))
cloned_m.setThrowables(convert(ths, globalParameters, localParameters));


// just for debugging
if (SigTest.debug) {
String s = cloned_m.toString();
/*
*annotation value can be parametrized class
*but we should NOT (I'm not 100% sure) erasure this.
*so just skip annotation for this checkings
*/
int fl = s.indexOf('\n');
if (fl > 0) {
s = s.substring(0, fl);
}
assert s.indexOf('<') == -1 && s.indexOf('?') == -1 && s.indexOf('{') == -1 : s;
}

return cloned_m;
}

Expand Down

0 comments on commit 424108d

Please sign in to comment.