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

Sean Reilly <[email protected]>
Newsgroups gmane.comp.lib.boost.ublas
Message-ID <CAJAKk0VyQ595-n6+D4ymmFonZEkXOpHZDSLaAqna=S8wF+280Q@mail.gmail.com>
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-mailBy
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
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.