Skip to content
This repository was archived by the owner on Aug 31, 2021. It is now read-only.

[Bug 19620] Update cursor when entering window#5452

Merged
livecodepanos merged 1 commit into
livecode:develop-8.1from
montegoulding:bugfix-19620
May 12, 2017
Merged

[Bug 19620] Update cursor when entering window#5452
livecodepanos merged 1 commit into
livecode:develop-8.1from
montegoulding:bugfix-19620

Conversation

@montegoulding
Copy link
Copy Markdown
Contributor

This patch fixes a regression caused when MCmousestackptr was
changed to an object handle in commit 654bd57. In the diff
below if (!MCmousestackptr) should be if (MCmousestackptr):

     MCmousestackptr = MCdispatcher->findstackd(dw);
-    if (MCmousestackptr != NULL)
+    if (!MCmousestackptr)
     {
         MCmousestackptr->resetcursor(True);

This patch fixes a regression caused when `MCmousestackptr` was
changed to an object handle in commit 654bd57. In the diff
below `if (!MCmousestackptr)` should be `if (MCmousestackptr)`:

         MCmousestackptr = MCdispatcher->findstackd(dw);
    -    if (MCmousestackptr != NULL)
    +    if (!MCmousestackptr)
         {
             MCmousestackptr->resetcursor(True);
@montegoulding montegoulding added this to the 8.1.5-rc-1 milestone May 12, 2017
@mention-bot
Copy link
Copy Markdown

@montegoulding, thanks for your PR! By analyzing the history of the files in this pull request, we identified @livecodeali and @livecodefraser to be potential reviewers.

@runrevmark
Copy link
Copy Markdown
Contributor

@livecode-vulcan review ok 73fa92a

@livecode-vulcan
Copy link
Copy Markdown
Contributor

💙 review by @runrevmark ok 73fa92a

livecode-vulcan added a commit that referenced this pull request May 12, 2017
[Bug 19620] Update cursor when entering window

This patch fixes a regression caused when `MCmousestackptr` was
changed to an object handle in commit 654bd57. In the diff
below `if (!MCmousestackptr)` should be `if (MCmousestackptr)`:

         MCmousestackptr = MCdispatcher->findstackd(dw);
    -    if (MCmousestackptr != NULL)
    +    if (!MCmousestackptr)
         {
             MCmousestackptr->resetcursor(True);
@livecode-vulcan
Copy link
Copy Markdown
Contributor

😎 test success 73fa92a

  • try-community-armv6-android-api8: success
  • try-community-armv6-android-api9: success
  • try-community-js-emscripten-sdk1.35: success
  • try-community-universal-ios-iphoneos10.3: success
  • try-community-universal-ios-iphonesimulator10.3: success
  • try-community-universal-mac-macosx10.6: success
  • try-community-universal-mac-macosx10.9: success
  • try-community-x86-linux-debian7: success
  • try-community-x86-linux-debian8: success
  • try-community-x86_64-linux-debian7: success
  • try-community-x86_64-linux-debian8: success
  • try-community-x86-win32: success
  • try-community-x86_64-win32: success

@livecodepanos livecodepanos merged commit 5fbaa96 into livecode:develop-8.1 May 12, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants