Page MenuHomePhorge

Set parent of KazvFile and KazvSaveFile, instead of QSharedPointer
AbandonedPublic

Authored by nannanko on Aug 24 2024, 7:13 AM.

Details

Reviewers
tusooa
Group Reviewers
O1: the Kazv Project
Summary

KazvFile and KazvSaveFile are only used within their job. Setting parent is simpler and less buggy than using QSharedPointer.

Type: skip

Test Plan

N/A

Diff Detail

Repository
rK kazv
Branch
nannanko/stacked
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 281
Build 550: GitLab CI for kazv
Build 549: arc lint + arc unit

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Aug 24 2024, 7:29 AM
Harbormaster failed remote builds in B281: Diff 468!
tusooa requested changes to this revision.Aug 27 2024, 7:05 PM

How is it "less buggy"? What bug does it fix?

This revision now requires changes to proceed.Aug 27 2024, 7:05 PM
In D178#3268, @tusooa wrote:

How is it "less buggy"? What bug does it fix?

Actually nothing. It's just that I think setting QObject parent is simpler than using a smart pointer. If you don’t like this change, forget it, because I haven’t found any related BUG for the time being.

In D178#3283, @nannanko wrote:
In D178#3268, @tusooa wrote:

How is it "less buggy"? What bug does it fix?

Actually nothing. It's just that I think setting QObject parent is simpler than using a smart pointer. If you don’t like this change, forget it, because I haven’t found any related BUG for the time being.

I don't see how it's simpler. parenting and shared pointers provide different types of guarantees.