Sign in Join

Live chat for Free Pascal, Lazarus, Delphi and every Pascal programmer.

Mailing lists / lazarus / 2003-07

Patch: SetCursor AV (win32)

8 messages · 4 people ·

Reply by email Subscribe to this list Your subscription

Read-only archive. Use j and k to move between messages.

Mattias Gaertner
to Mattias Gaertner: On Wed, 2 Jul 2003 17:07:37 +0200 ok Mattias
Show quoted text 33 lines
Vincent Snijders wrote: > On Wed, 2 Jul 2003 18:18:47 +0200 (W. Europe Daylight Time) > Michael Van Canneyt wrote: > >> >> >> On Wed, 2 Jul 2003, Mattias Gaertner wrote: >> >>> On Wed, 2 Jul 2003 17:07:37 +0200 >>> "Micha Nelissen" wrote: >>> >>>> Hi, >>>> >>>> Attached is a patch to fix AV's when TControl.SetCursor is called. >>>> A >>>> lookup map was needed to translate between TCursor and win32 IDC_* >>>> constants. It would be nice if those were defined as constants >>>> instead of >>>> functions, therefore I have included them as constants. >>> >>> ok >> >> Despite all these patches, all examples crash with a double exception >> message on my machine :/ >> >> Michael. > > Attached is a patch to solve this AV. It was due to too much (or not > enough) clean up in the critical section code. Hmm, it seems my patch (all.patch) was not applied? Mattias, can you check that? Getting a lot of conflicts here... I think I made the same changes as you, but you commented them, and I removed them.

Sorry. I hope this correct now.

Mattias

Reply by email

Micha Nelissen
to Mattias Gaertner: On Wed, 2 Jul 2003 17:07:37 +0200 ok Mattias
Vincent Snijders wrote 28 lines
On Wed, 2 Jul 2003 18:18:47 +0200 (W. Europe Daylight Time) Michael Van Canneyt wrote: > > > On Wed, 2 Jul 2003, Mattias Gaertner wrote: > >> On Wed, 2 Jul 2003 17:07:37 +0200 >> "Micha Nelissen" wrote: >> >>> Hi, >>> >>> Attached is a patch to fix AV's when TControl.SetCursor is called. >>> A >>> lookup map was needed to translate between TCursor and win32 IDC_* >>> constants. It would be nice if those were defined as constants >>> instead of >>> functions, therefore I have included them as constants. >> >> ok > > Despite all these patches, all examples crash with a double exception > message on my machine :/ > > Michael. Attached is a patch to solve this AV. It was due to too much (or not enough) clean up in the critical section code.

Hmm, it seems my patch (all.patch) was not applied? Mattias, can you check that? Getting a lot of conflicts here... I think I made the same changes as you, but you commented them, and I removed them.

Regards,

Micha.

Reply by email

Vincent Snijders
to Mattias Gaertner: On Wed, 2 Jul 2003 17:07:37 +0200 ok Mattias
Show quoted text 20 lines
On Wed, 2 Jul 2003, Mattias Gaertner wrote: > On Wed, 2 Jul 2003 17:07:37 +0200 > "Micha Nelissen" wrote: > > > Hi, > > > > Attached is a patch to fix AV's when TControl.SetCursor is called. A > > lookup map was needed to translate between TCursor and win32 IDC_* > > constants. It would be nice if those were defined as constants instead of > > functions, therefore I have included them as constants. > > ok Despite all these patches, all examples crash with a double exception message on my machine :/ Michael.

Attached is a patch to solve this AV. It was due to too much (or not enough) clean up in the critical section code.

-It also does some clean up.
-Moves EndPaint to the correct alphabetical position.

Regards,
Vincent.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: gz00026.gz
Type: application/octet-stream
Size: 1490 bytes
Desc: "critsect.patch.gz"
Url : http://localhost/pipermail/lazarus/attachments/20030702/d64cb471/gz00026.obj

Reply by email

Michael Van Canneyt
to Mattias Gaertner: On Wed, 2 Jul 2003 17:07:37 +0200 ok Mattias
Mattias Gaertner wrote 11 lines
On Wed, 2 Jul 2003 17:07:37 +0200 "Micha Nelissen" wrote: > Hi, > > Attached is a patch to fix AV's when TControl.SetCursor is called. A > lookup map was needed to translate between TCursor and win32 IDC_* > constants. It would be nice if those were defined as constants instead of > functions, therefore I have included them as constants. ok

Despite all these patches, all examples crash with a double exception message on my machine :/

Michael.

Reply by email

Micha Nelissen

Hi,

Attached is a patch to fix AV's when TControl.SetCursor is called. A lookup map was needed to translate between TCursor and win32 IDC_* constants. It would be nice if those were defined as constants instead of functions, therefore I have included them as constants.

Regards,

Micha.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: bin00048.bin
Type: application/octet-stream
Size: 2743 bytes
Desc: "cursor.patch"
Url : http://localhost/pipermail/lazarus/attachments/20030702/21c72f19/bin00048.bin

Reply by email

Mattias Gaertner
to Mattias Gaertner: On Wed, 2 Jul 2003 17:07:37 +0200 ok Mattias
Show quoted text 29 lines
On Wed, 2 Jul 2003 18:18:47 +0200 (W. Europe Daylight Time) Michael Van Canneyt wrote: > > > On Wed, 2 Jul 2003, Mattias Gaertner wrote: > > > On Wed, 2 Jul 2003 17:07:37 +0200 > > "Micha Nelissen" wrote: > > > > > Hi, > > > > > > Attached is a patch to fix AV's when TControl.SetCursor is called. A > > > lookup map was needed to translate between TCursor and win32 IDC_* > > > constants. It would be nice if those were defined as constants > > > instead of functions, therefore I have included them as constants. > > > > ok > > Despite all these patches, all examples crash with a double exception > message on my machine :/ > > Michael. Attached is a patch to solve this AV. It was due to too much (or not enough) clean up in the critical section code. -It also does some clean up. -Moves EndPaint to the correct alphabetical position.

Applied. Thanks.

Mattias

Reply by email

Micha Nelissen
to Michael Van Canneyt: Despite all these patches, all examples crash with a double exception message on my…
Michael Van Canneyt wrote 16 lines
On Wed, 2 Jul 2003, Mattias Gaertner wrote: > On Wed, 2 Jul 2003 17:07:37 +0200 > "Micha Nelissen" wrote: > >> Hi, >> >> Attached is a patch to fix AV's when TControl.SetCursor is called. A >> lookup map was needed to translate between TCursor and win32 IDC_* >> constants. It would be nice if those were defined as constants >> instead of functions, therefore I have included them as constants. > > ok Despite all these patches, all examples crash with a double exception message on my machine :/

:(. Can you give a stack trace or other debugging info? I am using FPC 1.0.8 release, if that helps.

Regards,

Micha.

Reply by email