Re: Should/can I use GitHub to fix issue #7902 Division by scalar should use enable_if<>

Nasos Iliopoulos <[email protected]>
Newsgroups gmane.comp.lib.boost.ublas
Message-ID <[email protected]>
Sean,
please fork from:
https://github.com/uBLAS/ublas/tree/develop

and not from the the boostorg repository.

General guidelines:
https://github.com/uBLAS/ublas/wiki


-Nasos

On 11/20/2014 02:42 PM, David Bellot wrote:
> Hi Sean,
>
> yes, for any bug fix, please use Github. For the project in your own 
> account and checkout branch develop.
> When you submit your pull request, it must be for branch develop.
>
> We're not using the old repository anymore.
>
> Thanks for contributing,
> David
>
>
> On Thu, Nov 20, 2014 at 5:35 PM, Sean Reilly <[email protected] 
> <mailto:[email protected]>> wrote:
>
>     Background:
>
>     I was looking at Boost 1.57 and noticed that my old 7902 bug was
>     still in there.
>
>     #7902: Bugs: Division by scalar should use enable_if<>
>     (matrix_expression) (new)
>     <https://svn.boost.org/trac/boost/ticket/7902>
>         Bug #6511 added enable_if<> to file: vector_expression.hpp
>         function: operator/( vector, scalar). The same feature needs
>         to be add to matrix_expression's operator/( matrix, scalar).
>         See attached e-mail
>         By campreilly@… — 01/17/13 21:55:55
>
>     This error is almost exactly the same as a previous bug that has
>     already been fixed in vector_expression.hpp under issue 6511. 
>     But, I forgot to mention matrix_expression.hpp in that bug,
>     perhaps because I'm a knucklehead.
>
>     #6511: Bugs: Division by scalar should use enable_if<> (closed:
>     fixed) <https://svn.boost.org/trac/boost/ticket/6511>
>         file: vector_expression.hpp lines 1409 through 1417 function:
>         operator/( vector, scalar) Should be using the enable_if<>
>         macros just like the operator*() in lines 1397 through 1407
>         (just above it). Doing so allows further overloading ...
>         By Sean Reilly <campreilly@…> — 02/01/12 15:38:09
>
>     Question:
>
>     It is a simple one line fix, but I haven't kept up with the
>     updated development and delivery process. Would you like me to
>     fork GitHub, fix it, and issue a pull request.  Is that how we are
>     handling these kinds of changes?
>
>     Sean Reilly
>
>     _______________________________________________
>     ublas mailing list
>     [email protected] <mailto:[email protected]>
>     http://lists.boost.org/mailman/listinfo.cgi/ublas
>     Sent to: [email protected] <mailto:[email protected]>
>
>
>
>
> _______________________________________________
> ublas mailing list
> [email protected]
> http://lists.boost.org/mailman/listinfo.cgi/ublas
> Sent to: [email protected]
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.