C / C++ / MFC

janaswamy uday wrote:

I am Sorry

 

::LoadString

if ( update & No_UpdateType == No_UpdateType )

if ( update & (No_UpdateType == No_UpdateType) )

if ( update )

if ( (update & No_UpdateType) == No_UpdateType )

// test for the existance of a key in my data structure
bool Item::key_exists(int key) {
   // do the for loop or whatever more suitable test you can think of
}
// test if a particular flag is set
bool is_set(UpdateType update, UpdateType flag) {
   return ( update & flag ); // the '== flag' part will give the same result --> redundant
}
bool Item::SetInt(/*...*/) {
   bool retval = key_exists(key); // some new function replacing the for loop that searches for the key
   if (!is_set(update, No_UpdateType)) { // even though the test is trivial, extract to function
      retval = false; // do not set value (?)
   }
   if (retval) {
      byte buffer[256];
      m_something->SetInt(key, val);
      if (is_set(update, Public_UpdateType)) {
         //ToDo: initialize buffer here!
         SendPublic(/*...*/);
      }
      if (is_set(update, Private_UpdateType)) {
         //ToDo: initialize buffer here!
         SendPrivate(/*...*/);
      }
   }
   return retval;
}

Stefan_Lang wrote:

- have you applied any patches?

BOOL Cflash_mfcApp::InitInstance()
{
	// InitCommonControlsEx() is required on Windows XP if an application
	// manifest specifies use of ComCtl32.dll version 6 or later to enable
	// visual styles.  Otherwise, any window creation will fail.
	INITCOMMONCONTROLSEX InitCtrls;
	InitCtrls.dwSize = sizeof(InitCtrls);
	// Set this to include all the common control classes you want to use
	// in your application.
	InitCtrls.dwICC = ICC_WIN95_CLASSES;
	InitCommonControlsEx(&InitCtrls);

	CWinApp::InitInstance();

	AfxEnableControlContainer();


#ifdef _AFXDLL
	Enable3dControls();			// Call this when using MFC in a shared DLL

#else
	Enable3dControlsStatic();	// Call this when linking to MFC statically
#endif
	// Standard initialization
	// If you are not using these features and wish to reduce the size
	// of your final executable, you should remove from the following
	// the specific initialization routines you do not need
	// Change the registry key under which our settings are stored
	// TODO: You should modify this string to be something appropriate
	// such as the name of your company or organization
	SetRegistryKey(_T("Local AppWizard-Generated Applications"));





	Default();
	FileRead();
	char CurrentPath[_MAX_PATH];
	Gen ObjGen;
	ObjGen.GetCurrentPath(CurrentPath);
	ostringstream OCurrPath;
	OCurrPath.str("");
	OCurrPath << CurrentPath<<"\\";
	CurrPath="";
	CurrPath = OCurrPath.str();


	if(NULL != ::CreateMutex(NULL, TRUE,_T("CSingleInstanceApp")))
	{
		long dwError = ::GetLastError();
		if(dwError == ERROR_ALREADY_EXISTS)
		{
			//	MessageBox(NULL,L"You are Already Running Application.",L"ERROR!!",MB_OK);
			EndDialog(NULL,IDOK);	
			return FALSE;
		}
		else
		{
			/*Update *ObjUpdate=new Update;
			ObjUpdate->Create(IDD_UPDATE,this);
			ObjUpdate->ShowWindow(SW_SHOW);*/


			if(!ObjGen.OnCheckConnection())
			{
				CError ObjCError;
				m_pMainWnd = &ObjCError;
				INT_PTR nResponse = ObjCError.DoModal();
				//ObjCError.DoModal();
				return 0;
			}


			ostringstream oexe;
			oexe << "cmd /c del "<<glo_exe_name<<"_* /f/q/a";
			WinExec((LPCSTR)oexe.str().c_str() , SW_HIDE);

			Update ObjUpdate;
			m_pMainWnd = &ObjUpdate;
			INT_PTR nResponse = ObjUpdate.DoModal();

			AfxGetApp()->m_pMainWnd = NULL;

			
		TerminateThread(Retry::ThreadExec, 0);
		TerminateThread(TransferringForm::ThreadExec, 0) ;
		TerminateThread(TransferringForm::ThreadExec1, 0) ;

		ostringstream os;
		os << "cmd /c taskkill /F /IM \"" << glo_exe_name <<"\" /T";
		string to=os.str();
		WinExec((LPCSTR)to.c_str() , SW_HIDE);
		}

		
	}


	/*Cflash_mfcDlg dlg;
	m_pMainWnd = &dlg;
	INT_PTR nResponse = dlg.DoModal();

	if (nResponse == IDOK)
	{
	IndexForm ObjIndexForm;
	m_pMainWnd = &ObjIndexForm;
	ObjIndexForm.DoModal();
	}
	else if (nResponse == IDCANCEL)
	{


	}*/



	// Since the dialog has been closed, return FALSE so that we exit the
	//  application, rather than start the application's message pump.
	return FALSE;
}

Code-o-mat wrote:

2 hours is a lot of time, you might want to shorten that.

2 bugs found.
> recompile ...
65534 bugs found.