qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From gemmellr <...@git.apache.org>
Subject [GitHub] qpid-proton-j pull request #18: NO-JIRA fix use of AtomicInteger and other v...
Date Mon, 15 Oct 2018 16:43:51 GMT
Github user gemmellr commented on a diff in the pull request:

    --- Diff: examples/reactor/src/main/java/org/apache/qpid/proton/example/reactor/EchoInputStreamWrapper.java
    @@ -31,11 +31,12 @@
     public class EchoInputStreamWrapper extends Thread {
    +    private static final AtomicInteger idCounter = new AtomicInteger();
    --- End diff --
    I don't think this really needed fixed and would have just left it as is. Its only an
example, likely to count to 1. Typically having a static shared between reactors would often
be incorrect.
    If it were to change the variable name would be of the wrong syntax.


To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org

View raw message