Author Topic: Too many SANITY_CHECK()'s in cbworkspace.cpp?  (Read 3447 times)

Offline MortenMacFly

  • Administrator
  • Lives here!
  • *****
  • Posts: 9660
Too many SANITY_CHECK()'s in cbworkspace.cpp?
« on: March 31, 2006, 08:16:34 pm »
Looking into cbworkspace.cpp at lines 106-113 I see:
Code
  m_IsOK = pWsp && (pWsp->Open(fname) || m_IsDefault);
  SANITY_CHECK();
  m_Title = pWsp ? pWsp->GetTitle() : wxString(wxEmptyString);
  SANITY_CHECK();

  SANITY_CHECK();
  m_Filename.SetExt(FileFilters::WORKSPACE_EXT);
  SetModified(modified);
Not that it is really of concern, but aren't these too many SANITY_CHECK()'s? I believe at least the two in the middle could be reduced to one. Otherwise it looks like kind of INSANITY_CHECK() to me... ;-)
With kind regards, Morten.
(Version 1.0 revision 2285 (gcc 3.4.5 Windows/unicode, build: Mar 30 2006 18:55:32))
Edit: Typos.
« Last Edit: March 31, 2006, 08:19:20 pm by MortenMacFly »
Compiler logging: Settings->Compiler & Debugger->tab "Other"->Compiler logging="Full command line"
C::B Manual: http://www.codeblocks.org/docs/main_codeblocks_en.html
C::B FAQ: http://wiki.codeblocks.org/index.php?title=FAQ