Forums WoW Modding Support Archives Azerothcore Discord Archives [DiscordArchive] but after std::move the content of „recvPacket“ should be invalid, right?

[DiscordArchive] but after std::move the content of „recvPacket“ should be invalid, right?

[DiscordArchive] but after std::move the content of „recvPacket“ should be invalid, right?

Pages (2): 1 2 Next
rektbyfaith
Administrator
0
01-07-2025, 06:16 AM
#1
Archived author: sudlud • Posted: 2025-01-07T06:16:09.957000+00:00
Original source

but after std::move the content of „recvPacket“ should be invalid, right?
rektbyfaith
01-07-2025, 06:16 AM #1

Archived author: sudlud • Posted: 2025-01-07T06:16:09.957000+00:00
Original source

but after std::move the content of „recvPacket“ should be invalid, right?

rektbyfaith
Administrator
0
01-07-2025, 06:18 AM
#2
Archived author: Takenbacon • Posted: 2025-01-07T06:18:04.904000+00:00
Original source

In 99.9% of data structures yes but this one has a constructor that seems to handle it fairly gracefully. I would say it's wrong from a design point of view as it's implying the data is moved and thus the lingering reference is no longer valid.
rektbyfaith
01-07-2025, 06:18 AM #2

Archived author: Takenbacon • Posted: 2025-01-07T06:18:04.904000+00:00
Original source

In 99.9% of data structures yes but this one has a constructor that seems to handle it fairly gracefully. I would say it's wrong from a design point of view as it's implying the data is moved and thus the lingering reference is no longer valid.

rektbyfaith
Administrator
0
01-07-2025, 06:18 AM
#3
Archived author: Takenbacon • Posted: 2025-01-07T06:18:29.420000+00:00
Original source

But it does "work"
rektbyfaith
01-07-2025, 06:18 AM #3

Archived author: Takenbacon • Posted: 2025-01-07T06:18:29.420000+00:00
Original source

But it does "work"

rektbyfaith
Administrator
0
01-07-2025, 08:27 AM
#4
Archived author: Deleted User • Posted: 2025-01-07T08:27:32.653000+00:00
Original source

Probably best to just setup to git actions
rektbyfaith
01-07-2025, 08:27 AM #4

Archived author: Deleted User • Posted: 2025-01-07T08:27:32.653000+00:00
Original source

Probably best to just setup to git actions

rektbyfaith
Administrator
0
01-07-2025, 08:28 AM
#5
Archived author: Deleted User • Posted: 2025-01-07T08:28:00.263000+00:00
Original source

But yes, I hadn’t noticed
rektbyfaith
01-07-2025, 08:28 AM #5

Archived author: Deleted User • Posted: 2025-01-07T08:28:00.263000+00:00
Original source

But yes, I hadn’t noticed

rektbyfaith
Administrator
0
01-07-2025, 08:28 AM
#6
Archived author: Takenbacon • Posted: 2025-01-07T08:28:34.349000+00:00
Original source

lol I've been trying to get asan just to run locally for about half an hour now, it's a disaster
rektbyfaith
01-07-2025, 08:28 AM #6

Archived author: Takenbacon • Posted: 2025-01-07T08:28:34.349000+00:00
Original source

lol I've been trying to get asan just to run locally for about half an hour now, it's a disaster

rektbyfaith
Administrator
0
01-07-2025, 08:28 AM
#7
Archived author: Takenbacon • Posted: 2025-01-07T08:28:36.360000+00:00
Original source

do. not. try.
rektbyfaith
01-07-2025, 08:28 AM #7

Archived author: Takenbacon • Posted: 2025-01-07T08:28:36.360000+00:00
Original source

do. not. try.

rektbyfaith
Administrator
0
01-07-2025, 08:28 AM
#8
Archived author: Takenbacon • Posted: 2025-01-07T08:28:43.073000+00:00
Original source

there's all sorts of shit fucked
rektbyfaith
01-07-2025, 08:28 AM #8

Archived author: Takenbacon • Posted: 2025-01-07T08:28:43.073000+00:00
Original source

there's all sorts of shit fucked

rektbyfaith
Administrator
0
01-07-2025, 08:30 AM
#9
Archived author: Deleted User • Posted: 2025-01-07T08:30:28.770000+00:00
Original source

Yes, I done an issue here: https://github.com/azerothcore/azerothco...sues/19670 and my last comment was a paste of covering scan. It take me 7h xD
[Embed: Feature: Improve the Dev, CLI and the compile workflow · Issue #196...]
Describe your feature request or suggestion in detail Since I started using AC about a week ago, I've noticed, as some of you have, a few problems with the workflow. It's not unique to AC s...
https://github.com/azerothcore/azerothco...sues/19670
rektbyfaith
01-07-2025, 08:30 AM #9

Archived author: Deleted User • Posted: 2025-01-07T08:30:28.770000+00:00
Original source

Yes, I done an issue here: https://github.com/azerothcore/azerothco...sues/19670 and my last comment was a paste of covering scan. It take me 7h xD
[Embed: Feature: Improve the Dev, CLI and the compile workflow · Issue #196...]
Describe your feature request or suggestion in detail Since I started using AC about a week ago, I've noticed, as some of you have, a few problems with the workflow. It's not unique to AC s...
https://github.com/azerothcore/azerothco...sues/19670

rektbyfaith
Administrator
0
01-07-2025, 08:31 AM
#10
Archived author: Deleted User • Posted: 2025-01-07T08:31:00.235000+00:00
Original source

I’m looking to fix few things when I’ve some times
rektbyfaith
01-07-2025, 08:31 AM #10

Archived author: Deleted User • Posted: 2025-01-07T08:31:00.235000+00:00
Original source

I’m looking to fix few things when I’ve some times

Pages (2): 1 2 Next
Recently Browsing
 1 Guest(s)
Recently Browsing
 1 Guest(s)