Status

MozillaClassic
P3
minor
VERIFIED FIXED
20 years ago
19 years ago

People

(Reporter: David Nebinger, Assigned: Chris McAfee)

Tracking

1998-09-04
HP
HP-UX

Details

(Reporter)

Description

20 years ago
Created by David Nebinger (dnebinger@synertech.highmark.com) on Wednesday, April 15, 1998 7:43:33 AM PDT
Additional Details :
HP's compiler **** out on line 844 of file
ns/cmd/xfe/icons/mkicons.c, and issues the error:
cc: line 844: internal error 5172: Backend Assert ** Cause
of inconsistency unknown. (5172)

Looking at line 844, there are a series of trigraphs linked
together that I think is causing the problem.  By taking a
less sophisticated approach (code follows below), the
compile completed without complaint.

Code replacement follows:
  /* Create a stream and decode the image. */
#ifndef HPUX
  stream = IL_NewStream (FO_PRESENT,
                         (strstr (file, ".gif") ? (void *)
IL_GIF :
                          strstr (file, ".jpg") ? (void *)
IL_JPEG :
                          strstr (file, ".jpeg") ? (void *)
IL_JPEG :
                          strstr (file, ".xbm") ? (void *)
IL_XBM :
                          (void *) IL_GIF),
                         url, &cx);
#else
  {
    void* il_type_tmp=(void*)IL_GIF;

    if ((strstr(file,".jpg"))||(strstr(file,".jpeg")))
il_type_tmp=(void*)IL_JPEG;
    else if (strstr(file,".xbm")) il_type_tmp=(void*)IL_XBM;

    stream=IL_NewStream(FO_PRESENT,il_type_tmp,url,&cx);
  }
#endif
(Assignee)

Updated

19 years ago
Component: Platform: Rhapsody → XFE
(Assignee)

Comment 1

19 years ago
What version of HP-UX and which compiler were you using?
I'd like to see this fail and test the fix before checking
this in, we've been building on HP-UX Ok for a while.

Updated

19 years ago
Status: NEW → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
Version: 1998-03-31 → 1998-09-04

Comment 2

19 years ago
This bug has been closed, certainly with the 4th September 1998 release of
Mozilla. Not only has the fix been incorporated, but mkicons.c is now actually
mkicons.cpp :-)

Updated

19 years ago
Status: RESOLVED → VERIFIED

Comment 3

19 years ago
marking verified against old code base
You need to log in before you can comment on or make changes to this bug.