Skip to content

Commit 8e3e5e8

Browse files
committed
Use range for loop syntax
1 parent 869217d commit 8e3e5e8

File tree

1 file changed

+6
-12
lines changed

1 file changed

+6
-12
lines changed

bindings/pyroot/cppyy/CPyCppyy/src/CPPOverload.cxx

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -256,13 +256,10 @@ static PyObject* mp_func_overloads_types(CPPOverload* pymeth, void*)
256256

257257
CPPOverload::Methods_t& methods = pymeth->fMethodInfo->fMethods;
258258

259-
CPPOverload::Methods_t::size_type nMethods = methods.size();
260-
if (nMethods == 0) // from template proxy with no instantiations
261-
return nullptr;
262-
263259
PyObject* overloads_types_dict = PyDict_New();
264-
for (CPPOverload::Methods_t::size_type i = 0; i < nMethods; ++i) {
265-
PyDict_SetItem(overloads_types_dict, methods[i]->GetSignature(), methods[i]->GetSignatureTypes());
260+
261+
for (const auto &method: methods) {
262+
PyDict_SetItem(overloads_types_dict, method->GetSignature(), method->GetSignatureTypes());
266263
}
267264

268265
return overloads_types_dict;
@@ -274,13 +271,10 @@ static PyObject* mp_func_overloads_names(CPPOverload* pymeth, void*)
274271

275272
CPPOverload::Methods_t& methods = pymeth->fMethodInfo->fMethods;
276273

277-
CPPOverload::Methods_t::size_type nMethods = methods.size();
278-
if (nMethods == 0) // from template proxy with no instantiations
279-
return nullptr;
280-
281274
PyObject* overloads_names_dict = PyDict_New();
282-
for (CPPOverload::Methods_t::size_type i = 0; i < nMethods; ++i) {
283-
PyDict_SetItem(overloads_names_dict, methods[i]->GetSignature(), methods[i]->GetSignatureNames());
275+
276+
for (const auto &method: methods) {
277+
PyDict_SetItem(overloads_names_dict, method->GetSignature(), method->GetSignatureNames());
284278
}
285279

286280
return overloads_names_dict;

0 commit comments

Comments
 (0)