help with gcc 3.1 error message

[ken]

I cannot see how I am getting this error message:

TreeAddress aResult( memory::Pointer(aBaseAddress) );

if (TreeFragment * pFragment = TreeAccessor::access(aResult,_anUpdater))
/data2/office/configmgr/source/tree/builddata.cxx: In member function
`configmgr ::data::TreeAddress configmgr::data::TreeNodeBuilder::
allocTreeFragment(configmg r::memory::UpdateAccessor&)':
/data2/office/configmgr/source/tree/builddata.cxx:658: error: no matching
function for
call to `configmgr::data::TreeAccessor::access(configmgr:: data::TreeAddress (&)
(configmgr::memory::Pointer), configmgr::memory::UpdateAccessor&)'
.../inc/treeaccessor.hxx:139: error: candidates are
: static configmgr::sharable:: TreeFragment*
configmgr::data::TreeAccessor::
access(const configmgr::data::TreeA ddress&,
configmgr::memory::UpdateAccessor&)
.../inc/treeaccessor.hxx:141: error: static const configmgr::shar
able::TreeFragment*
configmgr::data::TreeAccessor::access(const configmgr::data:
:TreeAddress&,
const configmgr::memory::Accessor&)

[Rob Williscroft]

ken wrote in news:pa***************************@gandalf.foskey. org:

I cannot see how I am getting this error message:

TreeAddress aResult( memory::Pointer(aBaseAddress) );
The above is a function declaration ie:

TreeAddress aResult( memory::Pointer aBaseAddress );

change it to:

TreeAddress aResult = memory::Pointer(aBaseAddress);

or maybe:

TreeAddress aResult( ( memory::Pointer(aBaseAddress) ) );


if (TreeFragment * pFragment =
TreeAccessor::access(aResult,_anUpdater))
/data2/office/configmgr/source/tree/builddata.cxx: In member function
`configmgr ::data::TreeAddress configmgr::data::TreeNodeBuilder::
allocTreeFragment(configmg r::memory::UpdateAccessor&)':
/data2/office/configmgr/source/tree/builddata.cxx:658: error: no
matching function for
call to
This is the bit you needed to read carefully:
`configmgr::data::TreeAccessor::access(configmgr:: data::TreeAddress
(&) (configmgr::memory::Pointer), configmgr::memory::UpdateAccessor&)'
Note the type of the first argument.
../inc/treeaccessor.hxx:139: error: candidates are
: static configmgr::sharable:: TreeFragment*
configmgr::data::TreeAccessor::
access(const configmgr::data::TreeA ddress&,
configmgr::memory::UpdateAccessor&)
../inc/treeaccessor.hxx:141: error: static const configmgr::shar
able::TreeFragment*
configmgr::data::TreeAccessor::access(const configmgr::data:
:TreeAddress&,
const configmgr::memory::Accessor&)

HTH

Rob.
--
http://www.victim-prime.dsl.pipex.com/

[Michael Kochetkov]

"Rob Williscroft" <rt*@freenet.REMOVE.co.uk> wrote in message
news:Xn**********************************@195.129. 110.200...

ken wrote in news:pa***************************@gandalf.foskey. org:

I cannot see how I am getting this error message:

TreeAddress aResult( memory::Pointer(aBaseAddress) );

The above is a function declaration ie:

TreeAddress aResult( memory::Pointer aBaseAddress );

change it to:

TreeAddress aResult = memory::Pointer(aBaseAddress);

or maybe:

TreeAddress aResult( ( memory::Pointer(aBaseAddress) ) );

No, it will not help. You might with to consider the proper static_cast
instead.

--
With regards,
Michael Kochetkov.

[ken]

On Sun, 20 Jul 2003 19:09:01 +0400, Michael Kochetkov wrote:

"Rob Williscroft" <rt*@freenet.REMOVE.co.uk> wrote in message
news:Xn**********************************@195.129. 110.200...

ken wrote in news:pa***************************@gandalf.foskey. org:

