[modularization] Removing iterator->conversion dependency

Vicente J. Botet Escriba

5 Jun 2014 5 Jun '14

11:39 p.m.

Hi, in order to remove the dependency from iterator to conversion I suggest to move |boost/implicit_cast.hpp to MPL. | |Vicente | /conversion/ http://www.pdimov.com/tmp/report-6d1f271/conversion.html || * from || |//boost/implicit_cast.hpp| |template <typename T> inline T implicit_cast (typename mpl::identity<T>::type x) { return x; } |

Show replies by date

Sebastian Redl

6 Jun 6 Jun

8:08 a.m.

New subject: [modularization] Removing iterator->conversion dependency

On 06 Jun 2014, at 1:39, Vicente J. Botet Escriba wrote:

...

Hi,

in order to remove the dependency from iterator to conversion I suggest to move |boost/implicit_cast.hpp to MPL.

That makes no semantic sense at all. Breaking dependencies is nice, but not at the cost of moving something to a place it doesn’t belong at all. Sebastian

Andrey Semashev

8:14 a.m.

New subject: [modularization] Removing iterator->conversion dependency

On Friday 06 June 2014 10:08:59 Sebastian Redl wrote:

...

On 06 Jun 2014, at 1:39, Vicente J. Botet Escriba wrote:

...
Hi,

in order to remove the dependency from iterator to conversion I suggest to move |boost/implicit_cast.hpp to MPL. That makes no semantic sense at all. Breaking dependencies is nice, but not at the cost of moving something to a place it doesn’t belong at all.

Daniel James

8:15 a.m.

New subject: [modularization] Removing iterator->conversion dependency

On 6 June 2014 09:08, Sebastian Redl wrote:

...

On 06 Jun 2014, at 1:39, Vicente J. Botet Escriba wrote:

...
in order to remove the dependency from iterator to conversion I suggest to move |boost/implicit_cast.hpp to MPL.

That makes no semantic sense at all. Breaking dependencies is nice, but not at the cost of moving something to a place it doesn’t belong at all.

I think there's a better solution. Splitting lexical cast into its own module might make conversion a much lighter dependency. I'm wary of creating too many submodules, but I think that one might be justified.

Antony Polukhin

8:19 a.m.

New subject: [modularization] Removing iterator->conversion dependency

2014-06-06 12:15 GMT+04:00 Daniel James :

...

On 6 June 2014 09:08, Sebastian Redl wrote:

...
On 06 Jun 2014, at 1:39, Vicente J. Botet Escriba <

vicente.botet@wanadoo.fr> wrote:

...
...
in order to remove the dependency from iterator to conversion I suggest

to move |boost/implicit_cast.hpp to MPL.

That makes no semantic sense at all. Breaking dependencies is nice, but not at the cost of moving something to a place it doesn’t belong at all.

I think there's a better solution. Splitting lexical cast into its own module might make conversion a much lighter dependency. I'm wary of creating too many submodules, but I think that one might be justified.

+1 See my previous post about moving part of MPL into the Boost.Core (identity, bool_, integral_constant_, or_, and_). This will remove the dependency of implicit_cast from MPL -- Best regards, Antony Polukhin

Peter Dimov

7 Jun 7 Jun

2:04 p.m.

New subject: [modularization] Removing iterator->conversiondependency

Daniel James wrote:

...

I think there's a better solution. Splitting lexical cast into its own module might make conversion a much lighter dependency. I'm wary of creating too many submodules, but I think that one might be justified.

lexical_cast already has its own entries in libraries.htm and maintainers.txt, so I think that it makes perfect sense for it to have its own module.

Vicente J. Botet Escriba

9 Jun 9 Jun

8 a.m.

New subject: [modularization] Removing iterator->conversiondependency

Le 07/06/14 16:04, Peter Dimov a écrit :

...

Daniel James wrote:

...
I think there's a better solution. Splitting lexical cast into its own module might make conversion a much lighter dependency. I'm wary of creating too many submodules, but I think that one might be justified.

lexical_cast already has its own entries in libraries.htm and maintainers.txt, so I think that it makes perfect sense for it to have its own module.

Please could we decide on this point? Best, Vicente

Daniel James

10:08 a.m.

