Skip to main content
aboutsummaryrefslogtreecommitdiffstats
path: root/org.eclipse.debug.ui/ui/org/eclipse/debug/internal
AgeCommit message (Collapse)AuthorFilesLines
2018-10-18Bug 540259 - NPE in ProcessConsole$InputReadJob.runI20181019-0450I20181019-0330I20181018-1800Andrey Loskutov1-3/+7
Change-Id: Ib4c5510e95e2d707e9b649c010cc6d16ffe01609 Signed-off-by: Andrey Loskutov <loskutov@gmx.de>
2018-10-18Bug 540242 - SWTException in InternalTreeModelViewer.getDisplayAndrey Loskutov2-13/+11
ElementMementoRequest should not try to call done() on already disposed viewer Change-Id: I09276f7f001a4c1725519183ee1a05d8485c6fb2 Signed-off-by: Andrey Loskutov <loskutov@gmx.de>
2018-10-18Bug 491273 - remove copy/pasted getClosestMonitor() implementationAndrey Loskutov1-39/+2
Change-Id: I66ec9cdcde9ea31977ae5b5f77282edcdd22a684 Signed-off-by: Andrey Loskutov <loskutov@gmx.de>
2018-09-14Bug 355011 - Deadlock on creating console in the build threadI20180914-0620Andrey Loskutov1-16/+61
Do not create console directly in the launchChanged() callback, because the current thread may hold already some lock, and if the ProcessConsole() init requires another one lock, we will have a classic deadlock. In case of bug 355011 current thread owns workspace lock and ProcessConsole() init requires UI lock. Change-Id: Ia01df4cb3f714b16fce84620538e66aa3de10b76 Signed-off-by: Andrey Loskutov <loskutov@gmx.de>
2018-09-14Bug 537903 - Wrong Debug View due to evaluation on debug context changeSimeon Andreev1-2/+12
Whenever an evaluation runs during thread suspend handling, its possible to create a delta which selects the suspended thread but not its first stack frame. This results due to very specific interleaving of thread running state and code in ThreadEventHandler.fireDeltaUpdatingSelectedFrame(). An evaluation can e.g. be triggered by a job in client code, either with IJavaThread.runEvaluation() or with IJavaObject.sendMessage() methods. No synchronization is possible between those methods and the suspend handling, since such synchronization can block debug event processing and so potentially lead to deadlocks. This change attempts to salvage the situation, after the bad case is encountered. If no top stack frame could be retrieved by ThreadEventHandler.fireDeltaUpdatingSelectedFrame(), but the thread is in suspended state, we query the top stack frame again. In case one evaluation ran in parallel with the suspend handling, at this point we know for sure that the evaluation is done and so the top stack frame is accessible. This change does not help in case client code bombards the thread with evaluations. It also does not help in case an evaluation ran throughout the entire thread suspend handling. Change-Id: I34252c0dac3d031bbbefc3a1328175cab0793ae5 Signed-off-by: Simeon Andreev <simeon.danailov.andreev@gmail.com>
2018-09-12Bug 538548 - Debug View in another perspective causes editor to pop-upY20180912-2200I20180913-1800I20180913-0415I20180912-2240Simeon Andreev1-1/+3
If a window has the Debug View open in an inactive perspective, breakpoint hits will cause an editor to open in that window. This hinders the use of multiple windows, each with a different type of perspective. Only windows which have the Debug View in their active perspective should show source on breakpoint hit. The editor is activated due to the Debug View reacting to selection events, which occur due to breakpoint hits. This change ensures that the Debug View will react to selection events only if its active. I.e. only if the active perspective of the window contains the Debug View. Change-Id: Ida92f5322e1a83b95af4bc43f25a9fac9d4519b6 Signed-off-by: Simeon Andreev <simeon.danailov.andreev@gmail.com>
2018-09-11Bug 538303 - Debug View wrong selection if activated by breakpointI20180912-0225I20180912-0210I20180911-1400Simeon Andreev1-1/+1
If a breakpoint is hit when the Debug View is not active, the Debug View is activated. In some cases, this leads to a bad selection and expanded state. When the Debug View is activated, either by a breakpoint or by a user, a state is saved in ViewerStateTracker. This state is cancelled by TreeModelContentProvider on breakpoint hit, however without removing COLLAPSED delta states. As such, applying the state later on will lead to a collapsed thread, if the state had a collapsed flag for that thread. This "overwrites" a thread selected by suspend handling. This change should ensure that COLLAPSED thread state in ViewerStateTracker is cleared when an expand is handled, regardless if the tree had to be expanded or was expanded already by another operation. Change-Id: I7c43b8317081747c6d8e85020e7040e928c4f39b Signed-off-by: Simeon Andreev <simeon.danailov.andreev@gmail.com>
2018-08-21Bug 535802 - EPL-2.0 for platform.debug plug-insY20180823-0800Y20180822-2200S4_9_0_M3I20180823-0450I20180823-0430I20180823-0100I20180822-2000I20180822-0800I20180821-2000Lars Vogel741-2233/+4446
Commands used for this conversion: git checkout master git reset --hard origin/master git pull find . -type f ! -name "*.exsd" ! -name "org.eclipse.jdt.ui.prefs" -exec sed -i 's/Eclipse Public License v1.0/Eclipse Public License 2.0/g' {} + find . -type f ! -name "*.exsd" ! -name "org.eclipse.jdt.ui.prefs" -exec sed -i -E 's/([[:blank:]]*[^[:blank:]])*([[:blank:]]*)http:\/\/www.eclipse.org\/legal\/epl-v10.html/\1\2https:\/\/www.eclipse.org\/legal\/epl-2.0\/\ \1\ \1\2SPDX-License-Identifier: EPL-2.0/' {} + # Get rid of the All rights reserved find . -type f ! -name "*.exsd" ! -name "org.eclipse.jdt.ui.prefs" ! -name "pom.xml" -exec sed -i -E 's/([[:blank:]]*[^[:blank:]])*([[:blank:]]*)(All rights reserved. )/\1\n\1\2/' {} + Change-Id: I3c49f3d254233c87acef9c83b3eb9ba831b9d6ff Signed-off-by: Lars Vogel <Lars.Vogel@vogella.com>
2018-08-20Bug 538072 - Show Command Line button is shown when there are no launchI20180820-0800Sarika Sinha1-0/+1
configurations Change-Id: Icd83fcd3fd3b61bbedec07c3873f4ee39376f11f
2018-08-14Bug 537525 - Follow up of Bug 312397 - Run Configuration Command Line toI20180819-2000I20180819-0800I20180818-1500I20180818-0800I20180817-2000I20180816-2000Sarika Sinha4-4/+57
Clipboard Change-Id: Idfb989cb7ff3d47c4162becea5227a7c04e975cc
2018-08-14Bug 537525 - Follow up of Bug 312397 - Run Configuration Command Line toY20180815-2200I20180815-2000I20180815-1250I20180814-2000I20180814-0910I20180814-0900Sarika Sinha2-1/+8
Clipboard Change-Id: If197112aa3f36344e0efc5d77be169ae743d9e27
2018-08-01Bug 312397 - Run Configuration Command Line to ClipboardY20180808-2200Y20180801-2200S4_9_0_M2I20180808-2000I20180807-2000I20180807-1425I20180807-0350I20180806-2000I20180806-0300I20180805-2000I20180804-1500I20180803-2000I20180801-2000I20180801-0820I20180801-0800Sarika Sinha1-0/+1
Change-Id: I23b39a6165a1a50723fd41aaeb5e91e945d7485c
2018-07-31Bug 312397 - Run Configuration Command Line to ClipboardI20180801-0550I20180731-2000I20180731-1305I20180731-1250I20180731-1025I20180731-0800Sarika Sinha2-9/+11
Change-Id: I6d6b01ad2bda796a43c77399a6a62b2f59edd9fc
2018-07-29Bug 312397 - Run Configuration Command Line to ClipboardI20180731-0340I20180730-2240I20180730-2000I20180730-0800I20180729-2000Sarika Sinha4-20/+50
Change-Id: I19d054bed10f834c52bc00f0397255f870b4d6c8
2018-07-28Bug 312397 - Run Configuration Command Line to ClipboardI20180729-0800I20180728-1500I20180728-0805Sarika Sinha4-2/+186
Change-Id: I766655f94324002a8e67310e7882c1a63a706c5b
2018-07-24Bug 537330 - Breakpoints View NPE on perspective switch during debugI20180727-2000Simeon Andreev1-0/+4
The error log indicates that at the time of BreakpointsView.contextActivated, the Breakpoints View had no tree model viewer. This caused a NPE to be logged in the Error Log. This change adds a guard to the method, similar to the guard found in VariablesView.contextActivated. This avoids the logged NPE and does not alter behavior otherwise; the rest of BreakpointsView.contextActivated does nothing if there is no tree model. Change-Id: I070a2714c40636fb8796a963b7a3d4c147c04bf9
2018-07-24Bug 537322 - Improved logging in exception case of flushMarkerCacheY20180725-2200I20180726-2000I20180725-2000I20180724-2000Simeon Andreev1-1/+5
Whenever an exception occurs during marker attribute setting in BreakpointWorkingSetCache.flushMarkerCache, the logged error does not include information for the attributes which are to be set on the marker. When logged e.g. from tests, its difficult to understand where the logged error is coming from. This change adds some extra information to the logged error. Change-Id: I0b9a1aadb9310b26778f200df4004cdbb8756eef Signed-off-by: Simeon Andreev <simeon.danailov.andreev@gmail.com>
2018-07-20Bug 465666: Fix bread crumb background in dark themeY20180723-0635Y20180723-0615I20180723-2000I20180722-2000I20180721-1500I20180720-2000Matthias Becker4-127/+21
Change-Id: Ifb6349ffd0af011933760e6ee8eef3f984209b8e Signed-off-by: Matthias Becker <ma.becker@sap.com>
2018-07-19Bug 521038 - [refactoring] Replace anonymous Runnable classes by lambdaAlexander Kurtakov51-1577/+1189
expression Changed some formatters to Eclipse so there is some formatting happening as the debug one is not clear where to get from and it's unrealistic to expect random contributors to do it. Change-Id: I731410854f9a4ac0c9fc18bcf3ad3dcf7d941c9a Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-07-15Bug 535151 - Show only launch types with children in export dialogY20180718-2200I20180718-2000I20180718-0130I20180717-2000I20180717-0320I20180716-2000I20180716-0715I20180715-2000Michael Keppler1-5/+26
Since a typical Eclipse user will have a lot of empty launch config types (i.e. with no defined launch configs or prototypes), which we can hide from the export launch config dialog. Had to move some private members from the nested class to the top level class to use them in both. Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de> Also-by: Karsten Thoms <karsten.thoms@itemis.de> Change-Id: I04a0e52b5c9d4427e5221591b33ea0aaf67488e7 Bug:535151
2018-07-14Bug 535466 - Menus in export launch dialog should have ellipsisMichael Keppler1-2/+2
Change-Id: Id1fffba4c3976e7f3efbc0dba8468a6fd0382f51 Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de> Bug:535466
2018-07-13Bug 536648 - Thread evaluation runs in parallel with thread label updateI20180714-1500I20180713-2000Simeon Andreev1-1/+1
When running an evaluation on a thread suspension, its possible that a label update of the Debug View runs in parallel. The label update can then retrieve the intermediate state of the evaluation. If no further label update is issued, the thread will be shown as running until the Debug View is refreshed. With this change, suspending a thread after an evaluation will also update the thread state in the Debug View. This avoids the wrong running icon of the thread. Change-Id: Id6ec6a9d48c041d0b2617adb8b7da91ba5140b15 Signed-off-by: Simeon Andreev <simeon.danailov.andreev@gmail.com>
2018-06-20Bug 534298 - [quick fix] The New PreferenceY20180705-0105Y20180628-0525I20180704-2000I20180704-0805I20180704-0605I20180704-0545I20180703-2000I20180702-2000I20180702-0435I20180701-2000I20180630-1500I20180629-2000I20180629-0420I20180629-0400I20180628-2000I20180628-0230I20180627-2020I20180627-2000I20180625-1545I20180621-2000Kalyan Prasad Tatavarthi1-1/+15
"PREF_COMPLIANCE_COMPATIBLE_JRE_NOT_AVAILABLE" creates a new problem marker But does not provide Quick Fix for the problem marker Change-Id: Ie9e1697dc4f83c6e8251d8b68c254cf964e95262 Signed-off-by: Kalyan Prasad Tatavarthi <kalyan_prasad@in.ibm.com>
2018-06-14Bug 535686 - use one queue for scheduling async tasksI20180618-0800I20180615-0655I20180615-0300I20180614-2045I20180614-2000Andrey Loskutov1-9/+22
Queue async tasks coming from ChildrenUpdate's jobs via scheduleViewerUpdate() in the same queue with async tasks coming from modelChanged(), so that they all run in more predictable order. This allows us to run async UI updates triggered by ElementContentProvider jobs before we run next batch of model change events. Additionally this should further reduce the number of Display.asyncExec() tasks coming from debugger. Change-Id: I4a16e9cc79ab453b35d064c85440b726a4772442 Signed-off-by: Andrey Loskutov <loskutov@gmx.de>
2018-06-14Bug 535686 - wait for pending ElementContentProvider jobsAndrey Loskutov1-1/+8
Before triggering another update round via doModelChanged(), wait for all previously scheduled ElementContentProvider jobs, so that they can read the model data before it changes again. Change-Id: I2534d140c295341f3a69793254b6f94d5e7ef04e Signed-off-by: Andrey Loskutov <loskutov@gmx.de>
2018-06-12Bug 535468 - Link with prototype does not set an initial selectionI20180613-0440I20180613-0300Michael Keppler1-0/+1
Set an initial selection in the dialog to indicate what is currently the linked prototype. Change-Id: I9414b8f020b7b80de95464db35cdc4d2481a94c3 Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de> Bug:535468
2018-06-12Bug 535809 - ElementContentProvider jobs have no familyAndrey Loskutov1-15/+18
Added family to all jobs and cleaned up the code a bit. Change-Id: I777599bce7f347a07d0d2c7051fdd695204a9e46 Signed-off-by: Andrey Loskutov <loskutov@gmx.de>
2018-06-12Bug 534681 - Remove redundant modifiers in org.eclipse.debug.uiKarsten Thoms74-638/+631
Change-Id: I1d0b999ad8ead1bad88e9f23aa32fd34418c63aa Signed-off-by: Karsten Thoms <karsten.thoms@itemis.de>
2018-06-08LaunchView cleanupAndrey Loskutov1-113/+6
Change-Id: Ic8081419430d04e55d593810e3ba4e21655eb8dc Signed-off-by: Andrey Loskutov <loskutov@gmx.de>
2018-06-08Bug 535707 - AssertionFailedException via LaunchView.showBreadcrumbPageAndrey Loskutov1-2/+7
DebugContextEvent expects a non-null context object. Change-Id: I6268e8cda92c619c2b900812be762ca86360c972 Signed-off-by: Andrey Loskutov <loskutov@gmx.de>
2018-06-08Bug 535462 - NPE when closing second window during debugSimeon Andreev1-1/+18
The bug occurs while debugging, having two windows and a custom debug context provider. When closing the second window, DebugWindowContextService.notify is called with the selection of the custom debug context provider. If this selection is e.g. a stack frame, SourceLookupService asks SourceLookupFacility to open a source file. This throws a NPE, since the workbench page window is closing. With this change SourceLookupFacility checks if the active workbench window is closing. If so, the source is not opened, avoiding the NPE. Note: updated platform.ui version range to see re-export updated ui.workbench API. Change-Id: Ifb177c1211248b0cda17c6d14201a210d5fec8dc Signed-off-by: Simeon Andreev <simeon.danailov.andreev@gmail.com> Signed-off-by: Andrey Loskutov <loskutov@gmx.de>
2018-06-07TreeModelContentProvider cleanupI20180607-2000Andrey Loskutov1-58/+37
Change-Id: I6e3b6faece08d4e4e52cd4a4c60d9c5b6ed09521 Signed-off-by: Andrey Loskutov <loskutov@gmx.de>
2018-06-07ModelDelta cleanupAndrey Loskutov1-34/+8
Change-Id: I962d7c4043e4f50104119c6059cf368da2eb51ea Signed-off-by: Andrey Loskutov <loskutov@gmx.de>
2018-06-01Bug 535148 - Export launch wizard confirmation dialog typoMichael Keppler1-1/+1
Add missing "to". Change-Id: I5f5ebd168a56d8fbe9bf21dc54c07fe957712236 Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de> Bug: 535148
2018-05-14Bug 517809 - Modes in Launches tab of Launch Group use mode namesS4_8_0_RC2S4_8_0_RC1I20180528-0800I20180527-2250I20180527-2000I20180527-0800I20180526-1500I20180526-0800I20180525-2000I20180524-0900I20180523-2000I20180523-0800I20180522-2000I20180522-0800I20180521-2000I20180521-0800I20180520-2000I20180520-0800I20180519-1500I20180519-0800I20180518-2000I20180516-2000I20180516-0800I20180515-2000I20180515-1755I20180515-1620I20180515-1440I20180515-1300I20180515-0950I20180515-0800I20180514-2000I20180514-0800Sarika Sinha2-70/+104
instead of labels Change-Id: I2e62b93ee3810b06f75278db08f423f08bf60b2d
2018-05-09Bug 534380 - Reset button is not working in New Configuration ->S4_8_0_M7I20180511-2000I20180509-2000Sarika Sinha1-1/+1
Prototype tab Change-Id: Ia43c51525afe9f6a1f0984abd8e01f3cf5545a57
2018-05-09Bug 533788 - Use StandardCharsetsI20180509-0800Alexander Kurtakov1-2/+3
Another bunch of such changes. Change-Id: I4012772607b8f7c165156b8d03fbd5bcbee4b581 Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-05-07Bug 534379 - GVT48: English Strings are found in New Prototype ->I20180508-2000I20180508-0800I20180507-2205I20180507-2000I20180507-0810I20180507-0800Sarika Sinha2-1/+7
Prototype tab Change-Id: I1b47534b83d9927090b0b55646bccee0618c85ba
2018-05-03Bug 532602 - Remove all Breakpoints dialog should use verbsI20180506-2000I20180506-0800I20180505-1500I20180504-2000I20180503-2000Lars Vogel1-1/+1
Change-Id: I66a7035fa798e0f33cf808ed3c02a37dca121774 Signed-off-by: Lars Vogel <Lars.Vogel@vogella.com>
2018-05-03Removes Non-Javadoc from RemoveAllBreakpointsActionLars Vogel1-29/+0
Change-Id: I64de6922580c1c0c0ec2606f24abce9e00a873f5 Signed-off-by: Lars Vogel <Lars.Vogel@vogella.com>
2018-05-03Bug 532602 - Remove all Breakpoints dialog should use verbsI20180503-0740Jonas Hungershausen3-18/+31
Change-Id: I9bdd8349a4116ca034cf78a2d2cc2a65289ec000 Signed-off-by: Jonas Hungershausen <jonas.hungershausen@vogella.com>
2018-04-26CopyrightI20180502-2000I20180502-0100I20180501-2000I20180430-2000I20180430-0715I20180429-2000I20180428-2020I20180428-1500I20180427-2000I20180426-2000I20180426-0500Sarika Sinha1-1/+1
Change-Id: I28a8c91e5eafc61156b3f59b50852fcebc57dafd
2018-04-25Bug 534015 - Remove Expression Icon Enabled even when an expression isKarthik Bhat1-1/+1
not added Delete Expression icon is enabled in debug session as soon as we click to add an expression. It should be enabled after an expression has been added and selected. Change-Id: Iec79c29965490fce911392598faf8c2935654c35 Signed-off-by: Karthik Bhat <kv.bhat@samsung.com>
2018-04-24Fix typos in launch prototypesI20180425-2000I20180425-1705I20180425-1650I20180425-1455I20180425-0910I20180425-0725I20180425-0505I20180425-0250I20180424-2000Michael Keppler2-5/+3
Also removed one completely unused message in that area. Change-Id: Ieac6caa0baaabc3dfabe89cbaa8cf61091be1eba Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de>
2018-04-18Bug 533751 - Direct use of IStructuredSelectionAlexander Kurtakov16-340/+52
Instead of getSelection and cast. Remove useless @see comments in touched files too. Change-Id: I60d49967e9e56ccaf047817a1ac25d9add2075da Signed-off-by: Alexander Kurtakov <akurtako@redhat.com>
2018-04-11Bug 532580 - Sort launchers in debug launcher selection dialogI20180412-2000I20180411-2000I20180411-0735I20180411-0530Michael Keppler1-7/+12
Sort the available launchers by name. That makes the dialog more consistent with the same workspace preferences and is easier to read for the user. Change-Id: I844eaaa5b8b99513f7b8259c2be69571689aed33 Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de> Bug:532580
2018-03-29Minor - Removes unneccesary non-Javadoc and @see from VariablesViewI20180410-2000I20180409-2000I20180408-2000I20180407-1500I20180406-2000I20180405-2000I20180404-2000Lars Vogel1-125/+1
Change-Id: I66b865f38dbd60f683360e83525a9b0482789023 Signed-off-by: Lars Vogel <Lars.Vogel@vogella.com>
2018-03-18Bug 532578 - Missing title when exporting launch configurationsMichael Keppler1-0/+1
When import or export wizards are used outside the general import/export wizard, then they need to be initialized. Change-Id: I698948437e8e2212a94cd43297c5d5281c80e23e Signed-off-by: Michael Keppler <Michael.Keppler@gmx.de> Bug: 532578
2018-03-09Bug 532211 - SWTException on launch tab changeI20180318-2000I20180317-1500I20180316-2000I20180316-0740I20180316-0730I20180316-0525I20180316-0510I20180316-0455I20180315-2000I20180314-2000I20180313-2000I20180312-2000I20180311-2000I20180310-1500I20180309-2000Brian de Alwis1-1/+1
Change-Id: I3e6128ad00119bffd68234afd73a11f6b6b8cfc2
2018-02-28Bug 531211 - Menu labels for configuration/prototype are wrongS4_8_0_M6I20180308-0630I20180307-2000I20180307-1025I20180307-0900I20180307-0825I20180307-0800I20180306-2000I20180306-0800I20180305-2000I20180305-0800I20180305-0300I20180304-2000I20180304-0800I20180303-1500I20180303-1200I20180303-0800I20180303-0335I20180303-0155I20180303-0055I20180302-2000I20180301-2000I20180228-2000Sarika Sinha1-14/+14
Change-Id: I80d7f97795db6830ac9a68cc33affe5b669bc7a3

Back to the top