You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@activemq.apache.org by "guai (JIRA)" <ji...@apache.org> on 2017/10/18 13:11:00 UTC

[jira] [Commented] (AMQ-6844) Misleading args order

    [ https://issues.apache.org/jira/browse/AMQ-6844?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16209298#comment-16209298 ] 

guai commented on AMQ-6844:
---------------------------

lazy asses
If you hit something like that in someone else's code I bet you'll think "what an idiot wrote this one, I will never do it"

> Misleading args order
> ---------------------
>
>                 Key: AMQ-6844
>                 URL: https://issues.apache.org/jira/browse/AMQ-6844
>             Project: ActiveMQ
>          Issue Type: Improvement
>          Components: JMS client
>    Affects Versions: 5.15.1
>            Reporter: guai
>
> org.apache.activemq.artemis.jms.client.ActiveMQConnectionFactory: 
> public ActiveMQConnectionFactory(String url, String user, String password) {...}
> org.apache.activemq.ActiveMQConnectionFactory:
> public ActiveMQConnectionFactory(String userName, String password, String brokerURL) {...}
> all three are strings, url first vs url last can cause problems while migrating and compiler won't even notice.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)