New subject: [modularization] Removing iterator->conversiondependency

On 9 June 2014 09:00, Vicente J. Botet Escriba wrote:

...

Le 07/06/14 16:04, Peter Dimov a écrit :

...
Daniel James wrote:

...
I think there's a better solution. Splitting lexical cast into its own module might make conversion a much lighter dependency. I'm wary of creating too many submodules, but I think that one might be justified.

lexical_cast already has its own entries in libraries.htm and maintainers.txt, so I think that it makes perfect sense for it to have its own module.

Please could we decide on this point?

Someone has already created a lexical_cast module in develop, it's not in master yet though.

Peter Dimov

10:45 a.m.

New subject: [modularization] Removing iterator->conversiondependency

Vicente J. Botet Escriba wrote:

...

Please could we decide on this point?

Already decided. lexical_cast is now its own module in develop.

Vicente J. Botet Escriba

11:47 a.m.

New subject: [modularization] Removing iterator->conversiondependency

Le 09/06/14 12:45, Peter Dimov a écrit :

...

Vicente J. Botet Escriba wrote:

...
Please could we decide on this point?

Already decided. lexical_cast is now its own module in develop.

Thanks, Vicente

Vicente J. Botet Escriba

6 Jun 6 Jun

9:40 p.m.

New subject: [modularization] Removing iterator->conversion dependency

Le 06/06/14 10:08, Sebastian Redl a écrit :

...

On 06 Jun 2014, at 1:39, Vicente J. Botet Escriba wrote:

...
Hi,

in order to remove the dependency from iterator to conversion I suggest to move |boost/implicit_cast.hpp to MPL. That makes no semantic sense at all. Breaking dependencies is nice, but not at the cost of moving something to a place it doesn’t belong at all.

Why this function could not be in MPL? Vicente

Andrey Semashev

9:47 p.m.

New subject: [modularization] Removing iterator->conversion dependency

On Friday 06 June 2014 23:40:27 Vicente J. Botet Escriba wrote:

...

Le 06/06/14 10:08, Sebastian Redl a écrit :

...
On 06 Jun 2014, at 1:39, Vicente J. Botet Escriba wrote:

...
Hi,

in order to remove the dependency from iterator to conversion I suggest to move |boost/implicit_cast.hpp to MPL.> That makes no semantic sense at all. Breaking dependencies is nice, but not at the cost of moving something to a place it doesn’t belong at all. Why this function could not be in MPL?

Because it has no relation to metaprogramming?

Vicente J. Botet Escriba

10:21 p.m.

New subject: [modularization] Removing iterator->conversion dependency

Le 06/06/14 23:47, Andrey Semashev a écrit :

...

On Friday 06 June 2014 23:40:27 Vicente J. Botet Escriba wrote:

...
Le 06/06/14 10:08, Sebastian Redl a écrit :

...
On 06 Jun 2014, at 1:39, Vicente J. Botet Escriba wrote:

...
Hi,

in order to remove the dependency from iterator to conversion I suggest to move |boost/implicit_cast.hpp to MPL.>

That makes no semantic sense at all. Breaking dependencies is nice, but not at the cost of moving something to a place it doesn’t belong at all. Why this function could not be in MPL? Because it has no relation to metaprogramming?

Ok I see :( Vicente

Andrey Semashev

8:18 a.m.

New subject: [modularization] Removing iterator->conversion dependency

On Friday 06 June 2014 01:39:24 Vicente J. Botet Escriba wrote:

...

Hi,

in order to remove the dependency from iterator to conversion I suggest to move |boost/implicit_cast.hpp to MPL.

|Vicente

/conversion/ http://www.pdimov.com/tmp/report-6d1f271/conversion.html

||

* from ||

|//boost/implicit_cast.hpp| | |template <typename T>

inline T implicit_cast (typename mpl::identity<T>::type x) { return x; }

I would suggest moving it to Core and modifying to not depend on MPL (i.e. reimplementing mpl::identity).

3825

Age (days ago)

3829

Last active (days ago)

List overview

Download

13 comments

6 participants

participants (6)

Andrey Semashev
Antony Polukhin
Daniel James
Peter Dimov
Sebastian Redl
Vicente J. Botet Escriba