Debugger: Avoid endless loop when setting breakpoint on functions

Change-Id: I3ead86ba50d1d68a46cdc58c06684b2ac8c091c0
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@digia.com>

src/plugins/debugger/breakhandler.cpp

@@ -1439,8 +1439,13 @@ bool BreakHandler::BreakpointItem::needsChange() const
         return true;
     if (data.command != response.command)
         return true;
-    if (data.lineNumber != response.lineNumber)
+    if (data.type == BreakpointByFileAndLine && data.lineNumber != response.lineNumber)
         return true;
+    // FIXME: Too strict, functions may have parameter lists, or not.
+    // if (data.type == BreakpointByFunction && data.functionName != response.functionName)
+    //     return true;
+    // if (data.type == BreakpointByAddress && data.address != response.address)
+    //     return true;
     return false;
 }