Skip to content

first - #4

Open
LostPointer wants to merge 3 commits into
mainfrom
test_actions
Open

first#4
LostPointer wants to merge 3 commits into
mainfrom
test_actions

Conversation

@LostPointer

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

clang-tidy made some suggestions

Comment thread task_01/src/main.cpp

int main() { return 0; }
int main() {
int *p = new int[10];

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

warning: initializing non-owner 'int *' with a newly created 'gsl::owner<>' [cppcoreguidelines-owning-memory]

  int *p = new int[10];
  ^

Comment thread task_01/src/main.cpp

int main() { return 0; }
int main() {
int *p = new int[10];

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

warning: Value stored to 'p' during its initialization is never read [clang-analyzer-deadcode.DeadStores]

  int *p = new int[10];
       ^
Additional context

task_01/src/main.cpp:3: Value stored to 'p' during its initialization is never read

  int *p = new int[10];
       ^

Comment thread task_01/src/main.cpp
int main() { return 0; }
int main() {
int *p = new int[10];
return 0;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

warning: Potential leak of memory pointed to by 'p' [clang-analyzer-cplusplus.NewDeleteLeaks]

  return 0;
  ^
Additional context

task_01/src/main.cpp:3: Memory is allocated

  int *p = new int[10];
           ^

task_01/src/main.cpp:4: Potential leak of memory pointed to by 'p'

  return 0;
  ^

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.

1 participant