Skip to content

Handle ctrl+map click packet (GM teleport)#4239

Merged
1 commit merged intootland:masterfrom
Unreal-TFS:map_click
Nov 9, 2022
Merged

Handle ctrl+map click packet (GM teleport)#4239
1 commit merged intootland:masterfrom
Unreal-TFS:map_click

Conversation

@MillhioreBT
Copy link
Copy Markdown
Contributor

Pull Request Prelude

  • I have followed [proper The Forgotten Server code styling][code].
  • I have read and understood the [contribution guidelines][cont] before making this PR.
  • I am aware that this PR may be closed if the above-mentioned criteria are not fulfilled.

Changes Proposed

MapClick!

Issues addressed: Nothing!

@ghost ghost requested review from DSpeichert, nekiro, ramon-bernardo and ranisalt November 7, 2022 05:53
Copy link
Copy Markdown
Contributor

@ramon-bernardo ramon-bernardo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not? 🆗

@ranisalt
Copy link
Copy Markdown
Member

ranisalt commented Nov 8, 2022

Cool. Would be a perfect packet for handling in Lua 😭

@ranisalt ranisalt changed the title MapClick Handle ctrl+map click packet (GM teleport) Nov 8, 2022
@ghost ghost merged commit b35bd82 into otland:master Nov 9, 2022
@Erza
Copy link
Copy Markdown
Contributor

Erza commented Nov 14, 2022

Cool. Would be a perfect packet for handling in Lua 😭

Do you have a system in mind for forwarding packets to be handled in Lua? The issue there would be that it has to go across 2 threads, potentially needing a copy of the packet

@ranisalt
Copy link
Copy Markdown
Member

ranisalt commented Dec 6, 2022

Do you have a system in mind for forwarding packets to be handled in Lua? The issue there would be that it has to go across 2 threads, potentially needing a copy of the packet

That is why I kept the copy in #4043

This pull request was closed.
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.

4 participants