1622 – parameters to TypeInfo_Struct.compare seem to be switched around.

D issues are now tracked on GitHub. This Bugzilla instance remains as a read-only archive.

Issue 1622 - parameters to TypeInfo_Struct.compare seem to be switched around.

Summary: parameters to TypeInfo_Struct.compare seem to be switched around.

Keywords:
Status:	RESOLVED FIXED
Alias:	None
Product:	D
Classification:	Unclassified
Component:	phobos (show other issues)
Version:	D1 (retired)
Hardware:	x86 All
Importance :	P2 trivial
Assignee:	Walter Bright
URL:
Depends on:
Blocks:

See Also:
Reported:	2007年10月28日 21:21 UTC by Tomas Lindquist Olsen
Modified:	2014年02月24日 15:59 UTC (History)
CC List:	0 users

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.

Description Tomas Lindquist Olsen 2007年10月28日 21:21:47 UTC

TypeInfo_Struct.compare in the implementation of 'module object;' is declared as:
int compare(void *p2, void *p1);
I think p2 and p1 should be reversed. This makes sense from the calls to xopCmp and memcmp.

Comment 1 Tomas Lindquist Olsen 2008年07月11日 13:52:31 UTC

this is an obvious and easy to fix error. shouldn't really take about a year to fix!

Comment 2 Walter Bright 2008年08月14日 02:59:09 UTC

Fixed dmd 1.034 and 2.018