-
Notifications
You must be signed in to change notification settings - Fork 115
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
Spelling #704
base: master
Are you sure you want to change the base?
Spelling #704
Conversation
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
Signed-off-by: Josh Soref <[email protected]>
@@ -909,7 +909,7 @@ MI_Char *restrictedQualifier[] = { | |||
ZT("Version"), | |||
ZT("ClassVersion") | |||
}; | |||
MI_Boolean CanQualifierBePropogated( _In_ MI_Qualifier *qualifier) | |||
MI_Boolean CanQualifierBePropagated( _In_ MI_Qualifier *qualifier) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a change to an identifier, it looks local, and is thus unlikely to be particularly exciting.
@@ -132,9 +132,9 @@ OI_EVENT("MuxIn_Open: cannot allocate ConnectionIn") | |||
void trace_MuxInOpen_AllocFailed(); | |||
|
|||
OI_EVENT("_StrandEntryOperation_Add: Canceled %d entries %p(%s): %p(%s)") | |||
void trace_StrandEntryOperation_AddCanceled(unsigned int numEntries, StrandMany * self, const char * selfTrand, Strand * entry, const char * entryStrandName); | |||
void trace_StrandEntryOperation_AddCanceled(unsigned int numEntries, StrandMany * self, const char * selfStrand, Strand * entry, const char * entryStrandName); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm guessing based on the rest of the content of the line. (Google Sheets made most of the corrections, but it didn't have an opinion here, so this was manual intervention.)
@@ -615,13 +615,13 @@ void trace_InstanceConversionFailed(const TChar * name, MI_Result r); | |||
OI_EVENT("invalid query expression: %T") | |||
void trace_InvalidQueryExpression(const TChar * filter); | |||
OI_EVENT("library unload did not call post result") | |||
void trace_LibraryUnload_DidnotPostResult(); | |||
void trace_LibraryUnload_DidNotPostResult(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a change to an API of sorts. Happy to drop if it's a problem.
@@ -1105,7 +1105,7 @@ void trace_DispHandleInteractionRequest(void * self, Interaction * interaction, | |||
OI_EVENT("Disp_HandleInteractionRequest: self (%p), interaction(%p), Unsupported msg(%p:%d:%T:%x)") | |||
void trace_DispUnsupportedMessage(void * self, Interaction * interaction, Message * msg, MI_Uint32 msgTag, const TChar * messageName, MI_Uint64 operationId); | |||
OI_EVENT("Disp_HandleRequest") | |||
void trace_DispHandlRequest(); | |||
void trace_DispHandleRequest(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While the previous one is stylistic, this one matches the name of the event on the preceding line. And it feels like a more problematic typo.
@@ -1415,7 +1415,7 @@ void trace_InitIndicationWithNullInput(); | |||
OI_EVENT("Multiple indication initialization of provider for class %T") | |||
void trace_MultipleIndication_InitOfProviderForClass(const TChar * className); | |||
|
|||
OI_EVENT("_Provider_InvokeSubscribe: Start Thread %x: provider (%p), msg (%p) with tag (%d), subcription (%p)") | |||
OI_EVENT("_Provider_InvokeSubscribe: Start Thread %x: provider (%p), msg (%p) with tag (%d), subscription (%p)") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
debug output change?
@@ -9,7 +9,7 @@ SET TESTDIR=%BINPATH%\tests | |||
SET LOGDIR=%CD% | |||
SET LOGFILENAME=%LOGDIR%\test.log | |||
|
|||
ECHO Current Direcotry: %LOGDIR% >%LOGFILENAME% 2>&1 | |||
ECHO Current Directory: %LOGDIR% >%LOGFILENAME% 2>&1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
user facing...
MI_Uint32 totalResponses; | ||
|
||
/* lower 16 bits is aninxed in self->enumerateContexts, upper 16 bits are random data (for validation) */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
???
@@ -4925,7 +4925,7 @@ static int _ValidateSubscribeRequest( | |||
selfCD, | |||
NULL, | |||
WSBUF_FAULT_INTERNAL_ERROR, | |||
ZT("mandatory parameters (className, namesapce) are not provided for subscribe request")); | |||
ZT("mandatory parameters (className, namespace) are not provided for subscribe request")); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
user facing?
@@ -8,17 +8,17 @@ | |||
#define ID_MI_DES_XML_INST_TOO_MANY_CLASSES 4002 | |||
#define ID_MI_DES_XML_INST_CANNOT_FIND_CLASS 4003 | |||
#define ID_MI_DES_XML_CLASS_DERIVATION_LIST_WRONG 4004 | |||
#define ID_MI_DES_XML_CLASS_SUPERCLASS_PARENT_MISSMATCH 4005 | |||
#define ID_MI_DES_XML_CLASS_SUPERCLASS_PARENT_MISMATCH 4005 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
api?
@@ -27,17 +27,17 @@ | |||
#define ID_MI_DES_XML_INST_TOO_MANY_CLASSES PAL_T("Failed to deserialize CIM-XML document. The instance is too complex to deserialize because it uses too many classes.") | |||
#define ID_MI_DES_XML_INST_CANNOT_FIND_CLASS PAL_T("Failed to deserialize CIM-XML document. Could not find the class definition %T while deserializing instance.") | |||
#define ID_MI_DES_XML_CLASS_DERIVATION_LIST_WRONG PAL_T("Failed to deserialize CIM-XML document. A class definition has a SUPERCLASS attribute and a DERIVATION attribute. The SUPERCLASS attribute value should be the first item in the DERIVATION attribute list." ) | |||
#define ID_MI_DES_XML_CLASS_SUPERCLASS_PARENT_MISSMATCH PAL_T("Failed to deserialize CIM-XML document. While decoding a class definition there was a mismatch between the SUPERCLASS attribute and the passed in parent class definition." ) | |||
#define ID_MI_DES_XML_CLASS_SUPERCLASS_PARENT_MISMATCH PAL_T("Failed to deserialize CIM-XML document. While decoding a class definition there was a mismatch between the SUPERCLASS attribute and the passed in parent class definition." ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i try to avoid harming whitespace indentation, but I lost this -- I saw it in one pass, but the page reloaded and I couldn't find it
This PR corrects misspellings identified by the check-spelling action.
The misspellings have been reported at jsoref@9ac24b5#commitcomment-57009226
The action reports that the changes in this PR would make it happy: jsoref@4bed8ca
Note: this PR does not include the action. If you're interested in running a spell check on every PR and push, that can be offered separately.
Disclaimer: GitHub does not like PRs w/ >250 commits (this has 465 at the time of filing). In general, reviewing windowed views is probably better than most alternatives (the standard git cli's diff output is fairly unhelpful for PRs like this).
Note: I fully expect to be asked to split this PR / reorganize it. If there are files that shouldn't be changed, let me know, dropping files is easy. If there are terms that can't be changed, that's also easy -- right now, a term is localized to a single commit, so dropping it means just dropping the commit.
It's possible for me to split by kind of content (comment vs code), but such things are moderately expensive and I try to do it as few times as possible and as late as possible.
I'll try to include annotations about the changes, but it'll take me a bit (it's a long PR)