Re: [boost] Review Request: Dataflow.Signals

From: Stjepan Rajko (stipe_at_[hidden])
Date: 2008-03-12 15:39:54

On Wed, Mar 12, 2008 at 6:58 AM, Christian Henning <chhenning_at_[hidden]> wrote:
> Alright, I can compile the code using MSVC 8. Cool now I can start
>  toying with it!
>

Great!  What made it change its mind?

>
>  > I seem to avoid the error but run into an ICE in the storage component
>  > (timed_generator inherits storage).  If I throw out timed_storage and
>  > just feed proc with values manually, I get errors regarding traits_of.
>  >  That I might be able to deal with in time - I would conjecture that
>  > for 7.1, the basic Dataflow.Signals layer might be doable but the
>  > generic components might be really tricky to get to work.
>  >
>
>  At the risk of talking out of my ... . A library to review would at
>  the very least need to compile with MSVC 7.1. I deal with the same
>  problem right now. Code works for MSVC 8 but trying to make it compile
>  with MSVC 7.1 is a pain. Also, my code looks good in a sense that I
>  wouldn't like it to mess it up with preprocessor stuff or
>  boost/config. Do you think you can rewrite the layer again to have a
>  work with 7.1?
>

I'm certainly willing to add as much 7.1 support as I can if that
makes it easier for you to use the library where you need it. I don't
think 7.1 support would be a requirement for review though - the
website states a submitted library should compile on at least two
compilers, without specific compilers listed.  Of course, the more
supported compilers the better :-)

Now that I have a copy of 7.1 installed, I can start chipping away at
it, but it might take a couple of weeks before I have some serious
time to devote to it.  I need to tighten up the tests anyway, and
doing that hand in hand with adding support for a new compiler will
make writing good tests easier.

Anyway, I'm glad you can at least play with the lib on 8.0 for now -
if you have any feedback or requests please let me know (here or
off-list), I'm most interested to hear how the library suits (or
doesn't suit) your needs!

Cheers,

Stjepan