The Wiert Corner – irregular stream of stuff

Jeroen W. Pluimers on .NET, C#, Delphi, databases, and personal interests

  • My badges

  • Twitter Updates

  • My Flickr Stream

  • Pages

  • All categories

  • Enter your email address to subscribe to this blog and receive notifications of new posts by email.

    Join 1,825 other followers

.NET/C#: stupid code of the day – Cosmos.Debug.VSDebugEngine

Posted by jpluimers on 2010/10/13

I wasn’t the first one to notice this one, but is hilereous:

bool enabled = fEnable == 0 ? false : true;

What about a simple logical negation (!) operator?

The above sample is from the “Visual Studio Debug Engine Sampledownloads.
And it is not there once, but 3 times!

<pre>File DebugEngineSample\Microsoft.VisualStudio.Debugger.SampleEngine\AD7.Impl\AD7BoundBreakpoint.cs:
54                  bool enabled = fEnable == 0 ? false : true;
60                  m_enabled = fEnable == 0 ? false : true;
File DebugEngineSample\Microsoft.VisualStudio.Debugger.SampleEngine\AD7.Impl\AD7PendingBreakpoint.cs:
183                     m_enabled = fEnable == 0 ? false : true;

Since the COSMOS people needed debugger support, it now is (temporarily) part of AD7BoundBreakpoint.cs adding code to the Cosmos.Debug.VSDebugEngine namespace on codeplex SVN.

Note that there are lots of on-line samples like this code.

It is worse than yoda-conditions :-)

–jeroen

Leave a Reply

Fill in your details below or click an icon to log in:

WordPress.com Logo

You are commenting using your WordPress.com account. Log Out /  Change )

Google photo

You are commenting using your Google account. Log Out /  Change )

Twitter picture

You are commenting using your Twitter account. Log Out /  Change )

Facebook photo

You are commenting using your Facebook account. Log Out /  Change )

Connecting to %s

This site uses Akismet to reduce spam. Learn how your comment data is processed.

 
%d bloggers like this: