On Wed, Mar 12, 2008 at 6:58 AM, Christian Henning <chhenning@gmail.com> 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