Bug 13349 – [Mago] Wrong breakpoint locations in v0.3.39-beta2

Status
RESOLVED
Resolution
FIXED
Severity
regression
Priority
P1
Component
visuald
Product
D
Version
D2
Platform
x86
OS
Windows
Creation time
2014-08-21T14:53:00Z
Last change time
2014-10-02T19:37:11Z
Assigned to
nobody
Creator
verylonglogin.reg

Attachments

IDFilenameSummaryContent-TypeSize
1440MagoTest.7zTestcaseapplication/x-7z-compressed1726
1443Mago-Bug-13349.7zTestcase with incorrect breakpoint lineapplication/x-7z-compressed2118

Comments

Comment #0 by verylonglogin.reg — 2014-08-21T14:53:36Z
When debugging in v0.3.39-beta2 with Mago it sometimes fills breakpoint lines background with red. Sometimes half of module lines background is filled when setting a single breakpoint (before and after the breakpoint) and on reaching the breakpoint execution cursor is set to the first red line, next step moves it to the correct position. Previous beta works fine.
Comment #1 by r.sagitario — 2014-08-23T12:10:25Z
I remember having seen something similar, but I cannot reproduce now. Is this Win32 or Win64? Can you provide an example? BTW: there have been two updates to beta2 without a new beta (because Visual D itself did not change, only mago). Maybe you are using an early version (last update was August, 8).
Comment #2 by verylonglogin.reg — 2014-08-23T12:30:15Z
The problem is on 32-bin Windows XP. Installer used is identical to the current version. Will try to segregate reduced testcase from codebase.
Comment #3 by r.sagitario — 2014-09-24T07:16:23Z
Did this happen to you again? I think this is currently the biggest blocker for a release, which I'd like to get out of the door soon.
Comment #4 by verylonglogin.reg — 2014-09-24T09:13:27Z
(In reply to Rainer Schuetze from comment #3) > Did this happen to you again? I think this is currently the biggest blocker > for a release, which I'd like to get out of the door soon. v0.3.39-beta3 has the same issue. Sorry for not reducing the testcase, will try to do it in this week.
Comment #5 by verylonglogin.reg — 2014-09-29T16:04:01Z
Created attachment 1440 Testcase
Comment #6 by r.sagitario — 2014-09-29T18:16:15Z
(In reply to Denis Shelomovskij from comment #5) > Testcase Thanks. That's a pretty reduced tetcase ;-) I suspected it would have to do with templates... I'll investigate.
Comment #7 by r.sagitario — 2014-10-02T06:56:39Z
Comment #8 by verylonglogin.reg — 2014-10-02T11:04:48Z
(In reply to Rainer Schuetze from comment #7) > Please try > https://github.com/D-Programming-Language/visuald/releases/tag/v0.3.39-rc1, > the issue should be gone now. As I wrote: > Sometimes half of module lines background is filled when setting > a single breakpoint (before and after the breakpoint) So the breakpoint line is still sometimes located before actual breakpoint. Sorry, it wasn't shown in the testcase. Will create a new one.
Comment #9 by verylonglogin.reg — 2014-10-02T11:35:27Z
Created attachment 1443 Testcase with incorrect breakpoint line
Comment #10 by r.sagitario — 2014-10-02T19:24:25Z
(In reply to Denis Shelomovskij from comment #8) > As I wrote: > > Sometimes half of module lines background is filled when setting > > a single breakpoint (before and after the breakpoint) Sorry, I wasn't reading carefully enough, I guess. > > So the breakpoint line is still sometimes located before actual breakpoint. > Sorry, it wasn't shown in the testcase. Will create a new one. Thanks for the new test case, please try the fix for it here: https://github.com/D-Programming-Language/visuald/releases/tag/v0.3.39-rc2
Comment #11 by verylonglogin.reg — 2014-10-02T19:37:11Z
(In reply to Rainer Schuetze from comment #10) > <snip> > Thanks for the new test case, please try the fix for it here: > https://github.com/D-Programming-Language/visuald/releases/tag/v0.3.39-rc2 Thanks, it works now. )