ifdef DEBUG with breakhere variables

Bug #1439414 reported by Sven D
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
KiCad
Invalid
Undecided
Unassigned

Bug Description

This is not a April Fool.

Today, I find in eeschema/class_library.cpp this

Line 221:

#if 0 && defined(DEBUG)
    if( !aName.Cmp( wxT( "TI_STELLARIS_BOOSTERPACK" ) ) )
    {
        int breakhere = 1;
        (void) breakhere;
    }
#endif

and line 242

#if defined(DEBUG) && 0
    if( !aAlias->GetName().Cmp( wxT( "TI_STELLARIS_BOOSTERPACK" ) ) )
    {
        int breakhere = 1;
        (void) breakhere;
    }
#endif

Revision history for this message
Chris Pavlina (pavlina-chris) wrote :

I'd hardly call it a bug, since it's #if 0'd off... but bizarre nonetheless

Revision history for this message
Nick Østergaard (nickoe) wrote : Re: ifdef DEBUG with breakhere variabales

There is a similar thing in the eagle_plugin.cpp. I don't know what the policy is on removal or not of theese.

tags: added: pcbnew
summary: - TI_STELLARIS_BOOSTERPACK
+ ifdef DEBUG with breakhere variabales
tags: added: code-cleanup
summary: - ifdef DEBUG with breakhere variabales
+ ifdef DEBUG with breakhere variables
Revision history for this message
Wayne Stambaugh (stambaughw) wrote : Re: [Bug 1439414] Re: ifdef DEBUG with breakhere variabales

They are for debugging purposes so there is no reason to remove them at
this time.

On 4/2/2015 6:19 AM, Nick Østergaard wrote:
> There is a similar thing in the eagle_plugin.cpp. I don't know what
> the policy is on removal or not of theese.
>
> ** Summary changed:
>
> - TI_STELLARIS_BOOSTERPACK
> + ifdef DEBUG with breakhere variabales
>
> ** Tags added: pcbnew
>

xzcvczx (xzcvczx)
Changed in kicad:
status: New → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.