-
Notifications
You must be signed in to change notification settings - Fork 118
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
roapi一些错误处理函数 #142
roapi一些错误处理函数 #142
Conversation
才看到有个谜之SDK6可链接性的测试,难道前面说的是这个么?我看半天也没看明白这个是怎么一回事,抱歉了。 |
if (auto pRoFailFastWithErrorContext = try_get_RoFailFastWithErrorContext()) | ||
{ | ||
pRoFailFastWithErrorContext(hrError); | ||
} |
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语义上,RoFailFastWithErrorContext调用后不应该继续执行……
能解释一下原因吗?
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.
没太懂,RoFailFastWithErrorContext不是这次你新增的吗?还是说抓个函数你有其他的参考来源?
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.
不好意思,眼花了。我记得好像参考了哪个来着,奇怪
} | ||
if ((error == S_OK) || (!languageException) || WindowsIsStringEmpty(message)) | ||
return FALSE; | ||
// The RoOriginateLanguageException function behaves like RoOriginateError but takes another parameter that stores extra information about the error. |
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.
Fallback逻辑为什么不调用RoOriginateError?
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.
他们都是win8的api,fallback也没用吧...
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.
他们都是win8的api,fallback也没用吧...
你的注释写着 RoOriginateLanguageException 8.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.
确实
话说大量扩充这类函数的目的是什么?有实际需要的应用的场景吗?我主要是担心Fallback逻辑错误。 |
有,不过可能不是很大就是,我在写一个要用winrt的程序,会链接到这些函数里,我希望它能正常在至少在win7上运行,但是因为有这些导入,导致运行不了。但实际上我根本不会运行到会触发这些错误的地方,我会自己进行hr的检查,但就是会链接到这些东西上就很烦了。如果您觉得这些意义不大的话,那我关掉就好了,其实我自己写一个链接进去也能顶掉默认的导入,但我觉得这样有点不舒服。 |
如果你有实际需要,大批量的提交PR是可以理解的。 |
60b6df8
to
9479234
Compare
b90ef53
into
Chuyu-Team:Fea/YY/PendingPR
- 添加 RoFailFastWithErrorContext - 添加 RoOriginateLanguageException - 添加 RoTransformError - 添加 RoTransformErrorW - 添加 RoGetAgileReference
No description provided.