Boost logo

Boost :

From: Nicola Musatti (Nicola.Musatti_at_[hidden])
Date: 2006-06-06 17:56:09


Nicola Musatti wrote:

> Robert Ramey wrote:
[...]
> I've just checked. Your changes cause 30 tests to pass which didn't when
> I started looking into this. However my proposed patch would have caused
> 34 more tests to pass. In principle it should be possible to get to a
> stage where only 4 regressions remain over bcc32 5.6.4 .
>
>>Note that I made a change in basic_text_oarchive.hpp
>>in lieu of your suggestion for basic_text_oprmitive.hpp. I believe
>>that this is in the correct place. We're still geting errors however
>>so could you please take another look at this?

The enclosed patch reduces regressions over 5.6.4 to a total of 20. See
how you like it.

Cheers,
Nicola Musatti


******Index: basic_text_oarchive.hpp
===================================================================
RCS file: /cvsroot/boost/boost/boost/archive/basic_text_oarchive.hpp,v
retrieving revision 1.9.2.1
diff -d -u -r1.9.2.1 basic_text_oarchive.hpp
--- basic_text_oarchive.hpp 30 May 2006 15:58:22 -0000 1.9.2.1
+++ basic_text_oarchive.hpp 6 Jun 2006 21:07:50 -0000
@@ -97,6 +97,18 @@
         * this->This() << x;
     }
 
+ void save_override(const class_id_type & t, int){
+ // note:t.t resolves borland ambguity
+ unsigned char x = t.t;
+ * this->This() << x;
+ }
+
+ void save_override(const class_id_reference_type & t, int){
+ // note:t.t resolves borland ambguity
+ unsigned char x = t.t;
+ * this->This() << x;
+ }
+
     void save_override(const class_name_type & t, int){
         const std::string s(t);
         * this->This() << s;


Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk