Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update packets up to 2019-12-11 #2585

Merged
merged 8 commits into from Dec 15, 2019
Merged

Conversation

4144
Copy link
Contributor

@4144 4144 commented Nov 23, 2019

Pull Request Prelude

  • I have followed [proper Hercules 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

Update packets table up to 2019-12-11.
Update shuffle packets up to 2019-12-11.
Update packets keys up to 2019-12-11.
Update packet ZC_NOTIFY_EFFECT3 to new version.
Update packet CZ_SE_CASHSHOP_OPEN to new version.
Add script function specialeffectnum.

@4144 4144 force-pushed the updatepackets branch 2 times, most recently from 9d6865d to cfb0a96 Compare December 5, 2019 17:53
@4144 4144 changed the title Update packets up to 2019-11-20 Update packets up to 2019-12-14 Dec 5, 2019
@4144 4144 force-pushed the updatepackets branch 2 times, most recently from bd912e1 to 909ebe1 Compare December 7, 2019 05:14
src/map/script.c Show resolved Hide resolved
doc/script_commands.txt Show resolved Hide resolved

Works same as specialeffect but also send effect numbers to client.
For PACKETVER >= 20191127 support two numbers (num1, num2).
For older packet versions only num1 supported.
Copy link
Member

Choose a reason for hiding this comment

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

Any info on what num2 affects?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

num2 is high dword of num. Script engine not support int64, because this here num and num2. Inside packet its one int32 or one int64 depend on packet version

@4144 4144 changed the title Update packets up to 2019-12-14 Update packets up to 2019-12-11 Dec 13, 2019
@MishimaHaruna MishimaHaruna added this to the Release v2019.12.15 milestone Dec 15, 2019
@MishimaHaruna MishimaHaruna merged commit 8c4a85d into HerculesWS:master Dec 15, 2019
@4144 4144 deleted the updatepackets branch December 18, 2019 04:02
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.

None yet

4 participants