Github user gemmellr commented on the pull request:
https://github.com/apache/qpid-proton/pull/58#issuecomment-172036582
Hi Sreeram,
Do you actually mean that there was an NPE inside EventImpl as the title
suggests? Given usage of the getTransport() inside EventImpl method, it doesnt
seem like that would be the case or that the change would affect it.
If on the other hand you mean that a (your?) onSessionInit() handler calls
getTransport() on the session init event (not entirely clear why it would?) and
then NPE's when using the result then I guess I can see that. Its not
particularly clear why that would only happen transiently though unless there
is actually another issue at play. I think seeing a stacktrace would be good
before making any changes.
P.S. no need to tag me when creating each PR, the mails all go to the same
place so it wont typically make me spot it or do anything with it any quicker ;)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
