gtoison commented on PR #254: URL: https://github.com/apache/commons-bcel/pull/254#issuecomment-1874995398
Hello @garydgregory, I have added a test to improve the coverage but I'm not sure what to do with `StringRepresentation.visitNestHost()`. I saw that the verifier failed because it was not implemented, so I've added it. But there's also this unit test checking that it is NOT implemented: https://github.com/apache/commons-bcel/blame/20cf556aad679dc4fc30a1cc027b138070fe8812/src/test/java/org/apache/bcel/verifier/VerifierTestCase.java#L100 Is it not implemented on purpose, maybe for compatibility with various JDK versions? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
