[jira] [Created] (MATH-810) Adding accessors to the "Pair" class (package "o.a.c.m.util")

classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Created] (MATH-810) Adding accessors to the "Pair" class (package "o.a.c.m.util")

Richard (Jira)
Gilles created MATH-810:
---------------------------

             Summary: Adding accessors to the "Pair" class (package "o.a.c.m.util")
                 Key: MATH-810
                 URL: https://issues.apache.org/jira/browse/MATH-810
             Project: Commons Math
          Issue Type: Wish
            Reporter: Gilles
            Priority: Trivial
             Fix For: 3.1


I propose to add two accessors:
{code}
public K getFirst() {
  return getKey();
}
{code}
and
{code}
public V getSecond() {
  return getValue();
}
{code}



--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

       
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (MATH-810) Adding accessors to the "Pair" class (package "o.a.c.m.util")

Richard (Jira)

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

Luc Maisonobe commented on MATH-810:
------------------------------------

Big +1.
               

> Adding accessors to the "Pair" class (package "o.a.c.m.util")
> -------------------------------------------------------------
>
>                 Key: MATH-810
>                 URL: https://issues.apache.org/jira/browse/MATH-810
>             Project: Commons Math
>          Issue Type: Wish
>            Reporter: Gilles
>            Priority: Trivial
>              Labels: api-change
>             Fix For: 3.1
>
>
> I propose to add two accessors:
> {code}
> public K getFirst() {
>   return getKey();
> }
> {code}
> and
> {code}
> public V getSecond() {
>   return getValue();
> }
> {code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

       
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (MATH-810) Adding accessors to the "Pair" class (package "o.a.c.m.util")

Richard (Jira)
In reply to this post by Richard (Jira)

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

S├ębastien Brisard commented on MATH-810:
----------------------------------------

+1
               

> Adding accessors to the "Pair" class (package "o.a.c.m.util")
> -------------------------------------------------------------
>
>                 Key: MATH-810
>                 URL: https://issues.apache.org/jira/browse/MATH-810
>             Project: Commons Math
>          Issue Type: Wish
>            Reporter: Gilles
>            Priority: Trivial
>              Labels: api-change
>             Fix For: 3.1
>
>
> I propose to add two accessors:
> {code}
> public K getFirst() {
>   return getKey();
> }
> {code}
> and
> {code}
> public V getSecond() {
>   return getValue();
> }
> {code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira


Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (MATH-810) Adding accessors to the "Pair" class (package "o.a.c.m.util")

Richard (Jira)
In reply to this post by Richard (Jira)

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

Gilles commented on MATH-810:
-----------------------------

I wonder: Is it better to redirect the new methods to the old ones, or to return the instance variable directly?

               

> Adding accessors to the "Pair" class (package "o.a.c.m.util")
> -------------------------------------------------------------
>
>                 Key: MATH-810
>                 URL: https://issues.apache.org/jira/browse/MATH-810
>             Project: Commons Math
>          Issue Type: Wish
>            Reporter: Gilles
>            Priority: Trivial
>              Labels: api-change
>             Fix For: 3.1
>
>
> I propose to add two accessors:
> {code}
> public K getFirst() {
>   return getKey();
> }
> {code}
> and
> {code}
> public V getSecond() {
>   return getValue();
> }
> {code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

       
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (MATH-810) Adding accessors to the "Pair" class (package "o.a.c.m.util")

Richard (Jira)
In reply to this post by Richard (Jira)

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

Gilles commented on MATH-810:
-----------------------------

Accessors added in revision 1357004.

               

> Adding accessors to the "Pair" class (package "o.a.c.m.util")
> -------------------------------------------------------------
>
>                 Key: MATH-810
>                 URL: https://issues.apache.org/jira/browse/MATH-810
>             Project: Commons Math
>          Issue Type: Wish
>            Reporter: Gilles
>            Priority: Trivial
>              Labels: api-change
>             Fix For: 3.1
>
>
> I propose to add two accessors:
> {code}
> public K getFirst() {
>   return getKey();
> }
> {code}
> and
> {code}
> public V getSecond() {
>   return getValue();
> }
> {code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

       
Reply | Threaded
Open this post in threaded view
|

[jira] [Resolved] (MATH-810) Adding accessors to the "Pair" class (package "o.a.c.m.util")

Richard (Jira)
In reply to this post by Richard (Jira)

     [ https://issues.apache.org/jira/browse/MATH-810?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gilles resolved MATH-810.
-------------------------

    Resolution: Fixed
   

> Adding accessors to the "Pair" class (package "o.a.c.m.util")
> -------------------------------------------------------------
>
>                 Key: MATH-810
>                 URL: https://issues.apache.org/jira/browse/MATH-810
>             Project: Commons Math
>          Issue Type: Wish
>            Reporter: Gilles
>            Priority: Trivial
>              Labels: api-change
>             Fix For: 3.1
>
>
> I propose to add two accessors:
> {code}
> public K getFirst() {
>   return getKey();
> }
> {code}
> and
> {code}
> public V getSecond() {
>   return getValue();
> }
> {code}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira