Skip to content

Manage CCharEntity::PJobPoints with unique_ptr#7690

Merged
zach2good merged 1 commit intoLandSandBoat:basefrom
AJBats:PJobPoints_unique_ptr_2
Jul 11, 2025
Merged

Manage CCharEntity::PJobPoints with unique_ptr#7690
zach2good merged 1 commit intoLandSandBoat:basefrom
AJBats:PJobPoints_unique_ptr_2

Conversation

@AJBats
Copy link
Copy Markdown
Contributor

@AJBats AJBats commented Jul 11, 2025

I affirm:

  • I understand that if I do not agree to the following points by completing the checkboxes my PR will be ignored.
  • I understand I should leave resolving conversations to the LandSandBoat team so that reviewers won't miss what was said.
  • I have read and understood the Contributing Guide and the Code of Conduct.
  • I have tested my code and the things my code has changed since the last commit in the PR and will test after any later commits.

What does this pull request do?

(Second attempt, first attempt was #7683) PJobPoints: Contributes to #5171 by converting PJobPoints raw pointer to unique_ptr.

Steps to test these changes

Test1: Gave a test character job points and added JP abilities.
Test2: Logged in and out of the server several times.

PJobPoints: Contributes to LandSandBoat#5171 by converting PJobPoints raw pointer to unique_ptr.
@AJBats AJBats marked this pull request as ready for review July 11, 2025 01:00
@zach2good zach2good merged commit 41d4b7d into LandSandBoat:base Jul 11, 2025
14 checks passed
@AJBats AJBats deleted the PJobPoints_unique_ptr_2 branch July 13, 2025 16:20
claybie pushed a commit to claybie/LeafSpringCog that referenced this pull request Aug 11, 2025
Manage CCharEntity::PJobPoints with unique_ptr
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