Skip to content

Commit 4596d33

Browse files
committed
BrowserSubprocess - TreatWarningAsError for debug builds
Statically cast size_t to int to remove warnings
1 parent b3fd1f5 commit 4596d33

File tree

6 files changed

+13
-8
lines changed

6 files changed

+13
-8
lines changed

CefSharp.BrowserSubprocess.Core/CefSharp.BrowserSubprocess.Core.vcxproj

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -110,6 +110,7 @@
110110
<PrecompiledHeader>Use</PrecompiledHeader>
111111
<BrowseInformation>true</BrowseInformation>
112112
<MultiProcessorCompilation>true</MultiProcessorCompilation>
113+
<TreatWarningAsError>true</TreatWarningAsError>
113114
</ClCompile>
114115
<Link>
115116
<GenerateDebugInformation>true</GenerateDebugInformation>
@@ -126,6 +127,7 @@
126127
<PreprocessorDefinitions>WIN32;_DEBUG;EXPORT;%(PreprocessorDefinitions)</PreprocessorDefinitions>
127128
<PrecompiledHeader>Use</PrecompiledHeader>
128129
<MultiProcessorCompilation>true</MultiProcessorCompilation>
130+
<TreatWarningAsError>true</TreatWarningAsError>
129131
</ClCompile>
130132
<Link>
131133
<GenerateDebugInformation>true</GenerateDebugInformation>

CefSharp.BrowserSubprocess.Core/JavascriptMethodHandler.cpp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,9 +12,10 @@ namespace CefSharp
1212
{
1313
bool JavascriptMethodHandler::Execute(const CefString& name, CefRefPtr<CefV8Value> object, const CefV8ValueList& arguments, CefRefPtr<CefV8Value>& retval, CefString& exception)
1414
{
15-
auto parameter = gcnew array<Object^>(arguments.size());
15+
auto size = static_cast<int>(arguments.size());
16+
auto parameter = gcnew array<Object^>(size);
1617

17-
for (std::vector<CefRefPtr<CefV8Value>>::size_type i = 0; i != arguments.size(); i++)
18+
for (int i = 0; i < size; i++)
1819
{
1920
parameter[i] = TypeUtils::ConvertFromCef(arguments[i], _callbackRegistry);
2021
}

CefSharp.BrowserSubprocess.Core/Serialization/JsObjectsSerialization.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,9 @@ namespace CefSharp
7070
{
7171
auto result = gcnew List<JavascriptObject^>();
7272
auto subList = list->GetList(index);
73-
for (size_t i = 0; i < subList->GetSize(); i++)
73+
auto size = static_cast<int>(subList->GetSize());
74+
75+
for (int i = 0; i < size; i++)
7476
{
7577
result->Add(DeserializeJsObject(subList, i));
7678
}

CefSharp.BrowserSubprocess.Core/Serialization/V8Serialization.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -149,9 +149,9 @@ namespace CefSharp
149149
if (type == VTYPE_LIST)
150150
{
151151
auto subList = list->GetList(index);
152-
size_t size = subList->GetSize();
152+
auto size = static_cast<int>(subList->GetSize());
153153
auto result = CefV8Value::CreateArray(size);
154-
for (size_t i = 0; i < size; i++)
154+
for (int i = 0; i < size; i++)
155155
{
156156
result->SetValue(i, DeserializeV8Object(subList, i));
157157
}

CefSharp.BrowserSubprocess.Core/Wrapper/Browser.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,7 @@ IFrame^ Browser::GetFrame(String^ name)
189189
/*--cef()--*/
190190
int Browser::GetFrameCount()
191191
{
192-
return _browser->GetFrameCount();
192+
return static_cast<int>(_browser->GetFrameCount());
193193
}
194194

195195
///
@@ -200,7 +200,7 @@ List<Int64>^ Browser::GetFrameIdentifiers()
200200
{
201201
std::vector<Int64> identifiers;
202202
_browser->GetFrameIdentifiers(identifiers);
203-
List<Int64>^ results = gcnew List<Int64>(identifiers.size());
203+
List<Int64>^ results = gcnew List<Int64>(static_cast<int>(identifiers.size()));
204204
for (UINT i = 0; i < identifiers.size(); i++)
205205
{
206206
results->Add(identifiers[i]);

CefSharp.Core.Runtime/Internals/Serialization/ObjectsSerialization.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ namespace CefSharp
5454
else if (type == VTYPE_LIST)
5555
{
5656
auto subList = list->GetList(index);
57-
auto array = gcnew List<Object^>(subList->GetSize());
57+
auto array = gcnew List<Object^>(static_cast<int>(subList->GetSize()));
5858
for (size_t i = 0; i < subList->GetSize(); i++)
5959
{
6060
array->Add(DeserializeObject(subList, i, javascriptCallbackFactory));

0 commit comments

Comments
 (0)