Skip to content

Fix Seaside continuation debugging with native code enabled and GemTools#80

Merged
dalehenrich merged 3 commits intoGsDevKit:gs_masterfrom
marianopeck:gs_master
Oct 19, 2015
Merged

Fix Seaside continuation debugging with native code enabled and GemTools#80
dalehenrich merged 3 commits intoGsDevKit:gs_masterfrom
marianopeck:gs_master

Conversation

@marianopeck
Copy link
Copy Markdown
Contributor

There was an scenario in which it was not possible to debug Seaside continuations from the object log. Most likely, this was when native code was enabled and when trying to debug from GemTools. The issue was related to the fact of using Breakpoints. After a large discussion with Dale, we fixed this by using a regular "self halt" rather than breakpoints.

There was an scenario in which it was not possible to debug Seaside
continuations from the object log. Most likely, this was when native
code was enabled and when trying to debug from GemTools. The issue was
related to the fact of using Breakpoints. After a large discussion with
Dale, we fixed this by using a regular "self halt" rather than
breakpoints.
Sorry, wrong committed file
There was an scenario in which it was not possible to debug Seaside
continuations from the object log. Most likely, this was when native
code was enabled and when trying to debug from GemTools. The issue was
related to the fact of using Breakpoints. After a large discussion with
Dale, we fixed this by using a regular "self halt" rather than
breakpoints.
@marianopeck marianopeck added this to the 3.1.4.2 milestone Oct 15, 2015
dalehenrich added a commit that referenced this pull request Oct 19, 2015
Fix Seaside continuation debugging with native code enabled and GemTools
@dalehenrich dalehenrich merged commit 55f1bac into GsDevKit:gs_master Oct 19, 2015
@marianopeck
Copy link
Copy Markdown
Contributor Author

Cool. Thanks Dale. One less override from my app and one little more shared back :)

@dalehenrich
Copy link
Copy Markdown
Member

:)

On 10/18/15 6:25 PM, marianopeck wrote:

Cool. Thanks Dale. One less override from my app and one little more
shared back :)


Reply to this email directly or view it on GitHub
#80 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants