Skip to content

Commit 3e9d9ef

Browse files
author
Ivan Dlugos
committed
C++ generated code - avoid unintended type conversion in a string vector loop
1 parent f03fecc commit 3e9d9ef

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

internal/generator/c/templates/binding-cpp.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ void {{$entity.Meta.CppNamespacePrefix}}{{$entity.Meta.CppName}}::_OBX_MetaInfo:
8383
{{- if $property.Meta.Optional}}
8484
outObject.{{$property.Meta.CppName}}{{if IsOptionalPtr $property.Meta.Optional}}.reset(new {{$property.Meta.CppType}}({{else}} = {{$property.Meta.CppType}}(){{end}}{{template "field-value-assign-post" $property.Meta}};{{end}}
8585
outObject.{{$property.Meta.CppName}}{{$property.Meta.CppValOp}}reserve(ptr->size());
86-
for (size_t i = 0; i < ptr->size(); i++) {
86+
for (flatbuffers::uoffset_t i = 0; i < ptr->size(); i++) {
8787
auto* itemPtr = ptr->Get(i);
8888
if (itemPtr) outObject.{{$property.Meta.CppName}}{{$property.Meta.CppValOp}}emplace_back(itemPtr->c_str());
8989
}

test/comparison/testdata/fbs/typeful/cpp/schema.obx.cpp.expected

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ void Typeful::_OBX_MetaInfo::fromFlatBuffer(const void* data, size_t, Typeful& o
9696
auto* ptr = table->GetPointer<const flatbuffers::Vector<flatbuffers::Offset<flatbuffers::String>>*>(30);
9797
if (ptr) {
9898
outObject.stringvector.reserve(ptr->size());
99-
for (size_t i = 0; i < ptr->size(); i++) {
99+
for (flatbuffers::uoffset_t i = 0; i < ptr->size(); i++) {
100100
auto* itemPtr = ptr->Get(i);
101101
if (itemPtr) outObject.stringvector.emplace_back(itemPtr->c_str());
102102
}

0 commit comments

Comments
 (0)