-
-
Notifications
You must be signed in to change notification settings - Fork 647
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: features OTCR natively in canary #3061
Draft
kokekanon
wants to merge
44
commits into
opentibiabr:main
Choose a base branch
from
kokekanon:kokekanon/all-feature-redemption
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from 7 commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
4c54d85
1/6 feature: Creature : AttachedEffect, Shader
kokekanon 745b7b8
2/7 feature: Map Shader
kokekanon ee26730
3/7 feature: TypingIcon
kokekanon 23e166a
review: mehah
kokekanon eae4112
4/7 feature: Item Shader
kokekanon 74ea55f
5/7 send disableFeature / enableFeature
kokekanon b09547e
review: dudantas
kokekanon 9484492
fix sonarcloud bot
kokekanon f490ce4
fix sonarcloud bot 2
kokekanon 1437b7e
fix: wrong review
dudantas 67777a9
Code format - (Clang-format)
github-actions[bot] 6766389
fix: itemShader old protocol otcr
kokekanon 7c45b97
clean variable
kokekanon bd1854c
fix return get shader
kokekanon 0ff915e
missing content player:getAttachedEffects
kokekanon f538950
simple test Talkations. delete when finished
kokekanon 0217e80
Lua code format - (Stylua)
github-actions[bot] 4b5fbad
Merge remote-tracking branch 'upstream/main' into kokekanon/all-featu…
kokekanon 9fe26c5
format
kokekanon 7956f08
Merge remote-tracking branch 'upstream/main' into kokekanon/all-featu…
kokekanon 8d6961d
improve: variable name
kokekanon 558b212
Merge branch 'kokekanon/all-feature-redemption' of https://github.com…
kokekanon 3744f16
game_outfit functional version, not cleaned or optimized
kokekanon 789354a
this requires review canary team .no break v8 and cipsoft
kokekanon 6a23e40
Lua code format - (Stylua)
github-actions[bot] 36da43e
fix: review sonarcloud ?
kokekanon 8e939e5
Code format - (Clang-format)
github-actions[bot] c551152
Merge remote-tracking branch 'upstream/main' into kokekanon/all-featu…
kokekanon 5b5dc84
Merge branch 'kokekanon/all-feature-redemption' of https://github.com…
kokekanon 9e763f7
simple test
kokekanon 300f92c
Lua code format - (Stylua)
github-actions[bot] 3bb868b
Merge remote-tracking branch 'upstream/main' into kokekanon/all-featu…
kokekanon ef754d0
Merge branch 'kokekanon/all-feature-redemption' of https://github.com…
kokekanon 6b27bb9
Merge remote-tracking branch 'upstream/main' into kokekanon/all-featu…
kokekanon 6097de3
Merge remote-tracking branch 'upstream/main' into kokekanon/all-featu…
kokekanon ec0bd43
murge
kokekanon e30fd35
fix: compilation windows
kokekanon 29919ed
Code format - (Clang-format)
github-actions[bot] 6ed239c
Merge remote-tracking branch 'upstream/main' into kokekanon/all-featu…
kokekanon 2c46d0a
Merge remote-tracking branch 'upstream/main' into kokekanon/all-featu…
kokekanon 687d0b2
Merge remote-tracking branch 'upstream/main' into kokekanon/all-featu…
kokekanon dce1886
Merge remote-tracking branch 'upstream/main' into kokekanon/all-featu…
kokekanon f44bf10
Merge remote-tracking branch 'upstream/main' into kokekanon/all-featu…
kokekanon 8350458
Merge remote-tracking branch 'upstream/main' into kokekanon/all-featu…
kokekanon File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -1262,6 +1262,18 @@ class Player final : public Creature, public Cylinder, public Bankable { | |
|
||
uint16_t getPlayerVocationEnum() const; | ||
|
||
void sendAttachedEffect(const std::shared_ptr<Creature> &creature, uint16_t effectId) const; | ||
void sendDetachEffect(const std::shared_ptr<Creature> &creature, uint16_t effectId) const; | ||
void sendShader(const std::shared_ptr<Creature> &creature, const std::string_view &shaderName) const; | ||
void sendMapShader(const std::string_view &shaderName) const; | ||
const std::string_view &getMapShader() const { | ||
return mapShader; | ||
} | ||
void setMapShader(const std::string_view &shaderName) { | ||
this->mapShader = shaderName; | ||
} | ||
void sendPlayerTyping(const std::shared_ptr<Creature> &creature, uint8_t typing) const; | ||
|
||
private: | ||
friend class PlayerLock; | ||
std::mutex mutex; | ||
|
@@ -1371,6 +1383,7 @@ class Player final : public Creature, public Cylinder, public Bankable { | |
std::string name; | ||
std::string guildNick; | ||
std::string loyaltyTitle; | ||
std::string_view mapShader; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Here, it will remain as std::string. std::string_view is used to avoid copying the string, but in the final context where it needs to be stored or modified, it must be a std::string. |
||
|
||
Skill skills[SKILL_LAST + 1]; | ||
LightInfo itemsLight; | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can remove the & as well; passing a std::string_view by reference actually increases overhead. std::string_view is intended to be passed by value since it is simply a lightweight view of the string.