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

Coding - ASCII BRep double reading using stream #214

Closed
wants to merge 1 commit into from

Conversation

dpasukhi
Copy link
Member

@dpasukhi dpasukhi commented Dec 27, 2024

Simplify GetReal function by removing unnecessary buffer handling
_
Tempolary changes, if will be performance benefits - will rewrite on using stream directly on place

Simplify GetReal function by removing unnecessary buffer handling
@dpasukhi dpasukhi added 2. Enhancement New feature or request 1. Coding Coding rules, trivial changes and misprints labels Dec 27, 2024
@dpasukhi dpasukhi self-assigned this Dec 27, 2024
@dpasukhi
Copy link
Member Author

No benefits, no regressions.
As a result, we can move to stream directly. It can be done later ( as no benefits )

@dpasukhi dpasukhi closed this Dec 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. Coding Coding rules, trivial changes and misprints 2. Enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant