I have a proposed fix that includes the Period declaration for Natural, but
takes out the one for Size (which doesn't make any sense anyway). Since we now have both a signed and unsigned QL_INTEGER declaration, this should compile on anything. Thoughts? On Thursday 08 March 2007 04:19:37 Luigi Ballabio wrote: > On Thu, 2007-03-08 at 09:52 +0000, Francois du Vignaud wrote: > > Modified Files: > > period.hpp > > Log Message: > > msvc compilation error fix ->roll back of last J Wang commit: > > fix so that code is ISO compliant > > Francois, > I had the same problem on gcc 4.1, but I guess Joe had some other > problem with his compiler, so just reverting the change is only likely > to start an edit war like those on Wikipedia. The correct way to proceed > would be to contact Joe and compare notes. > > I understand that you folks at Caboto want the library to compile, so > it's fine if you commit the fix to distribute it to your colleagues (of > course, your real problem is that you're relying on the main repository > for production work, instead of having an internal one.) > > Just don't let the problem drop. > > Later, > Luigi > > > ---------------------------------------- > > The box said "Use Windows 95 or better," so I got a Macintosh. -- ------------------------------------------------------------------------------- Joseph Wang Ph.D. - [hidden email] China Derivatives Researcher and Software Developer - QuantLib http://en.wikiversity.org/wiki/User:Roadrunner ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ QuantLib-dev mailing list [hidden email] https://lists.sourceforge.net/lists/listinfo/quantlib-dev |
On Wed, 2007-03-14 at 19:09 -0500, Joseph Wang wrote:
> I have a proposed fix that includes the Period declaration for Natural, but > takes out the one for Size (which doesn't make any sense anyway). Since we > now have both a signed and unsigned QL_INTEGER declaration, this should > compile on anything. I think one might still have problems if QL_INTEGER is redefined to long. In the end, I just tried putting in a template version. Let me know if it works. Luigi ---------------------------------------- There is no opinion so absurd that some philosopher will not express it. -- Marcus Tullius Cicero, "Ad familiares" ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ QuantLib-dev mailing list [hidden email] https://lists.sourceforge.net/lists/listinfo/quantlib-dev |
On Wednesday 21 March 2007 12:15:08 Luigi Ballabio wrote:
> I think one might still have problems if QL_INTEGER is redefined to > long. In the end, I just tried putting in a template version. Let me > know if it works. Neat!!!! Works fine!!! Let me go through the rest of the code to see if there are any types that should be template functions. --- -------------------------------------------------------------------- Joseph Wang Ph.D. - [hidden email] China Derivatives Researcher and Software Developer - QuantLib http://en.wikiversity.org/wiki/User:Roadrunner ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ QuantLib-dev mailing list [hidden email] https://lists.sourceforge.net/lists/listinfo/quantlib-dev |
Free forum by Nabble | Edit this page |