Skip to content

afanasy: delete copy and move operators of af::Msg#626

Open
grische wants to merge 1 commit into
CGRU:masterfrom
grische:delete-copy-move-operators
Open

afanasy: delete copy and move operators of af::Msg#626
grische wants to merge 1 commit into
CGRU:masterfrom
grische:delete-copy-move-operators

Conversation

@grische

@grische grische commented Jun 11, 2025

Copy link
Copy Markdown
Contributor

While debugging, I accidentally used the copy assignment in a test which lead me down completely wrong rabbit hole.
Deleting them explicitely will throw compile errors in case they are being used somewhere.

avoids accidental usage of the operators
@timurhai

Copy link
Copy Markdown
Member

We use only those constructors and operators, that a implemented.
Debugging such a complex program is not a good approach.
The best way it to catch a bug.
ps
If there are any errors in such mature classes it should not work at all.
And it works in many companies 24/7.
You just faced some job that cause special bug.

@grische

grische commented Jun 18, 2025

Copy link
Copy Markdown
Contributor Author

This has nothing to do with any bug, it's just code improvements that make it easier to work on the cgru code.

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.

2 participants