Skip to content

Remove outdated limitation for GetPrivateProfileSection #1929

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

Merged
merged 2 commits into from
Jan 22, 2025

Conversation

justanotheranonymoususer
Copy link
Contributor

@justanotheranonymoususer justanotheranonymoususer commented Nov 5, 2024

Tested back to Windows 7, there's on such limitation. It was probably relevant in very old versions.

BTW not sure why there's the three files - A, W and just getprivateprofilesection.md. I assume the third file can be removed.

BTW there's a similar limitation for GetProfileSection, I didn't check whether it's still relevant, and it's a useless function nowadays anyway.

Copy link

@justanotheranonymoususer : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@justanotheranonymoususer justanotheranonymoususer changed the title patch-remove-getprivateprofilesection-limt Remove outdated limitation for GetPrivateProfileSection Nov 5, 2024
Copy link
Contributor

@riverar riverar left a comment

Choose a reason for hiding this comment

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

Rather than delete important information (for older OSes), we could instead preface it. For example: Prior to Windows 7, the maximum profile section size is 32,767 characters.

@justanotheranonymoususer
Copy link
Contributor Author

@justanotheranonymoususer
Copy link
Contributor Author

Updated, hopefully better now

@GrantMeStrength GrantMeStrength merged commit 21bfbd5 into MicrosoftDocs:docs Jan 22, 2025
1 check passed
@justanotheranonymoususer justanotheranonymoususer deleted the patch-13 branch January 23, 2025 09:17
@justanotheranonymoususer
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants