EasySoap++

Tracker: Bugs

5 SOAPServer: IMPLEMENT_DISPATCH doesn't specify namespace - ID: 1528386

Last Update: Comment added ( dcrowley )

Details:

The IMPLEMENT_DISPATCH macro in the CVS version of
SOAPServer.h doesn't work correctly in general, because
it uses unqualified references to
SOAPDispatchHandlerInterface and
SOAPHeaderHandlerInterface.

Rather than require typedefs or using
declarations/directives where user-supplied subclasses
are defined, these can be replaced with
namespace-qualified references as in the supplied patch.

Submitted:

Zak Kipling ( zak-k ) - 2006-07-25 13:40

Priority:

Status:

Closed

Resolution:

Fixed

Assigned:

David Crowley

Category:

bug

Group:

None

Visibility:

Public

Comments ( 2 )

Date: 2006-11-13 17:26 Sender: dcrowley Logged In: YES user_id=27458 It's fixed now. I did the tag first, noticed the patch, applied the patch, then did the release but didn't fix the tag. Thanks for reminding me.
Date: 2006-11-13 13:53 Sender: zak-k Logged In: YES user_id=764271 Thanks for applying this -- and I'm also pleased to see the appearance of 0.8.0 :) However I note, with slight confusion, that this patch appears to have made it into the 0.8.0 zipfile, even though it has only been committed to CVS HEAD, and not BRANCH_080 or EASYSOAP_0_8_0: http://easysoap.cvs.sourceforge.net/easysoap/easysoap/include/easysoap/SOAPServer.h?view=log Is this intentional? This appears to be the sole difference between the tagged version and the zipfile (which is identical to current CVS HEAD).

Date: 2006-11-13 17:26
Sender: dcrowley

Logged In: YES
user_id=27458

It's fixed now. I did the tag first, noticed the patch,
applied the patch, then did the release but didn't fix the
tag. Thanks for reminding me.

Date: 2006-11-13 13:53
Sender: zak-k

Logged In: YES
user_id=764271

Thanks for applying this -- and I'm also pleased to see the
appearance of 0.8.0 :)

However I note, with slight confusion, that this patch
appears to have made it into the 0.8.0 zipfile, even though
it has only been committed to CVS HEAD, and not BRANCH_080
or EASYSOAP_0_8_0:

http://easysoap.cvs.sourceforge.net/easysoap/easysoap/include/easysoap/SOAPServer.h?view=log

Is this intentional? This appears to be the sole difference
between the tagged version and the zipfile (which is
identical to current CVS HEAD).

Log in to comment.

Attached File ( 1 )

Filename	Description	Download
implement_dispatch.diff	Namespace-qualification patch for IMPLEMENT_DISPATCH	Download

Changes ( 6 )

Field	Old Value	Date	By
status_id	Open	2006-11-10 18:04	dcrowley
resolution_id	None	2006-11-10 18:04	dcrowley
category_id	None	2006-11-10 18:04	dcrowley
assigned_to	nobody	2006-11-10 18:04	dcrowley
close_date	-	2006-11-10 18:04	dcrowley
File Added	186165: implement_dispatch.diff	2006-07-25 13:40	zak-k

EasySoap++

Tracker: Bugs

Comments ( 2 )

Log in to comment.

Attached File ( 1 )

Changes ( 6 )

About SourceForge

Find Software

Develop Software

Community

Help