diff --git a/.clang-tidy b/.clang-tidy index cbec3cc4416..fe09f0bcd72 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -1,5 +1,5 @@ --- -Checks: '-*,readability-*,bugprone-*,misc-*,performance-*,modernize-redundant-void-arg,modernize-deprecated-headers,modernize-use-override,modernize-use-auto,modernize-use-nullptr,modernize-use-default-member-init,modernize-use-equals-default,modernize-use-equals-delete,modernize-pass-by-value,-readability-else-after-return,-readability-named-parameter,-readability-braces-around-statements,-readability-inconsistent-declaration-parameter-name,-readability-isolate-declaration,-readability-magic-numbers,-readability-convert-member-functions-to-static,-bugprone-macro-parentheses,-bugprone-narrowing-conversions,-bugprone-branch-clone,-misc-non-private-member-variables-in-classes,-misc-unused-parameters' +Checks: '-*,readability-*,bugprone-*,misc-*,performance-*,modernize-redundant-void-arg,modernize-deprecated-headers,modernize-use-override,modernize-use-auto,modernize-use-nullptr,modernize-use-default-member-init,modernize-use-equals-default,modernize-use-equals-delete,modernize-pass-by-value,modernize-loop-convert,-readability-else-after-return,-readability-named-parameter,-readability-braces-around-statements,-readability-inconsistent-declaration-parameter-name,-readability-isolate-declaration,-readability-magic-numbers,-readability-convert-member-functions-to-static,-bugprone-macro-parentheses,-bugprone-narrowing-conversions,-bugprone-branch-clone,-misc-non-private-member-variables-in-classes,-misc-unused-parameters' WarningsAsErrors: '' HeaderFilterRegex: '.*' AnalyzeTemporaryDtors: false diff --git a/Examples/BasketLosses/BasketLosses.cpp b/Examples/BasketLosses/BasketLosses.cpp index 5d263f08d8d..5b380055aae 100644 --- a/Examples/BasketLosses/BasketLosses.cpp +++ b/Examples/BasketLosses/BasketLosses.cpp @@ -77,10 +77,9 @@ int main(int, char* []) { names.push_back(std::string("Acme") + boost::lexical_cast(i)); std::vector > defTS; - for(Size i=0; i( - ext::make_shared(0, TARGET(), hazardRates[i], - Actual365Fixed()))); + ext::make_shared(0, TARGET(), hazardRate, Actual365Fixed()))); defTS.back()->enableExtrapolation(); } std::vector issuers; diff --git a/Examples/Bonds/Bonds.cpp b/Examples/Bonds/Bonds.cpp index 621cdcd83bb..11bf9466a2f 100644 --- a/Examples/Bonds/Bonds.cpp +++ b/Examples/Bonds/Bonds.cpp @@ -163,8 +163,8 @@ int main(int, char* []) { }; std::vector< ext::shared_ptr > quote; - for (Size i=0; i cp(new SimpleQuote(marketQuotes[i])); + for (double marketQuote : marketQuotes) { + ext::shared_ptr cp(new SimpleQuote(marketQuote)); quote.push_back(cp); } diff --git a/Examples/CDS/CDS.cpp b/Examples/CDS/CDS.cpp index 2f4a1001b9f..7d4290ecdd8 100644 --- a/Examples/CDS/CDS.cpp +++ b/Examples/CDS/CDS.cpp @@ -119,9 +119,8 @@ void example01() { vector > hr_curve_data = hazardRateStructure->nodes(); cout << "Calibrated hazard rate values: " << endl; - for (Size i = 0; i < hr_curve_data.size(); i++) { - cout << "hazard rate on " << hr_curve_data[i].first << " is " - << hr_curve_data[i].second << endl; + for (auto& i : hr_curve_data) { + cout << "hazard rate on " << i.first << " is " << i.second << endl; } cout << endl; @@ -345,8 +344,8 @@ std::copy(cdsSchedule.begin(), cdsSchedule.end(), // output rate curve std::cout << "ISDA rate curve: " << std::endl; - for(Size i=0;ilatestDate(); + for (auto& i : isdaRateHelper) { + Date d = i->latestDate(); std::cout << d << "\t" << setprecision(6) << rateTs->zeroRate(d,Actual365Fixed(),Continuous).rate() << "\t" << rateTs->discount(d) << std::endl; @@ -395,8 +394,8 @@ std::copy(cdsSchedule.begin(), cdsSchedule.end(), 0, WeekendsOnly(), isdaCdsHelper, Actual365Fixed())); std::cout << "ISDA credit curve: " << std::endl; - for(Size i=0;ilatestDate(); + for (auto& i : isdaCdsHelper) { + Date d = i->latestDate(); Real pd = defaultTs->defaultProbability(d); Real t = defaultTs->timeFromReference(d); std::cout << d << ";" << pd << ";" << 1.0 - pd << ";" << @@ -404,7 +403,6 @@ std::copy(cdsSchedule.begin(), cdsSchedule.end(), } - // // set up sample CDS trade // ext::shared_ptr trade = @@ -613,8 +611,8 @@ void example03() { // check the curves std::cout << "ISDA yield curve:" << std::endl; std::cout << "date;time;zeroyield" << std::endl; - for (Size i = 0; i < isdaYieldHelpers.size(); i++) { - Date d = isdaYieldHelpers[i]->latestDate(); + for (auto& isdaYieldHelper : isdaYieldHelpers) { + Date d = isdaYieldHelper->latestDate(); Real t = isdaYts->timeFromReference(d); std::cout << d << ";" << t << ";" << isdaYts->zeroRate(d, Actual365Fixed(), Continuous).rate() @@ -623,8 +621,8 @@ void example03() { std::cout << "ISDA credit curve:" << std::endl; std::cout << "date;time;survivalprob" << std::endl; - for (Size i = 0; i < isdaCdsHelpers.size(); i++) { - Date d = isdaCdsHelpers[i]->latestDate(); + for (auto& isdaCdsHelper : isdaCdsHelpers) { + Date d = isdaCdsHelper->latestDate(); Real t = isdaCts->timeFromReference(d); std::cout << d << ";" << t << ";" << isdaCts->survivalProbability(d) << std::endl; diff --git a/Examples/FittedBondCurve/FittedBondCurve.cpp b/Examples/FittedBondCurve/FittedBondCurve.cpp index 860051f72aa..f491dac9b72 100644 --- a/Examples/FittedBondCurve/FittedBondCurve.cpp +++ b/Examples/FittedBondCurve/FittedBondCurve.cpp @@ -89,13 +89,13 @@ int main(int, char* []) { const Size numberOfBonds = 15; Real cleanPrice[numberOfBonds]; - for (Size i=0; i > quote; - for (Size i=0; i cp(new SimpleQuote(cleanPrice[i])); + for (double i : cleanPrice) { + ext::shared_ptr cp(new SimpleQuote(i)); quote.push_back(cp); } @@ -227,8 +227,8 @@ int main(int, char* []) { 20.0, 25.0, 30.0, 40.0, 50.0 }; std::vector