[jitsi-dev] [jitsi/jitsi] code clean up (#277)


#1

replaced if/else constructs with switches
replaced foreach loops
added log entries
fixed order in static fields
switched comparisons of Strings to not generate null pointers
You can view, comment on, or merge this pull request online at:

  https://github.com/jitsi/jitsi/pull/277

-- Commit Summary --

  * code cleanup
  * code cleanup

-- File Changes --

    M src/net/java/sip/communicator/impl/protocol/jabber/CallPeerJabberImpl.java (19)
    M src/net/java/sip/communicator/impl/protocol/jabber/CallPeerMediaHandlerJabberImpl.java (44)
    M src/net/java/sip/communicator/impl/protocol/jabber/IceUdpTransportManager.java (112)
    M src/net/java/sip/communicator/impl/protocol/sip/CallPeerMediaHandlerSipImpl.java (48)
    M src/net/java/sip/communicator/impl/protocol/sip/CallSipImpl.java (66)
    M src/net/java/sip/communicator/impl/protocol/sip/OperationSetBasicTelephonySipImpl.java (15)
    M src/net/java/sip/communicator/impl/protocol/sip/sdp/SdpUtils.java (11)
    M src/net/java/sip/communicator/service/protocol/media/CallPeerMediaHandler.java (19)
    M src/net/java/sip/communicator/service/protocol/media/TransportManager.java (143)

-- Patch Links --

https://github.com/jitsi/jitsi/pull/277.patch
https://github.com/jitsi/jitsi/pull/277.diff

···

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi/pull/277