Hi! -------- Original-Nachricht -------- > Datum: Thu, 05 Jun 2008 06:14:29 +0100 > Von: Luis Sergio Oliveira <[EMAIL PROTECTED]> > An: [email protected] > Betreff: Re: [argouml-dev] C++ module preprocessor
--<snip>-- > it is very hard to review your changes since the module sources you have > sent me are pre-revision 210 - when the whole repository was recovered. ??? I ran a svn update recently and it showed no modifications in the repository, so I thought my c++ module was current. --<snip>-- > > PS: I'll proceed with the assumption that only the following files were > changed: > argouml-cpp/build.xml > argouml-cpp/src/org/argouml/language/cpp/reveng/CppImport.java > Added argouml-cpp/lib/anarres-cpp.jar That's more or less it. I tried to add the anarres jar to the cpp module, modified the cpp manifest, but Argo did not load the module afterwards (because the cpp Classpath was not used?), so I added the anarres jar to the Argo lib directory, and modified the Argo manifest accordingly. > Some more changes in argouml/src/argouml-build and > argouml/src/argouml-app that I won't review since these will be done in > a different way... Seee above. That was just, because Argo did not load the module with the anarres jar included. Maybe you know, how to make that work. --<snip>-- > As Tom stated we need to include anarres-cpp.jar in the argo-cpp jar. > This means that the Ant's jar target must be changed to include copying > the anarres-cpp.jar from argouml-cpp/lib/ into argouml-cpp/build/lib/ > and the argouml-cpp/src/org/manifest.mf must include lib/anarres-cpp.jar > in the "Class-Path" entry. That's what I had. It was in lib/anarres-cpp.jar , but doesn't that mean, that it should be copied in ext/lib under the modules directory? --<snip>-- > In my view, the existing RE with the preprocessor will simply fail to > work with most existing C++ projects. It will simply remove one - the > first - of the limitations of the following list (available in CppImport): > + "preprocessed files only, i.e., works on full translation > units;" --<snip>-- You are right, but I think it's still a step in the right direction, so I think it should be done. > But, I think that the current code and probably the code with the > preprocessor integrated will be worth including in the 0.26 release. > Mostly as a pre-alpha show-off of what we are doing already to recruit > developers to help removing more limitations from that list ;-). Yup. Although I doubt if anyone will join the project to work on this... --<snip>-- > Given the changes being minor I would agree. I'm a bit afraid that the > changes concerning GUI will delay 0.26, but, this is argouml-core and if > it is squeezed there I'm also in favor of extending argouml-cpp to take > advantage of it. Even without any GUI modification, it's still a step in the right direction... Ciao, Andreas -- Psssst! Schon vom neuen GMX MultiMessenger gehört? Der kann`s mit allen: http://www.gmx.net/de/go/multimessenger --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
