Skip to content

Profiler-IsEnabled - #1503

Merged
jywu-mysoft merged 1 commit into
microsoft:masterfrom
jthelin:Profiler-IsEnabled
Jul 30, 2019
Merged

Profiler-IsEnabled#1503
jywu-mysoft merged 1 commit into
microsoft:masterfrom
jthelin:Profiler-IsEnabled

Conversation

@jthelin

@jthelin jthelin commented Jul 25, 2019

Copy link
Copy Markdown
Contributor

Description: Avoid use of Hungarian naming convention for cross-platform API names.

Motivation and Context

I'm taking my cue here from the "ONNX Runtime coding conventions and standard" document which say we use the "Google C++ style guide", and that says "Do not use Hungarian notation"

https://github.com/microsoft/onnxruntime/blob/master/docs/Coding_Conventions_and_Standards.md

https://google.github.io/styleguide/cppguide.html#Windows_Code

X-ref: internal PR 4824

Avoid use of Hungarian naming convention for cross-platform API code.

I'm taking my cue here from the "ONNX Runtime coding conventions and standard" document which say we use the "Google C++ style guide", and that says "Do not use Hungarian notation"
https://github.com/microsoft/onnxruntime/blob/master/docs/Coding_Conventions_and_Standards.md
https://google.github.io/styleguide/cppguide.html#Windows_Code

X-ref: internal PR 4824
@jthelin
jthelin requested a review from a team as a code owner July 25, 2019 22:34
@hariharans29

Copy link
Copy Markdown
Member

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 22 pipeline(s).

@jywu-mysoft
jywu-mysoft merged commit fb7bdd1 into microsoft:master Jul 30, 2019
@jthelin
jthelin deleted the Profiler-IsEnabled branch July 30, 2019 21:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants