Merge pull request #28 from sraboy/minor_fixes

Minor Fixes
mohabouje · Feb 12, 2018 · cfe59a9 · cfe59a9
2 parents 0a24843 + 023be57
commit cfe59a9
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/src/wintoastlib.cpp b/src/wintoastlib.cpp
@@ -637,7 +637,7 @@ INT64 WinToast::showToast(_In_ const WinToastTemplate& toast, _In_  IWinToastHan
     return FAILED(hr) ? -1 : id;
 }
 
-ComPtr<IToastNotifier> WinToast::notifier(bool* succeded) const  {
+ComPtr<IToastNotifier> WinToast::notifier(_In_ bool* succeded) const  {
 	ComPtr<IToastNotificationManagerStatics> notificationManager;
 	ComPtr<IToastNotifier> notifier;
 	HRESULT hr = DllImporter::Wrap_GetActivationFactory(WinToastStringWrapper(RuntimeClass_Windows_UI_Notifications_ToastNotificationManager).Get(), &notificationManager);
@@ -657,9 +657,9 @@ bool WinToast::hideToast(_In_ INT64 id) {
     const bool find = _buffer.find(id) != _buffer.end();
 	if (find) {
 		bool succeded = false;
-		ComPtr<IToastNotifier> not = notifier(&succeded);
+		ComPtr<IToastNotifier> notify = notifier(&succeded);
 		if (succeded) {
-			not->Hide(_buffer[id].Get());
+			notify->Hide(_buffer[id].Get());
 		}
 		_buffer.erase(id);
 	}
@@ -668,11 +668,11 @@ bool WinToast::hideToast(_In_ INT64 id) {
 
 void WinToast::clear() {
 	bool succeded = false;
-	ComPtr<IToastNotifier> not = notifier(&succeded);
+	ComPtr<IToastNotifier> notify = notifier(&succeded);
 	if (succeded) {
 		auto end = _buffer.end();
 		for (auto it = _buffer.begin(); it != end; ++it) {
-			not->Hide(it->second.Get());
+			notify->Hide(it->second.Get());
 		}
 	}
     _buffer.clear();
@@ -896,7 +896,7 @@ void WinToastTemplate::setAttributionText(_In_ const std::wstring& attributionTe
     _attributionText = attributionText;
 }
 
-void WinToastTemplate::addAction(const std::wstring & label)
+void WinToastTemplate::addAction(_In_ const std::wstring & label)
 {
 	_actions.push_back(label);
 }
diff --git a/src/wintoastlib.h b/src/wintoastlib.h
@@ -138,7 +138,7 @@ namespace WinToastLib {
         HRESULT     setTextFieldHelper(_In_ IXmlDocument *xml, _In_ const std::wstring& text, _In_ int pos);
         HRESULT     setAttributionTextFieldHelper(_In_ IXmlDocument *xml, _In_ const std::wstring& text);
         HRESULT     addActionHelper(_In_ IXmlDocument *xml, _In_ const std::wstring& action, _In_ const std::wstring& arguments);
-		ComPtr<IToastNotifier> WinToast::notifier(_In_ bool* succeded) const;
+		ComPtr<IToastNotifier> notifier(_In_ bool* succeded) const;
     };
 }
 #endif // WINTOASTLIB_H