Re: Synopsis Question

Luke Petre wrote:
> I should note that these are changes I made to get things working under 
> MSVC.  Other than adding the array information to Decoder.cc, I wasn't 
> trying to add functionality.

I see. How do you compile synopsis with MSVC ? Did you set up project
files ? Also, how do you deal with symbol exports ? I have been meaning
to add the machinery to support declspec(dllexport) as well as the new
gcc's visibility attributes, but none of this has happened yet. I'll
set up an enhancement request for that right now.


>>>   + Named the anonymous namespace in ErrorHandler.cc
>>
>>
>>
>> Can you explain that change ?
> 
> 
> 
> MSVC would error that the symbol 'callback' was not defined in the 
> global scope.  So I named the anonymous namespace _ErrorHandler in order 
> to get access to that global variable.

Oh ? Does MSVC not support anonymous namespaces ?

> 
>>
>>>   + Duplicated function params in swalker.hh
>>
>>
>>
>> see below.
>>
>>>   + type.cc, no such thing as Type::Type.  changed to Types::Type
>>
>>
>>
>> Funny that this error didn't get caught by the compiler !
> 
> 
> see above.  the change in swalker.hh should really have been caught by 
> the compiler too.

Right. I didn't even see what was wrong when first reviewing your patch !

>> In that light I would particularly welcome any help on the *new*
>> code to bring it to the same level as the old translator so we
>> don't get any regressions from the switch and only then add enhancements.
> 
> 
> Where is the code for the new translator?  I'm sorry to say that I 
> haven't been keeping up with the work you've been doing, but I'll be 
> happy to try the new stuff out when I find some time.

It's all (still) in the sandbox/ area:

Cxx/ is the new AST Translator (which will be exposed as 'Parsers.Cxx.Parser')
cross/ is the new cross referencer (replacing 'Parsers.Cxx.link')
bpl/ is a set of python bindings for the PTree and SymbolTable APIs, using boost.python
wave/ is the new Cpp preprocessor (which will be exposed as 'Parsers.Cpp.Parser')

Regards,
		Stefan