Skip to content

Commit 751e8f5

Browse files
author
Lukasz A.J. Wrona
committed
Adhere to lintage
1 parent b544a4b commit 751e8f5

File tree

3 files changed

+30
-22
lines changed

3 files changed

+30
-22
lines changed

src/solvers/refinement/bv_refinement.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ class bv_refinementt:public bv_pointerst
3232
bool refine_arithmetic=true;
3333
};
3434
public:
35-
struct infot : public configt
35+
struct infot:public configt
3636
{
3737
const namespacet *ns=nullptr;
3838
propt *prop=nullptr;

src/solvers/refinement/string_refinement.cpp

+25-19
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,8 @@ optionalt<string_exprt> expr_cast<string_exprt>(const exprt& expr)
139139
}
140140

141141
template<typename T>
142-
T expr_cast_v(const exprt& expr) {
142+
T expr_cast_v(const exprt& expr)
143+
{
143144
const auto maybe=expr_cast<T>(expr);
144145
INVARIANT(maybe, "Bad conversion");
145146
return *maybe;
@@ -535,7 +536,8 @@ std::vector<exprt> concretize_results(
535536
const std::map<exprt, std::set<exprt>>& current_index_set,
536537
const std::vector<string_constraintt> &universal_axioms)
537538
{
538-
for(const auto &it : symbol_resolve) {
539+
for(const auto &it : symbol_resolve)
540+
{
539541
concretize_string(
540542
get,
541543
found_length,
@@ -547,7 +549,8 @@ std::vector<exprt> concretize_results(
547549
ns,
548550
it.second);
549551
}
550-
for(const auto &expr : created_strings) {
552+
for(const auto &expr : created_strings)
553+
{
551554
concretize_string(
552555
get,
553556
found_length,
@@ -654,7 +657,7 @@ void string_refinementt::set_to(const exprt &expr, bool value)
654657
{
655658
if(is_char_array(ns, rhs.type()))
656659
{
657-
for (const auto& lemma : set_char_array_equality(lhs, rhs))
660+
for(const auto& lemma : set_char_array_equality(lhs, rhs))
658661
add_lemma(lemma, false);
659662
}
660663
const bool not_handled=add_axioms_for_string_assigns(
@@ -665,7 +668,7 @@ void string_refinementt::set_to(const exprt &expr, bool value)
665668
ns,
666669
lhs,
667670
subst_rhs);
668-
if (!not_handled)
671+
if(!not_handled)
669672
return;
670673
}
671674

@@ -753,7 +756,7 @@ decision_proceduret::resultt string_refinementt::dec_solve()
753756
symbol_resolve);
754757
if(!success)
755758
{
756-
for (const auto& lemma : lemmas)
759+
for(const auto& lemma : lemmas)
757760
add_lemma(lemma);
758761
debug() << "check_SAT: got SAT but the model is not correct" << eom;
759762
}
@@ -772,16 +775,16 @@ decision_proceduret::resultt string_refinementt::dec_solve()
772775
initial_index_set(index_set, current_index_set, ns, universal_axioms);
773776
update_index_set(index_set, current_index_set, ns, cur);
774777
cur.clear();
775-
for (const auto& lemma :
776-
generate_instantiations(current_index_set, universal_axioms))
778+
for(const auto& lemma :
779+
generate_instantiations(current_index_set, universal_axioms))
777780
add_lemma(lemma);
778781
display_current_index_set(debug(), ns, current_index_set);
779782

780783
while((loop_bound_--)>0)
781784
{
782785
decision_proceduret::resultt res=supert::dec_solve();
783786

784-
if (res == resultt::D_SATISFIABLE)
787+
if(res==resultt::D_SATISFIABLE)
785788
{
786789
bool success;
787790
std::vector<exprt> lemmas;
@@ -798,7 +801,7 @@ decision_proceduret::resultt string_refinementt::dec_solve()
798801
symbol_resolve);
799802
if(!success)
800803
{
801-
for (const auto& lemma : lemmas)
804+
for(const auto& lemma : lemmas)
802805
add_lemma(lemma);
803806
debug() << "check_SAT: got SAT but the model is not correct" << eom;
804807
}
@@ -821,7 +824,7 @@ decision_proceduret::resultt string_refinementt::dec_solve()
821824
current_index_set.clear();
822825
update_index_set(index_set, current_index_set, ns, cur);
823826
cur.clear();
824-
for (const auto& lemma :
827+
for(const auto& lemma :
825828
generate_instantiations(current_index_set, universal_axioms))
826829
add_lemma(lemma);
827830
display_current_index_set(debug(), ns, current_index_set);
@@ -843,7 +846,7 @@ decision_proceduret::resultt string_refinementt::dec_solve()
843846
generator.get_created_strings(),
844847
current_index_set,
845848
universal_axioms);
846-
for (const auto& lemma : lemmas)
849+
for(const auto& lemma : lemmas)
847850
add_lemma(lemma);
848851
display_current_index_set(debug(), ns, current_index_set);
849852
return resultt::D_SATISFIABLE;
@@ -867,7 +870,9 @@ decision_proceduret::resultt string_refinementt::dec_solve()
867870
for(const exprt &lemma : lemmas)
868871
add_lemma(lemma);
869872
}
870-
} else {
873+
}
874+
else
875+
{
871876
debug() << "check_SAT: default return " << static_cast<int>(res) << eom;
872877
return res;
873878
}
@@ -1923,7 +1928,8 @@ class find_index_visitort: public const_expr_visitort
19231928
/// \param [in] str: the string which must be indexed
19241929
/// \param [in] qvar: the universal variable that must be in the index
19251930
/// \return an index expression in `expr` on `str` containing `qvar`
1926-
static exprt find_index(const exprt &expr, const exprt &str, const symbol_exprt &qvar)
1931+
static exprt find_index(
1932+
const exprt &expr, const exprt &str, const symbol_exprt &qvar)
19271933
{
19281934
find_index_visitort v(str, qvar);
19291935
expr.visit(v);
@@ -1972,7 +1978,8 @@ static std::vector<exprt> instantiate_not_contains(
19721978
// << from_expr(ns, "", s1) << eom;
19731979
const auto& i0=index_set.find(s0.content());
19741980
const auto& i1=index_set.find(s1.content());
1975-
if (i0 != index_set.end() && i1 != index_set.end()) {
1981+
if(i0!=index_set.end() && i1!=index_set.end())
1982+
{
19761983
return ::instantiate_not_contains(
19771984
axiom, i0->second, i1->second, generator);
19781985
}
@@ -2025,9 +2032,8 @@ exprt substitute_array_lists(exprt expr, size_t string_max_length)
20252032
/// \return an expression
20262033
exprt string_refinementt::get(const exprt &expr) const
20272034
{
2028-
const auto super_get = [this](const exprt& expr) {
2029-
return supert::get(expr);
2030-
};
2035+
const auto super_get=[this](const exprt& expr)
2036+
{ return supert::get(expr); };
20312037
exprt ecopy(expr);
20322038
replace_expr(symbol_resolve, ecopy);
20332039
if(is_char_array(ns, ecopy.type()))
@@ -2047,7 +2053,7 @@ exprt string_refinementt::get(const exprt &expr) const
20472053
}
20482054
else if(ecopy.id()==ID_struct)
20492055
{
2050-
if (const auto string=expr_cast<string_exprt>(ecopy))
2056+
if(const auto string=expr_cast<string_exprt>(ecopy))
20512057
{
20522058
const exprt &content=string->content();
20532059
const exprt &length=string->length();

src/solvers/refinement/string_refinement.h

+4-2
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,8 @@ Author: Alberto Griggio, [email protected]
3232
class string_refinementt final: public bv_refinementt
3333
{
3434
private:
35-
struct configt {
35+
struct configt
36+
{
3637
unsigned refinement_bound=0;
3738
/// Make non-deterministic character arrays have at least one character
3839
bool string_non_empty=false;
@@ -98,5 +99,6 @@ class string_refinementt final: public bv_refinementt
9899
};
99100

100101
exprt substitute_array_lists(exprt expr, size_t string_max_length);
101-
exprt concretize_arrays_in_expression(exprt expr, std::size_t string_max_length);
102+
exprt concretize_arrays_in_expression(
103+
exprt expr, std::size_t string_max_length);
102104
#endif

0 commit comments

Comments
 (0)