Skip to content

[C++ification] Glue and utilities code changes - #2611

Merged
jonpryor merged 1 commit into
dotnet:masterfrom
grendello:cppify+2
Jan 15, 2019
Merged

[C++ification] Glue and utilities code changes#2611
jonpryor merged 1 commit into
dotnet:masterfrom
grendello:cppify+2

Conversation

@grendello

@grendellogrendello commented Jan 11, 2019

Copy link
Copy Markdown
Contributor
  • Use new/delete/delete[] more
  • Avoid calling the printf family of functions in favor of new, template, string_concat in the Utils class. When sprintf is used to concatenate just strings we can do better by not calling the C varargs functions and generate very efficient code.
  • Replace C strdup calls with our, based on new, strdup implementation

@grendellogrendello added do-not-merge PR should not be merged. full-mono-integration-build For PRs; run a full build (~6-10h for mono bumps), not the faster PR subset (~2h for mono bumps) labels Jan 11, 2019
@grendello
grendelloforce-pushed the cppify+2 branch 5 times, most recently from 9fbb8eb to d450659CompareJanuary 14, 2019 11:41
 * Use `new/delete/delete[]` more
* Avoid calling the `printf` family of functions in favor of new, template,
`string_concat` in the `Utils` class. When `sprintf` is used to concatenate
just strings we can do better by not calling the C varargs functions and
generate very efficient code.
* Replace C `strdup` calls with our, based on `new`, strdup implementation
@grendellogrendello removed the do-not-merge PR should not be merged. label Jan 14, 2019
@jonpryor
jonpryor merged commit 42656af into dotnet:masterJan 15, 2019
@grendello
grendello deleted the cppify+2 branch January 15, 2019 19:07
@github-actionsgithub-actionsBot locked and limited conversation to collaborators Feb 1, 2024
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

full-mono-integration-buildFor PRs; run a full build (~6-10h for mono bumps), not the faster PR subset (~2h for mono bumps)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@grendello@jonpryor