alarsyo/spot
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Fix two dead assignments detected by clang's static analyzer.

Browse source 
* src/tgbaalgos/scc.cc, src/tgbatest/ltl2tgba.cc: Remove assignments to
unread variables.
This commit is contained in:
Alexandre Duret-Lutz 2013-01-17 11:36:59 +01:00
parent dcd4644d42
commit b99cfa88bb
2 changed files with 8 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

7
src/tgbaalgos/scc.cc
View file

@ -1,6 +1,6 @@
// -*- coding: utf-8 -*- // -*- coding: utf-8 -*-
// Copyright (C) 2008, 2009, 2011, 2012 Laboratoire de Recherche et // Copyright (C) 2008, 2009, 2011, 2012, 2013 Laboratoire de Recherche
// Développement de l'Epita. // et Développement de l'Epita.
// //
// This file is part of Spot, a model checking library. // This file is part of Spot, a model checking library.
// //
@ -233,9 +233,8 @@ namespace spot
continue; continue;
} }
// If we know the state, reuse the previous object. // We already know the state.
dest->destroy(); dest->destroy();
dest = spi->first;
// Have we reached a maximal SCC? // Have we reached a maximal SCC?
if (spi->second >= 0) if (spi->second >= 0)

13
src/tgbatest/ltl2tgba.cc
View file

@ -1,6 +1,6 @@
// -*- coding: utf-8 -*- // -*- coding: utf-8 -*-
// Copyright (C) 2007, 2008, 2009, 2010, 2011, 2012 Laboratoire de Recherche et // Copyright (C) 2007, 2008, 2009, 2010, 2011, 2012, 2013 Laboratoire
// Développement de l'Epita (LRDE). // de Recherche et Développement de l'Epita (LRDE).
// Copyright (C) 2003, 2004, 2005, 2006, 2007 Laboratoire d'Informatique de // Copyright (C) 2003, 2004, 2005, 2006, 2007 Laboratoire d'Informatique de
// Paris 6 (LIP6), département Systèmes Répartis // Paris 6 (LIP6), département Systèmes Répartis
// Coopératifs (SRC), Université Pierre et Marie Curie. // Coopératifs (SRC), Université Pierre et Marie Curie.
@ -1283,8 +1283,7 @@ main(int argc, char** argv)
if (system) if (system)
{ {
const spot::tgba* product = product_to_free = a = product_to_free = a = new spot::tgba_product(system, a);
new spot::tgba_product(system, a);
assume_sba = false; assume_sba = false;
@ -1296,14 +1295,12 @@ main(int argc, char** argv)
degeneralize_opt = DegenTBA; degeneralize_opt = DegenTBA;
if (degeneralize_opt == DegenTBA) if (degeneralize_opt == DegenTBA)
{ {
product = product_degeneralized = a = product_degeneralized = a = new spot::tgba_tba_proxy(a);
new spot::tgba_tba_proxy(product);
} }
else if (degeneralize_opt == DegenSBA) else if (degeneralize_opt == DegenSBA)
{ {
tm.start("degeneralize product"); tm.start("degeneralize product");
product = product_degeneralized = a = product_degeneralized = a = spot::degeneralize(a);
spot::degeneralize(product);
tm.stop("degeneralize product"); tm.stop("degeneralize product");
assume_sba = true; assume_sba = true;
} }