Developer forums (C::B DEVELOPMENT STRICTLY!) > Development

Too many SANITY_CHECK()'s in cbworkspace.cpp?

(1/1)

MortenMacFly:
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);

--- End code ---
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.

Navigation

[0] Message Index