Skip to content

[Trivial] Placate clang warning string-plus-int#947

Merged
codeofalltrades merged 1 commit intoVeil-Project:masterfrom
Zannick:clang1
May 26, 2021
Merged

[Trivial] Placate clang warning string-plus-int#947
codeofalltrades merged 1 commit intoVeil-Project:masterfrom
Zannick:clang1

Conversation

@Zannick
Copy link
Collaborator

@Zannick Zannick commented May 18, 2021

Replace eg "tv"+2 with the suggested syntax of &"tv"[2].

Testing Results

Built veild and run with gcc, clang.

Replace eg `"tv"+2` with the suggested syntax of `&"tv"[2]`.
@WetOne
Copy link
Collaborator

WetOne commented May 19, 2021

utACK 219a159

Copy link
Collaborator

@CaveSpectre11 CaveSpectre11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK 219a159

Copy link
Collaborator

@codeofalltrades codeofalltrades left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 219a159

@codeofalltrades codeofalltrades added the Tag: Waiting For QA A pull review is waiting for QA to test the pull request label May 23, 2021
@codeofalltrades codeofalltrades merged commit 9840f3c into Veil-Project:master May 26, 2021
@Zannick Zannick deleted the clang1 branch October 22, 2021 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Tag: Build System Tag: Waiting For QA A pull review is waiting for QA to test the pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants