Bommels05/Mozilla
Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix yet another instance of the qsort bug that was biting OS/2 only. See explanation in cvs log comment for revision 1.25 of xpt_link.c. Thanks to Javier Pedemonte <pedemont@us.ibm.com>. a=brendan@mozilla.org

Browse Source 
git-svn-id: svn://10.0.0.236/trunk@73813 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
jband%netscape.com 2000-07-07 07:32:50 +00:00
parent 6fac7551f9
commit dc8c60aa5e
1 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
mozilla/xpcom/typelib/xpt/tools/xpt_link.c
View File

@ -448,7 +448,7 @@ main(int argc, char **argv)
if (trueNumberOfInterfaces>1) {
for (i=1; i<trueNumberOfInterfaces; i++) {
/* Only complain if the IIDs are identical and nonzero. */
if (compare_IDEs_by_IID(&IDE_array[i-1], &IDE_array[i]) == 0 &&
if (compare_IIDs(&IDE_array[i-1].iid, &IDE_array[i].iid) == 0 &&
compare_IDE_with_zero(&IDE_array[i]) != 0) {
fprintf(stderr, "FATAL ERROR:\n"
"Duplicate IID detected (");
@ -531,7 +531,11 @@ compare_IDEs_by_IID(const void *ap,
{
const XPTInterfaceDirectoryEntry *ide1 = ap, *ide2 = bp;
return compare_IIDs(&ide1->iid, &ide2->iid);
int answer = compare_IIDs(&ide1->iid, &ide2->iid);
if(!answer)
answer = compare_strings(ide1->name, ide2->name);
return answer;
}
/* For detecting unresolved interfaces. */
@ -552,7 +556,11 @@ compare_fixElements_by_IID(const void *ap,
{
const fixElement *fix1 = ap, *fix2 = bp;
return compare_IIDs(&fix1->iid, &fix2->iid);
int answer = compare_IIDs(&fix1->iid, &fix2->iid);
if(!answer)
answer = compare_strings(fix1->name, fix2->name);
return answer;
}
static int