> I cannot see how I am getting this error message:
>
> TreeAddress aResult( memory::Pointer(aBaseAddress) );

The above is a function declaration ie:

TreeAddress aResult( memory::Pointer aBaseAddress );

change it to:

TreeAddress aResult = memory::Pointer(aBaseAddress);

or maybe:

TreeAddress aResult( ( memory::Pointer(aBaseAddress) ) );

No, it will not help. You might with to consider the proper static_cast
instead.

Static cast was the original code. It compiled fine until 3.3.1

TreeAddress aResult( static_cast<memory::Pointer>(aBaseAddress) );

Isn't memory::Pointer(aBaseAddress) instantiating a temporary of type
memory::Pointer, not declaring a variable ABaseAddress. The brackets
are calling the constructor.

I now understand how to work around this, I just don't understand what I did
wrong.

--
Ta
KenF

[John Harrison]

"ken" <ke*@gandalf.foskey.org> wrote in message
news:pa****************************@gandalf.foskey .org...

On Sun, 20 Jul 2003 19:09:01 +0400, Michael Kochetkov wrote:

"Rob Williscroft" <rt*@freenet.REMOVE.co.uk> wrote in message
news:Xn**********************************@195.129. 110.200...

ken wrote in news:pa***************************@gandalf.foskey. org:

> I cannot see how I am getting this error message:
>
> TreeAddress aResult( memory::Pointer(aBaseAddress) );

The above is a function declaration ie:

TreeAddress aResult( memory::Pointer aBaseAddress );

change it to:

TreeAddress aResult = memory::Pointer(aBaseAddress);

or maybe:

TreeAddress aResult( ( memory::Pointer(aBaseAddress) ) ); No, it will not help. You might with to consider the proper static_cast
instead.

Static cast was the original code. It compiled fine until 3.3.1

TreeAddress aResult( static_cast<memory::Pointer>(aBaseAddress) );

Isn't memory::Pointer(aBaseAddress) instantiating a temporary of type
memory::Pointer, not declaring a variable ABaseAddress. The brackets
are calling the constructor.

I now understand how to work around this, I just don't understand what I

did wrong.

No, memory::Pointer(aBaseAddress) is a parameter type, followed by a
parameter name. The brackets are ignored. I.e.

TreeAddress aResult( memory::Pointer(aBaseAddress) );

is exactly the same as

TreeAddress aResult( memory::Pointer aBaseAddress );

which is a function prototype.

There is a rule in C++, if something could be an expression or a
declaration, its treated as a declaration. See the recent thread 'Temporary
creation vs. variable declaration' for other examples.

john

[Michael Kochetkov]

"Rob Williscroft" <rt*@freenet.REMOVE.co.uk> wrote in message
news:Xn**********************************@195.129. 110.131...

Michael Kochetkov wrote in news:3f********@news.trustworks.com:

"Rob Williscroft" <rt*@freenet.REMOVE.co.uk> wrote in message
news:Xn**********************************@195.129. 110.200...

ken wrote in news:pa***************************@gandalf.foskey. org:

> I cannot see how I am getting this error message:
>
> TreeAddress aResult( memory::Pointer(aBaseAddress) );

The above is a function declaration ie:

TreeAddress aResult( memory::Pointer aBaseAddress );

change it to:

TreeAddress aResult = memory::Pointer(aBaseAddress);

or maybe:

TreeAddress aResult( ( memory::Pointer(aBaseAddress) ) );

No, it will not help. You might with to consider the proper static_cast
instead.

Maybe, but if so, you and the OP have non-comforming compilers.

Right you are. I have overlooked the opening parenthesis place (I have
considered TreeAddress (aResult( memory::Pointer(aBaseAddress) ) ); ).
Sorry.

--
With regards,
Michael Kochetkov.

[ken]

On Mon, 21 Jul 2003 07:08:31 +0100, John Harrison wrote:

There is a rule in C++, if something could be an expression or a
declaration, its treated as a declaration. See the recent thread 'Temporary
creation vs. variable declaration' for other examples.

The penny drops, thanks for your help.

--
KenF