Skip to content
This repository was archived by the owner on Aug 28, 2020. It is now read-only.

Commit 1fc2ec6

Browse files
committed
confirm start debug if there are no init breakpoints set and break at first line is unchecked
1 parent d6b44d6 commit 1fc2ec6

File tree

1 file changed

+23
-6
lines changed

1 file changed

+23
-6
lines changed

pugdebug/pugdebug.py

Lines changed: 23 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
import os
1313

1414
from PyQt5.QtCore import QObject
15-
from PyQt5.QtWidgets import QErrorMessage
15+
from PyQt5.QtWidgets import QErrorMessage, QMessageBox
1616

1717
from pugdebug.debugger import PugdebugDebugger
1818
from pugdebug.syntaxer import PugdebugFormatter
@@ -357,13 +357,30 @@ def start_debug(self):
357357
358358
Start a debugging session.
359359
"""
360-
self.variable_viewer.clear()
361-
self.stacktrace_viewer.clear()
360+
break_at_first_line = int(get_setting('debugger/break_at_first_line'))
361+
362+
start_debugging = True
363+
364+
if break_at_first_line == 0 and len(self.init_breakpoints) == 0:
365+
messageBox = QMessageBox()
366+
messageBox.setText("There are no breakpoints set and the break at"
367+
" first line setting is turned off.")
368+
messageBox.setInformativeText("Are you sure you want to start"
369+
" debugging?")
370+
messageBox.setStandardButtons(QMessageBox.Yes | QMessageBox.No)
371+
answer = messageBox.exec()
372+
373+
if answer == QMessageBox.No:
374+
start_debugging = False
375+
376+
if start_debugging:
377+
self.variable_viewer.clear()
378+
self.stacktrace_viewer.clear()
362379

363-
self.document_viewer.remove_line_highlights()
380+
self.document_viewer.remove_line_highlights()
364381

365-
self.debugger.start_debug()
366-
self.main_window.set_statusbar_text("Waiting for connection...")
382+
self.debugger.start_debug()
383+
self.main_window.set_statusbar_text("Waiting for connection...")
367384

368385
def handle_debugging_started(self):
369386
"""Handle when debugging starts

0 commit comments

Comments
 (0)