AMDG Phil Bouchard wrote:

"Steven Watanabe" <watanabesj@gmail.com> wrote in message news:480399C5.60009@providere-consulting.com...

...

I just tried it out on msvc 9.0.

It breaks on sh_owned_base_nt.hpp line 107 because of the windows min/max macros

Thanks I just added some undefs since I haven't seen the necessity of pushing / popping them.

You should probably use (std::numeric_limits<>::min)() which disables macro expansion. Otherwise, the inspection program will complain. In Christ, Steven Watanabe

AMDG Phil Bouchard wrote:

"Steven Watanabe" <watanabesj@gmail.com> wrote in message news:48051355.3010309@providere-consulting.com...

...

You should probably use (std::numeric_limits<>::min)() which disables macro expansion. Otherwise, the inspection program will complain.

Res gestae.

Ok. Now it compiles. Some comments on the documentation. "Memory usage having a power of 2 (sizeof(void *) * 2)" What does this mean? The use of the term "cubic" always catches me because I automatically associate it with O(n^3) which is not what you mean. It would also be nice to know what kinds of cycles it doesn't handle. Finally, here are the warnings from msvc 9.0 /w4 Warning 1 warning C4355: 'this' : used in base member initializer list c:\documents and settings\steven\my documents\vendor\boost\shifted_ptr\shifted_ptr\boost\detail\intrusive_list.hpp 39 Warning 2 warning C4355: 'this' : used in base member initializer list c:\documents and settings\steven\my documents\vendor\boost\shifted_ptr\shifted_ptr\boost\detail\intrusive_list.hpp 39 Warning 3 warning C4100: 'ti' : unreferenced formal parameter c:\documents and settings\steven\my documents\vendor\boost\shifted_ptr\shifted_ptr\boost\detail\sh_owned_base_nt.hpp 206 Warning 4 warning C4099: 'boost::detail::sh::owned_base' : type name first seen using 'class' now seen using 'struct' c:\documents and settings\steven\my documents\vendor\boost\shifted_ptr\shifted_ptr\boost\shifted_ptr.hpp 42 Warning 5 warning C4355: 'this' : used in base member initializer list c:\documents and settings\steven\my documents\vendor\boost\shifted_ptr\shifted_ptr\boost\shifted_ptr.hpp 61 Warning 6 warning C4100: 'other' : unreferenced formal parameter c:\documents and settings\steven\my documents\visual studio 2008\projects\shifted_ptr\shifted_ptr\shifted_ptr.cpp 38 Warning 7 warning C4244: 'argument' : conversion from 'int' to 'unsigned short', possible loss of data c:\documents and settings\steven\my documents\boost\boost\date_time\microsec_time_clock.hpp 179 Warning 8 warning C4244: 'argument' : conversion from 'int' to 'unsigned short', possible loss of data c:\documents and settings\steven\my documents\boost\boost\date_time\microsec_time_clock.hpp 179 Warning 9 warning C4244: 'argument' : conversion from 'int' to 'unsigned short', possible loss of data c:\documents and settings\steven\my documents\boost\boost\date_time\microsec_time_clock.hpp 179 Warning 10 warning C4100: 'ti' : unreferenced formal parameter c:\documents and settings\steven\my documents\vendor\boost\shifted_ptr\shifted_ptr\boost\detail\sh_owned_base_nt.hpp 136 Warning 11 warning C4100: 'ti' : unreferenced formal parameter c:\documents and settings\steven\my documents\vendor\boost\shifted_ptr\shifted_ptr\boost\detail\sh_owned_base_nt.hpp 136 Warning 12 warning C4512: 'boost::algorithm::detail::find_format_store<ForwardIteratorT,FormatterT,FormatResultT>' : assignment operator could not be generated c:\documents and settings\steven\my documents\boost\boost\algorithm\string\detail\find_format_store.hpp 65 You may just have to #pragma warning push/pop disable 4355 In Christ, Steven Watanabe

AMDG Phil Bouchard wrote:

"Steven Watanabe" <watanabesj@gmail.com> wrote in message news:48061126.40705@providere-consulting.com...

[...]

...

It would also be nice to know what kinds of cycles it doesn't handle.

[...]

Sorry for the quick email... I have to go but it clears everything up the heap, there is no possible cyclicism.

really? #include <boost/shifted_ptr.hpp> #include <vector> #include <iostream> static int count; struct X { X() { ++count; } ~X() { --count; } X(const X& other) : vec(other.vec) { ++count; } std::vector<boost::shifted_ptr<X> > vec; }; int main() { { boost::shifted_ptr<X> x = boost::new_sh<X>(); x->vec.push_back(x); } std::cout << count << std::endl; } In Christ, Steven Watanabe

AMDG Phil Bouchard wrote:

I have discovered also another bug related to the set counter. Now Vault contains the corrected version.

Ok. Another issue. Is it possible to deal with this: The nodes will never be cleaned up until the list is destroyed. Or that's what I thought.... The nodes don't seem to be cleaned up at all. #include <boost/shifted_ptr.hpp> #include <vector> #include <iostream> static int count; using boost::shifted_ptr; using boost::new_sh; struct node { node() { ++count; } ~node() { --count; } node(const node&) { ++count; } shifted_ptr<node> prior; shifted_ptr<node> next; }; struct list { public: list() {} void clear() { front.reset(); back.reset(); } void insert() { if(front.get() == 0) { front = back = new_sh<node>(); } else { back->next = new_sh<node>(); back->next->prior = back; back = back->next; } } private: shifted_ptr<node> front; shifted_ptr<node> back; }; int main() { list l; for(int j = 0; j < 10; ++j) { for(int i = 0; i < 100; ++i) { l.insert(); } l.clear(); } std::cout << count << std::endl; } In Christ, Steven Watanabe

"Steven Watanabe" <watanabesj@gmail.com> wrote in message news:48061126.40705@providere-consulting.com... [...]

"Memory usage having a power of 2 (sizeof(void *) * 2)" What does this mean?

sizeof(void *) + sizeof(void *) == sizeof(shifted_ptr<void>)

The use of the term "cubic" always catches me because I automatically associate it with O(n^3) which is not what you mean.

It's like a memory matrix of the 3rd order bringing 3 distinct ways of isolating unreferenced objects. [...]

You may just have to #pragma warning push/pop disable 4355

I just made the corrections. Vc 8 isn't reporting such warnings. -Phil