Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Duplicate variable definition #600

Closed
khaes-kth opened this issue Aug 30, 2021 · 2 comments
Closed

Duplicate variable definition #600

khaes-kth opened this issue Aug 30, 2021 · 2 comments
Labels
bug Something isn't working

Comments

@khaes-kth
Copy link
Collaborator

When rule 2142 fixes are applied on this repo (commit: 34edf588a), a variable definition is duplicated:

     HttpGet url = new HttpGet(String.format("http://localhost:%d/chatProxied/test", serverPort));
 
     try (CloseableHttpClient chc = HttpClientBuilder.create().build();
-            CloseableHttpResponse chr = chc.execute(url)) {
+            CloseableHttpResponse chr = chc.execute(url);CloseableHttpResponse chr = chc.execute(url)) {
       try (InputStream is = chr.getEntity().getContent()) {
         byte[] readData = readBlock(is);
         assertTrue("No data received (message1)", readData.length > 0);
@monperrus
Copy link
Contributor

Will be closed when INRIA/spoon#4309 is merged

@monperrus
Copy link
Contributor

INRIA/spoon#4309 merged

algomaster99 added a commit that referenced this issue Dec 10, 2021
algomaster99 added a commit that referenced this issue Dec 10, 2021
algomaster99 added a commit that referenced this issue Dec 10, 2021
algomaster99 added a commit that referenced this issue Dec 10, 2021
algomaster99 added a commit that referenced this issue Dec 10, 2021
algomaster99 added a commit that referenced this issue Dec 10, 2021
khaes-kth pushed a commit that referenced this issue Dec 10, 2021
* Upgrade spoon to 10.0.1-beta-2

* Revert "fix: disable custom security manager for tests which indirectly look up the path of mvn executable"

This reverts commit 2f7125e.

* Add test case for #600

* Add test case for #600

* Add test case for #603

* Add test case for #602
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

No branches or pull requests

2 participants