Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

fix: remove memory leak invector.c#1302

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
vil02 wants to merge1 commit intoTheAlgorithms:master
base:master
Choose a base branch
Loading
fromvil02:memory_leak_in_vector_patch

Conversation

@vil02
Copy link
Member

@vil02vil02 commentedSep 12, 2023•
edited
Loading

Description of Change

There is a memory leak invector.c. This PR fixes it (and removes sometrailing spaces). TheAwesome CI Workflowpasses on my end.

References

Checklist

  • Added description of change
  • PR title follows semanticcommit guidelines
  • Search previous suggestions before making a new one, as yours may be a duplicate.
  • I acknowledge that all my contributions will be made under the project's license.

Notes:
Removes memory leak invector.c.

@vil02vil02 marked this pull request as ready for reviewSeptember 12, 2023 17:24
@vil02vil02force-pushed thememory_leak_in_vector_patch branch fromd182608 toecd6993CompareOctober 2, 2023 11:52
@github-actions
Copy link
Contributor

This pull request has been automatically marked as abandoned because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@vil02
Copy link
MemberAuthor

@Panquesito7,@tjgurwara99,@alexpantyukhin could some of you please add adont-close orapproved label?

@tjgurwara99tjgurwara99 added the dont-closeThis issue/pull request shouldn't be closed labelNov 27, 2023
@vil02
Copy link
MemberAuthor

Happy birthday#1302! I wish you that you will get lots of accepting reviews and that you will be merged some day!

tjgurwara99 reacted with laugh emoji

@Artursrossi
Copy link

@vil02 It's incredible that this PR is from 2023 and still hasn't been approved. Isn't it possible that only 1 of the code reviewers to approve this ? Because there is a big stagnation in the C repository, compared to other languages.
And another thing, could you give your opinion on a PR I opened recently? (#1463)

vil02 reacted with laugh emoji

@vil02
Copy link
MemberAuthor

Dear#1302,

today is your birthday again! On this special occasion I wish you:

  • lots of likes,
  • plenty of positive comments,
  • approving reviews,
  • and that you will get merged some day!

In short: happy birthday!

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@tjgurwara99tjgurwara99tjgurwara99 approved these changes

@realstealthninjarealstealthninjarealstealthninja approved these changes

@Panquesito7Panquesito7Awaiting requested review from Panquesito7Panquesito7 is a code owner

@alexpantyukhinalexpantyukhinAwaiting requested review from alexpantyukhinalexpantyukhin is a code owner

Assignees

No one assigned

Labels

dont-closeThis issue/pull request shouldn't be closed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@vil02@Artursrossi@tjgurwara99@realstealthninja
[8]ページ先頭
©2009-2025 Movatter.jp