andy/floor-limit #29

Merged
jake merged 3 commits from andy/floor-limit into master 2 years ago
andy commented 2 years ago
Collaborator
There is no content yet.
andy added 2 commits 2 years ago
eff9fba20a very important packet data
andy added 1 commit 2 years ago
2dd1161ae7 rename to match the rest of the code
Owner

why is C3 changed here? its being used already and works fine?

why is C3 changed here? its being used already and works fine?
Poster
Collaborator

why is C3 changed here? its being used already and works fine?

The struct missed the amount being dropped if you dropped a partial stack of an item

> why is C3 changed here? its being used already and works fine? The struct missed the amount being dropped if you dropped a partial stack of an item
Owner

thats covered in the followup message 0x29, is that duplicated in both packets?

also if that packet had that extra u32 and we weren't parsing it, it would error out about mismatched pkt length.

thats covered in the followup message 0x29, is that duplicated in both packets? also if that packet had that extra u32 and we weren't parsing it, it would error out about mismatched pkt length.
Poster
Collaborator

sega duplicated it.

[recv buf] [32, 0, 96, 0, 0, 0, 0, 0, 195, 6, 0, 0, 0, 0, 0, 0, 206, 254, 94, 67, 184, 255, 125, 67, 16, 0, 1, 0, 4, 0, 0, 0]
modify: epoll_fd=4, fd=6, ev=Event { key: 18446744073709551615, readable: true, writable: false }
[recv buf] [20, 0, 96, 0, 0, 0, 0, 0, 41, 3, 0, 0, 16, 0, 1, 0, 4, 0, 0, 0, 0, 0, 0, 0]
[recv buf] [12, 0, 96, 0, 0, 0, 0, 0, 45, 1, 0, 0, 0, 0, 0, 0]
main_loop: sleeping for 75 us
[recv from ClientId(1)] Message(
    Message {
        msg [GameMessage]: DropCoordinates(
            DropCoordinates {
                client [u8]: 0,
                target [u8]: 0,
                map_area [u16]: 0,
                room [u16]: 0,
                x [f32]: 222.99533,
                z [f32]: 253.9989,
                item_id [u32]: 65552,
                amount [u32]: 4,
            },
        ),
    },
)
[recv from ClientId(1)] Message(
    Message {
        msg [GameMessage]: PlayerNoLongerHasItem(
            PlayerNoLongerHasItem {
                client [u8]: 0,
                target [u8]: 0,
                item_id [u32]: 65552,
                amount [u32]: 4,
            },
        ),
    },
)
sega duplicated it. ``` [recv buf] [32, 0, 96, 0, 0, 0, 0, 0, 195, 6, 0, 0, 0, 0, 0, 0, 206, 254, 94, 67, 184, 255, 125, 67, 16, 0, 1, 0, 4, 0, 0, 0] modify: epoll_fd=4, fd=6, ev=Event { key: 18446744073709551615, readable: true, writable: false } [recv buf] [20, 0, 96, 0, 0, 0, 0, 0, 41, 3, 0, 0, 16, 0, 1, 0, 4, 0, 0, 0, 0, 0, 0, 0] [recv buf] [12, 0, 96, 0, 0, 0, 0, 0, 45, 1, 0, 0, 0, 0, 0, 0] main_loop: sleeping for 75 us [recv from ClientId(1)] Message( Message { msg [GameMessage]: DropCoordinates( DropCoordinates { client [u8]: 0, target [u8]: 0, map_area [u16]: 0, room [u16]: 0, x [f32]: 222.99533, z [f32]: 253.9989, item_id [u32]: 65552, amount [u32]: 4, }, ), }, ) [recv from ClientId(1)] Message( Message { msg [GameMessage]: PlayerNoLongerHasItem( PlayerNoLongerHasItem { client [u8]: 0, target [u8]: 0, item_id [u32]: 65552, amount [u32]: 4, }, ), }, ) ```
jake merged commit e1a8ad7fc5 into master 2 years ago
jake deleted branch andy/floor-limit 2 years ago
continuous-integration/drone/push Build is passing
continuous-integration/drone/pr Build is passing
The pull request has been merged as e1a8ad7fc5.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.