FIX: Compiler Issues C1001 or Hangs with Missing Closing Brace

WINDOWS   | WINDOWS NT

The information in this article applies to:

• The Microsoft C/C++ Compiler (CL.EXE), included with:

      - Microsoft Visual C++ for Windows, versions 1.0 and 1.5
      - Microsoft Visual C++ 32-bit Edition, version 1.0
  

SYMPTOMS

Compiling the sample code below from the Visual Workbench in Visual C++ version 1.0 or an MS-DOS prompt under Windows causes the compiler to hang with no error messages being displayed. If the sample code is compiled under MS-DOS alone the following error occurs:

   fatal error C1001: INTERNAL COMPILER ERROR
   (compiler file 'msc1.cpp', line 581)

   fatal error C1001: INTERNAL COMPILER ERROR
   (compiler file 'msc1.cpp', line 585)

   fatal error C1001: INTERNAL COMPILER ERROR
   (compiler file 'msc1.cpp', line 602)
   Please choose the Technical Support command on the Visual
   Workbench Help menu, or open the Technical Support help file for
   more information.

CAUSE

The code being compiled is missing a closing brace on the switch statement. This may also occur in other situations where a closing brace is missing. For example, if the closing brace closes a function that is the last function in a file, the same errors occur. To illustrate, remove the closing brace from the CStroke::FinishStroke() member function in SCRIBDOC.CPP in step 5 of the scribble tutorial.

RESOLUTION

Fix the code by adding the missing closing brace.

STATUS

Microsoft has confirmed this to be a bug in the products listed at the beginning of this article. This problem was corrected in C/C++ version 9.0, included with Visual C++ 32-bit Edition version 2.0.

MORE INFORMATION

Sample Code

/* Compile options needed: none
*/

#include <afxwin.h>    // MFC core and standard components

    CTestApp();

CTestApp::CTestApp() { }

CTestApp NEAR theApp;

LPSTR FunctionName (char* pszFileName) {

    DWORD dwMaxLength;
    unsigned int nChars;
    LPSTR LpSrc, lpDst, lpNew;
    static char szPath [_MAX_PATH];

    while (*LpSrc)
    {
        switch (*LpSrc)
        {
            case ('\\'):
            case ('/'):
                *lpDst++ = *LpSrc++;
                nChars = 0;
                break ;

            case ('<'):
            case ('>'):
            case ('"'):
            case ('|'):
                ASSERT (FALSE);
                LpSrc++;
                break;

            default:
                if (nChars < dwMaxLength)
                    *lpDst++ = *LpSrc++;
                else
                    LpSrc++;
                nChars++;
                break;

        // Without the following closing brace for the switch
        // statement, the compiler issues the C1001 error or hangs
        // instead of generating the appropriate syntax error message.

        //}
    }

    return lpNew;