Skip to content
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

Fix Memory DIsplay (Closes #706) #728

Closed
wants to merge 3 commits into from
Closed

Fix Memory DIsplay (Closes #706) #728

wants to merge 3 commits into from

Conversation

PythonSmall-Q
Copy link
Member

What does this PR aim to accomplish?:

Fix Memory DIsplay (Closes #706)


By submitting this pull request, I confirm the following:

  1. I have read and understood the contributors guide, as well as this entire template. I understand which branch to base my commits and Pull Requests against.
  2. I have commented my proposed changes within the code and I have tested my changes.
  3. I am willing to help maintain this change if there are issues with it later.
  4. It is compatible with the GNU General Public License v3.0
  5. I have squashed any insignificant commits. (git rebase)
  6. I have checked that another pull request for this purpose does not exist.
  7. I have considered, and confirmed that this submission will be valuable to others.
  8. I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
  9. I give this submission freely, and claim no ownership to its content.

  • I have read the above and my PR is ready for review. Check this box to confirm

Signed-off-by: Shan Wenxiao <seanoj_noreply@yeah.net>
@PythonSmall-Q PythonSmall-Q requested a review from boomzero August 31, 2024 10:33
@hendragon-bot hendragon-bot bot added the user-script This issue or pull request is related to the main user script label Aug 31, 2024
@PythonSmall-Q PythonSmall-Q linked an issue Aug 31, 2024 that may be closed by this pull request
2 tasks
@boomzero
Copy link
Member

这样内存显示会炸

@boomzero boomzero closed this Aug 31, 2024
Copy link
Contributor

Qodana for JS

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

@boomzero
Copy link
Member

因为内存和代码长度单位不一样,但是都用这个

@boomzero
Copy link
Member

If it is really this easy, I would have fixed this.

@boomzero
Copy link
Member

I should point out that this is still easy, so carry on.

@boomzero
Copy link
Member

But currently you are undoing one of my previous commits.

@boomzero
Copy link
Member

316b0da

@boomzero boomzero deleted the #706 branch September 3, 2024 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S user-script This issue or pull request is related to the main user script
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] 代码长度单位换算错误
2 participants