- Pharo-dev - lists.pharo.org

[Pharo-project] Issue 3262 in pharo: Don't cache the debugger's debuggerMap, it's not safe.
by pharo＠googlecode.com Nov. 13, 2010

Nov. 13, 2010

Status: Accepted Owner: stephane.ducasse Labels: Milestone-1.2 New issue 3262 by stephane.ducasse: Don't cache the debugger's debuggerMap, it's not safe. http://code.google.com/p/pharo/issues/detail?id=3262 http://lists.squeakfoundation.org/pipermail/packages/2010-August/003602.html Name: Tools-eem.258 Ancestors: Tools-eem.257 Don't cache the debugger's debuggerMap, its not safe. Add preference for and support of opening a debugger for an error in a background process when the UI process is busy. ==================================================== Eliot Miranda uploaded a new version of Tools to project The Trunk: http://source.squeak.org/trunk/Tools-eem.258.mcz ==================== Summary ==================== Name: Tools-eem.258 Author: eem Time: 14 August 2010, 6:45:20.412 pm UUID: 261ba269-1565-45b1-9f7b-b6ce8cee312c Ancestors: Tools-eem.257 Don't cache the debugger's debuggerMap, its not safe. Add preference for and support of opening a debugger for an error in a background process when the UI process is busy. =============== Diff against Tools-eem.257 =============== Item was added: + ----- Method: Debugger class>>interruptUIProcessIfBlockedOnErrorInBackgroundProcess: (in category 'preferences') ----- + interruptUIProcessIfBlockedOnErrorInBackgroundProcess: aBoolean + InterruptUIProcessIfBlockedOnErrorInBackgroundProcess := aBoolean! Item was changed: CodeHolder subclass: #Debugger + instanceVariableNames: 'interruptedProcess interruptedController contextStack contextStackTop contextStackIndex contextStackList receiverInspector contextVariablesInspector externalInterrupt proceedValue selectingPC savedCursor isolationHead failedProject errorWasInUIProcess labelString message' + classVariableNames: 'ContextStackKeystrokes ErrorRecursion InterruptUIProcessIfBlockedOnErrorInBackgroundProcess' - instanceVariableNames: 'interruptedProcess interruptedController contextStack contextStackTop contextStackIndex contextStackList receiverInspector contextVariablesInspector externalInterrupt proceedValue selectingPC debuggerMap savedCursor errorWasInUIProcess labelString message' - classVariableNames: 'ContextStackKeystrokes ErrorRecursion' poolDictionaries: '' category: 'Tools-Debugger'! !Debugger commentStamp: '<historical>' prior: 0! I represent the machine state at the time of an interrupted process. I also represent a query path into the state of the process. The debugger is typically viewed through a window that views the stack of suspended contexts, the code for, and execution point in, the currently selected message, and inspectors on both the receiver of the currently selected message, and the variables in the current context. Special note on recursive errors: Some errors affect Squeak's ability to present a debugger. This is normally an unrecoverable situation. However, if such an error occurs in an isolation layer, Squeak will attempt to exit from the isolation layer and then present a debugger. Here is the chain of events in such a recovery. * A recursive error is detected. * The current project is queried for an isolationHead * Changes in the isolationHead are revoked * The parent project of isolated project is returned to * The debugger is opened there and execution resumes. If the user closes that debugger, execution continues in the outer project and layer. If, after repairing some damage, the user proceeds from the debugger, then the isolationHead is re-invoked, the failed project is re-entered, and execution resumes in that world. ! Item was added: + ----- Method: Debugger class>>ifPreferredInterruptUIProcessIfBlocked: (in category 'private') ----- + ifPreferredInterruptUIProcessIfBlocked: errorWasInUIProcess + | sema | + errorWasInUIProcess ifTrue: + [^self]. + self interruptUIProcessIfBlockedOnErrorInBackgroundProcess ifFalse: + [^self]. + "Only interrupt the UI if it is unresponsive (and so is doing something that needs + interrupting). Test using addDeferredUIMessage: to see if it is running UI activities." + sema := Semaphore new. + Project current addDeferredUIMessage: [sema signal]. + (sema waitTimeoutMSecs: 100) ifTrue: + [[Project current interruptName: 'Interrupt from Background Error'] fork] + + "| s | + s := Semaphore new. + [self assert: 0 > 1000. s signal] fork. + s wait"! Item was changed: ----- Method: Debugger class>>openOn:context:label:contents:fullView: (in category 'opening') ----- openOn: process context: context label: title contents: contentsStringOrNil fullView: bool "Open a notifier in response to an error, halt, or notify. A notifier view just shows a short view of the sender stack and provides a menu that lets the user open a full debugger." + | controller errorWasInUIProcess block | - | controller errorWasInUIProcess | Smalltalk isMorphic ifTrue: [errorWasInUIProcess := Project spawnNewProcessIfThisIsUI: process] ifFalse: [controller := ScheduledControllers activeControllerProcess == process ifTrue: [ScheduledControllers activeController]]. + block := [ - WorldState addDeferredUIMessage: [ [ | debugger | debugger := self new process: process controller: controller context: context. Smalltalk isMorphic ifTrue: ["schedule debugger in deferred UI message to address redraw problems after opening a debugger e.g. from the testrunner." "WorldState addDeferredUIMessage: ["bool ifTrue: [debugger openFullNoSuspendLabel: title] ifFalse: [debugger openNotifierContents: contentsStringOrNil label: title]] ifFalse: ["deferred UI message would require special controller in MVC" bool ifTrue: [debugger openFullNoSuspendLabel: title] ifFalse: [debugger openNotifierContents: contentsStringOrNil label: title]]. debugger errorWasInUIProcess: errorWasInUIProcess. Preferences logDebuggerStackToFile ifTrue: [ Smalltalk logError: title inContext: context to: 'SqueakDebug.log']. Smalltalk isMorphic ifFalse: [ScheduledControllers searchForActiveController "needed since openNoTerminate (see debugger #open...) does not set up activeControllerProcess if activeProcess (this fork) is not the current activeControllerProcess (see #scheduled:from:)"]. ] on: Error do: [:ex | self primitiveError: 'Orginal error: ', title asString, '. Debugger error: ', ([ex description] on: Error do: ['a ', ex class printString]), ':' ] ]. + self ifPreferredInterruptUIProcessIfBlocked: errorWasInUIProcess. + Project current addDeferredUIMessage: block. + process suspend! - process suspend. - ! Item was changed: ----- Method: Debugger>>debuggerMap (in category 'accessing') ----- debuggerMap + ^self selectedContext debuggerMap! - ^debuggerMap ifNil: - [debuggerMap := self selectedContext debuggerMap].! Item was added: + ----- Method: Debugger class>>interruptUIProcessIfBlockedOnErrorInBackgroundProcess (in category 'preferences') ----- + interruptUIProcessIfBlockedOnErrorInBackgroundProcess + <preference: 'Interrupt UI process on background error' + category: 'debug' + description: 'When enabled, the debugger will interrupt the UI process if an error occurs in a background process and the UI process is blocked.' + type: #Boolean> + ^InterruptUIProcessIfBlockedOnErrorInBackgroundProcess ifNil: [false]! ====================================================

1 1

Re: [Pharo-project] Issue 2818 in pharo: Eliot and friends compiler fixes
by pharo＠googlecode.com Nov. 13, 2010

Nov. 13, 2010

Comment #3 on issue 2818 by stephane.ducasse: Eliot and friends compiler fixes http://code.google.com/p/pharo/issues/detail?id=2818 Split into parts FIrst issue done in Issue 3259

1 2

[Pharo-project] Issue 3260 in pharo: Compiler
by pharo＠googlecode.com Nov. 13, 2010

Nov. 13, 2010

Status: Accepted Owner: stephane.ducasse New issue 3260 by stephane.ducasse: Compiler http://code.google.com/p/pharo/issues/detail?id=3260 If you fill an issue for the first time, please read "How to report bugs" at http://www.pharo-project.org/community/issue-tracking Pharo image: <core, dev or web> Pharo core version: <copy from World/System/About> Virtual machine used: <ex: pharo-vm-0.15.2f-linux> Class browser used if applicable: <ex: O2PackageBrowserAdaptor. If you don't know, print "SystemBrowser default"> Steps to reproduce: 1. 2. 3. Paste or attach stack trace if applicable (look at the file PharoDebug.log located in the same directory as your image):

1 0

[Pharo-project] Issue 3224 in pharo: Migrating MethodReference to MethodReferenceWithSource
by pharo＠googlecode.com Nov. 13, 2010

Nov. 13, 2010

Status: Accepted Owner: stephane.ducasse CC: Benjamin.VanRyseghem, johanbri...@me.com New issue 3224 by stephane.ducasse: Migrating MethodReference to MethodReferenceWithSource http://code.google.com/p/pharo/issues/detail?id=3224 The attached cses should be tested over multiple scenarios. This is not clear that we did not miss a point

1 3

[Pharo-project] [update 1.2] #12239
by stephane ducasse Nov. 13, 2010

Nov. 13, 2010

12239 ----- - Issue 2793: Compiler fixes. Thanks Eliot Miranda. - Issue 2771: should add PharoTreatedInbox to the list of repositories. (apparently did not work). Stef

1 0

Re: [Pharo-project] Issue 2771 in pharo: should add PharoTreatedInbox to the list of repositories
by pharo＠googlecode.com Nov. 13, 2010

Nov. 13, 2010

Updates: Status: Fixed Cc: marcus.denker Comment #4 on issue 2771 by stephane.ducasse: should add PharoTreatedInbox to the list of repositories http://code.google.com/p/pharo/issues/detail?id=2771 just perform cleanRepositories during the next integration phase.

1 1

Re: [Pharo-project] Issue 2793 in pharo: Compiler fixes
by pharo＠googlecode.com Nov. 13, 2010

Nov. 13, 2010

Updates: Status: Fixed Comment #1 on issue 2793 by stephane.ducasse: Compiler fixes http://code.google.com/p/pharo/issues/detail?id=2793 in slice-2793-CompilerFixes in inbox

1 1

Nov. 13, 2010

12238 ----- - Update Regex to sync with latest version of lukas. http://source.lukas-renggli.ch/unsorted/ now up to 40 Stef

1 0

Re: [Pharo-project] Issue 3205 in pharo: monticello's error on Linux
by pharo＠googlecode.com Nov. 13, 2010

Nov. 13, 2010

Comment #1 on issue 3205 by elendilo: monticello's error on Linux http://code.google.com/p/pharo/issues/detail?id=3205 I have taken a look at the pharo.sh script in the one-click image distribution. It helped me to substitute "-encoding utf8" instead of the default "-encoding latin1" and it works.

1 0

Nov. 13, 2010

12237 ----- - Issue 3087: optimize hasBindingThatBeginsWith:. Tx Levente, Mariano and Guillermo.

1 0