blob: f5cae698f92431071f0b504f5c6865cc6f45090a [file] [log] [blame]
Ted Kremenek45021952009-02-14 17:08:39 +00001//== RangeConstraintManager.cpp - Manage range constraints.------*- C++ -*--==//
2//
3// The LLVM Compiler Infrastructure
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9//
Mike Stump1eb44332009-09-09 15:08:12 +000010// This file defines RangeConstraintManager, a class that tracks simple
Ted Kremenek45021952009-02-14 17:08:39 +000011// equality and inequality constraints on symbolic values of GRState.
12//
13//===----------------------------------------------------------------------===//
14
15#include "SimpleConstraintManager.h"
16#include "clang/Analysis/PathSensitive/GRState.h"
17#include "clang/Analysis/PathSensitive/GRStateTrait.h"
18#include "clang/Analysis/PathSensitive/GRTransferFuncs.h"
Chandler Carruthf465e852009-11-11 19:10:59 +000019#include "clang/Analysis/ManagerRegistry.h"
Ted Kremenek45021952009-02-14 17:08:39 +000020#include "llvm/Support/Compiler.h"
21#include "llvm/Support/Debug.h"
22#include "llvm/ADT/FoldingSet.h"
23#include "llvm/ADT/ImmutableSet.h"
24#include "llvm/Support/raw_ostream.h"
25
26using namespace clang;
27
Ted Kremenek9beefec2009-02-17 19:28:04 +000028namespace { class VISIBILITY_HIDDEN ConstraintRange {}; }
29static int ConstraintRangeIndex = 0;
Ted Kremenek45021952009-02-14 17:08:39 +000030
Ted Kremenek9beefec2009-02-17 19:28:04 +000031/// A Range represents the closed range [from, to]. The caller must
32/// guarantee that from <= to. Note that Range is immutable, so as not
33/// to subvert RangeSet's immutability.
Ted Kremenekb103f012009-02-18 05:22:01 +000034namespace {
35class VISIBILITY_HIDDEN Range : public std::pair<const llvm::APSInt*,
36 const llvm::APSInt*> {
Ted Kremenek45021952009-02-14 17:08:39 +000037public:
38 Range(const llvm::APSInt &from, const llvm::APSInt &to)
Ted Kremenek9beefec2009-02-17 19:28:04 +000039 : std::pair<const llvm::APSInt*, const llvm::APSInt*>(&from, &to) {
Ted Kremenek45021952009-02-14 17:08:39 +000040 assert(from <= to);
41 }
42 bool Includes(const llvm::APSInt &v) const {
Ted Kremenek9beefec2009-02-17 19:28:04 +000043 return *first <= v && v <= *second;
Ted Kremenek45021952009-02-14 17:08:39 +000044 }
45 const llvm::APSInt &From() const {
Ted Kremenek9beefec2009-02-17 19:28:04 +000046 return *first;
Ted Kremenek45021952009-02-14 17:08:39 +000047 }
48 const llvm::APSInt &To() const {
Ted Kremenek9beefec2009-02-17 19:28:04 +000049 return *second;
Ted Kremenek45021952009-02-14 17:08:39 +000050 }
Ted Kremenek9beefec2009-02-17 19:28:04 +000051 const llvm::APSInt *getConcreteValue() const {
52 return &From() == &To() ? &From() : NULL;
Ted Kremenek45021952009-02-14 17:08:39 +000053 }
54
55 void Profile(llvm::FoldingSetNodeID &ID) const {
Ted Kremenek9beefec2009-02-17 19:28:04 +000056 ID.AddPointer(&From());
57 ID.AddPointer(&To());
Ted Kremenek45021952009-02-14 17:08:39 +000058 }
59};
60
Ted Kremenekb103f012009-02-18 05:22:01 +000061
62class VISIBILITY_HIDDEN RangeTrait : public llvm::ImutContainerInfo<Range> {
63public:
64 // When comparing if one Range is less than another, we should compare
Ted Kremeneke53f8202009-02-18 17:42:44 +000065 // the actual APSInt values instead of their pointers. This keeps the order
66 // consistent (instead of comparing by pointer values) and can potentially
67 // be used to speed up some of the operations in RangeSet.
Ted Kremenekb103f012009-02-18 05:22:01 +000068 static inline bool isLess(key_type_ref lhs, key_type_ref rhs) {
Mike Stump1eb44332009-09-09 15:08:12 +000069 return *lhs.first < *rhs.first || (!(*rhs.first < *lhs.first) &&
Ted Kremenekb103f012009-02-18 05:22:01 +000070 *lhs.second < *rhs.second);
71 }
72};
73
Ted Kremenek9beefec2009-02-17 19:28:04 +000074/// RangeSet contains a set of ranges. If the set is empty, then
75/// there the value of a symbol is overly constrained and there are no
76/// possible values for that symbol.
Ted Kremenekb103f012009-02-18 05:22:01 +000077class VISIBILITY_HIDDEN RangeSet {
78 typedef llvm::ImmutableSet<Range, RangeTrait> PrimRangeSet;
Ted Kremenek45021952009-02-14 17:08:39 +000079 PrimRangeSet ranges; // no need to make const, since it is an
80 // ImmutableSet - this allows default operator=
Mike Stump1eb44332009-09-09 15:08:12 +000081 // to work.
Ted Kremenek45021952009-02-14 17:08:39 +000082public:
Ted Kremenek9beefec2009-02-17 19:28:04 +000083 typedef PrimRangeSet::Factory Factory;
84 typedef PrimRangeSet::iterator iterator;
85
86 RangeSet(PrimRangeSet RS) : ranges(RS) {}
87 RangeSet(Factory& F) : ranges(F.GetEmptySet()) {}
88
89 iterator begin() const { return ranges.begin(); }
90 iterator end() const { return ranges.end(); }
Mike Stump1eb44332009-09-09 15:08:12 +000091
Ted Kremenek9beefec2009-02-17 19:28:04 +000092 bool isEmpty() const { return ranges.isEmpty(); }
Mike Stump1eb44332009-09-09 15:08:12 +000093
Ted Kremenek9beefec2009-02-17 19:28:04 +000094 /// Construct a new RangeSet representing '{ [from, to] }'.
95 RangeSet(Factory &F, const llvm::APSInt &from, const llvm::APSInt &to)
96 : ranges(F.Add(F.GetEmptySet(), Range(from, to))) {}
Mike Stump1eb44332009-09-09 15:08:12 +000097
Ted Kremenek9beefec2009-02-17 19:28:04 +000098 /// Profile - Generates a hash profile of this RangeSet for use
99 /// by FoldingSet.
100 void Profile(llvm::FoldingSetNodeID &ID) const { ranges.Profile(ID); }
101
102 /// getConcreteValue - If a symbol is contrained to equal a specific integer
103 /// constant then this method returns that value. Otherwise, it returns
104 /// NULL.
105 const llvm::APSInt* getConcreteValue() const {
106 return ranges.isSingleton() ? ranges.begin()->getConcreteValue() : 0;
Ted Kremenek45021952009-02-14 17:08:39 +0000107 }
108
Ted Kremenek9beefec2009-02-17 19:28:04 +0000109 /// AddEQ - Create a new RangeSet with the additional constraint that the
110 /// value be equal to V.
111 RangeSet AddEQ(BasicValueFactory &BV, Factory &F, const llvm::APSInt &V) {
112 // Search for a range that includes 'V'. If so, return a new RangeSet
113 // representing { [V, V] }.
114 for (PrimRangeSet::iterator i = begin(), e = end(); i!=e; ++i)
115 if (i->Includes(V))
116 return RangeSet(F, V, V);
117
118 return RangeSet(F);
Ted Kremenek45021952009-02-14 17:08:39 +0000119 }
120
Ted Kremenek9beefec2009-02-17 19:28:04 +0000121 /// AddNE - Create a new RangeSet with the additional constraint that the
122 /// value be not be equal to V.
123 RangeSet AddNE(BasicValueFactory &BV, Factory &F, const llvm::APSInt &V) {
124 PrimRangeSet newRanges = ranges;
Mike Stump1eb44332009-09-09 15:08:12 +0000125
Ted Kremenek9beefec2009-02-17 19:28:04 +0000126 // FIXME: We can perhaps enhance ImmutableSet to do this search for us
127 // in log(N) time using the sorted property of the internal AVL tree.
128 for (iterator i = begin(), e = end(); i != e; ++i) {
129 if (i->Includes(V)) {
130 // Remove the old range.
131 newRanges = F.Remove(newRanges, *i);
132 // Split the old range into possibly one or two ranges.
133 if (V != i->From())
134 newRanges = F.Add(newRanges, Range(i->From(), BV.Sub1(V)));
135 if (V != i->To())
136 newRanges = F.Add(newRanges, Range(BV.Add1(V), i->To()));
Mike Stump1eb44332009-09-09 15:08:12 +0000137 // All of the ranges are non-overlapping, so we can stop.
Ted Kremenek9beefec2009-02-17 19:28:04 +0000138 break;
Ted Kremenek45021952009-02-14 17:08:39 +0000139 }
140 }
Mike Stump1eb44332009-09-09 15:08:12 +0000141
Ted Kremenek9beefec2009-02-17 19:28:04 +0000142 return newRanges;
Ted Kremenek45021952009-02-14 17:08:39 +0000143 }
144
Ted Kremenek9beefec2009-02-17 19:28:04 +0000145 /// AddNE - Create a new RangeSet with the additional constraint that the
146 /// value be less than V.
147 RangeSet AddLT(BasicValueFactory &BV, Factory &F, const llvm::APSInt &V) {
148 PrimRangeSet newRanges = F.GetEmptySet();
149
150 for (iterator i = begin(), e = end() ; i != e ; ++i) {
151 if (i->Includes(V) && i->From() < V)
152 newRanges = F.Add(newRanges, Range(i->From(), BV.Sub1(V)));
153 else if (i->To() < V)
154 newRanges = F.Add(newRanges, *i);
155 }
Mike Stump1eb44332009-09-09 15:08:12 +0000156
Ted Kremenek9beefec2009-02-17 19:28:04 +0000157 return newRanges;
Ted Kremenek45021952009-02-14 17:08:39 +0000158 }
159
Ted Kremenek9beefec2009-02-17 19:28:04 +0000160 RangeSet AddLE(BasicValueFactory &BV, Factory &F, const llvm::APSInt &V) {
161 PrimRangeSet newRanges = F.GetEmptySet();
Ted Kremenek45021952009-02-14 17:08:39 +0000162
Ted Kremenek9beefec2009-02-17 19:28:04 +0000163 for (iterator i = begin(), e = end(); i != e; ++i) {
164 // Strictly we should test for includes *V + 1, but no harm is
Ted Kremenek45021952009-02-14 17:08:39 +0000165 // done by this formulation
Ted Kremenek9beefec2009-02-17 19:28:04 +0000166 if (i->Includes(V))
167 newRanges = F.Add(newRanges, Range(i->From(), V));
168 else if (i->To() <= V)
169 newRanges = F.Add(newRanges, *i);
Ted Kremenek45021952009-02-14 17:08:39 +0000170 }
Mike Stump1eb44332009-09-09 15:08:12 +0000171
Ted Kremenek9beefec2009-02-17 19:28:04 +0000172 return newRanges;
Ted Kremenek45021952009-02-14 17:08:39 +0000173 }
174
Ted Kremenek9beefec2009-02-17 19:28:04 +0000175 RangeSet AddGT(BasicValueFactory &BV, Factory &F, const llvm::APSInt &V) {
176 PrimRangeSet newRanges = F.GetEmptySet();
Ted Kremenek45021952009-02-14 17:08:39 +0000177
Ted Kremenek9beefec2009-02-17 19:28:04 +0000178 for (PrimRangeSet::iterator i = begin(), e = end(); i != e; ++i) {
179 if (i->Includes(V) && i->To() > V)
180 newRanges = F.Add(newRanges, Range(BV.Add1(V), i->To()));
181 else if (i->From() > V)
182 newRanges = F.Add(newRanges, *i);
Ted Kremenek45021952009-02-14 17:08:39 +0000183 }
Mike Stump1eb44332009-09-09 15:08:12 +0000184
Ted Kremenek9beefec2009-02-17 19:28:04 +0000185 return newRanges;
Ted Kremenek45021952009-02-14 17:08:39 +0000186 }
187
Ted Kremenek9beefec2009-02-17 19:28:04 +0000188 RangeSet AddGE(BasicValueFactory &BV, Factory &F, const llvm::APSInt &V) {
189 PrimRangeSet newRanges = F.GetEmptySet();
Ted Kremenek45021952009-02-14 17:08:39 +0000190
Ted Kremenek9beefec2009-02-17 19:28:04 +0000191 for (PrimRangeSet::iterator i = begin(), e = end(); i != e; ++i) {
192 // Strictly we should test for includes *V - 1, but no harm is
Ted Kremenek45021952009-02-14 17:08:39 +0000193 // done by this formulation
Ted Kremenek9beefec2009-02-17 19:28:04 +0000194 if (i->Includes(V))
195 newRanges = F.Add(newRanges, Range(V, i->To()));
196 else if (i->From() >= V)
197 newRanges = F.Add(newRanges, *i);
Ted Kremenek45021952009-02-14 17:08:39 +0000198 }
199
Ted Kremenek9beefec2009-02-17 19:28:04 +0000200 return newRanges;
Ted Kremenek45021952009-02-14 17:08:39 +0000201 }
202
Ted Kremenek53ba0b62009-06-24 23:06:47 +0000203 void print(llvm::raw_ostream &os) const {
Ted Kremenek9beefec2009-02-17 19:28:04 +0000204 bool isFirst = true;
Ted Kremenek45021952009-02-14 17:08:39 +0000205 os << "{ ";
Ted Kremenek9beefec2009-02-17 19:28:04 +0000206 for (iterator i = begin(), e = end(); i != e; ++i) {
207 if (isFirst)
208 isFirst = false;
209 else
Ted Kremenek45021952009-02-14 17:08:39 +0000210 os << ", ";
Mike Stump1eb44332009-09-09 15:08:12 +0000211
Ted Kremenek45021952009-02-14 17:08:39 +0000212 os << '[' << i->From().toString(10) << ", " << i->To().toString(10)
213 << ']';
214 }
Mike Stump1eb44332009-09-09 15:08:12 +0000215 os << " }";
Ted Kremenek9beefec2009-02-17 19:28:04 +0000216 }
Mike Stump1eb44332009-09-09 15:08:12 +0000217
Ted Kremenek45021952009-02-14 17:08:39 +0000218 bool operator==(const RangeSet &other) const {
219 return ranges == other.ranges;
220 }
221};
Ted Kremenekb103f012009-02-18 05:22:01 +0000222} // end anonymous namespace
Ted Kremenek45021952009-02-14 17:08:39 +0000223
Ted Kremenek9beefec2009-02-17 19:28:04 +0000224typedef llvm::ImmutableMap<SymbolRef,RangeSet> ConstraintRangeTy;
Ted Kremenek45021952009-02-14 17:08:39 +0000225
226namespace clang {
227template<>
Ted Kremenek9beefec2009-02-17 19:28:04 +0000228struct GRStateTrait<ConstraintRange>
229 : public GRStatePartialTrait<ConstraintRangeTy> {
Mike Stump1eb44332009-09-09 15:08:12 +0000230 static inline void* GDMIndex() { return &ConstraintRangeIndex; }
Ted Kremenek45021952009-02-14 17:08:39 +0000231};
Mike Stump1eb44332009-09-09 15:08:12 +0000232}
233
Ted Kremenek45021952009-02-14 17:08:39 +0000234namespace {
Ted Kremenekb103f012009-02-18 05:22:01 +0000235class VISIBILITY_HIDDEN RangeConstraintManager : public SimpleConstraintManager{
Mike Stump1eb44332009-09-09 15:08:12 +0000236 RangeSet GetRange(const GRState *state, SymbolRef sym);
Ted Kremenek45021952009-02-14 17:08:39 +0000237public:
Ted Kremenekf1b82272009-06-18 23:20:05 +0000238 RangeConstraintManager() {}
Ted Kremenek45021952009-02-14 17:08:39 +0000239
240 const GRState* AssumeSymNE(const GRState* St, SymbolRef sym,
Ted Kremeneka591bc02009-06-18 22:57:13 +0000241 const llvm::APSInt& V);
Ted Kremenek45021952009-02-14 17:08:39 +0000242
243 const GRState* AssumeSymEQ(const GRState* St, SymbolRef sym,
Ted Kremeneka591bc02009-06-18 22:57:13 +0000244 const llvm::APSInt& V);
Ted Kremenek45021952009-02-14 17:08:39 +0000245
246 const GRState* AssumeSymLT(const GRState* St, SymbolRef sym,
Ted Kremeneka591bc02009-06-18 22:57:13 +0000247 const llvm::APSInt& V);
Ted Kremenek45021952009-02-14 17:08:39 +0000248
249 const GRState* AssumeSymGT(const GRState* St, SymbolRef sym,
Ted Kremeneka591bc02009-06-18 22:57:13 +0000250 const llvm::APSInt& V);
Ted Kremenek45021952009-02-14 17:08:39 +0000251
252 const GRState* AssumeSymGE(const GRState* St, SymbolRef sym,
Ted Kremeneka591bc02009-06-18 22:57:13 +0000253 const llvm::APSInt& V);
Ted Kremenek45021952009-02-14 17:08:39 +0000254
255 const GRState* AssumeSymLE(const GRState* St, SymbolRef sym,
Ted Kremeneka591bc02009-06-18 22:57:13 +0000256 const llvm::APSInt& V);
Ted Kremenek45021952009-02-14 17:08:39 +0000257
Ted Kremenek45021952009-02-14 17:08:39 +0000258 const llvm::APSInt* getSymVal(const GRState* St, SymbolRef sym) const;
Mike Stump1eb44332009-09-09 15:08:12 +0000259
Ted Kremenek9beefec2009-02-17 19:28:04 +0000260 // FIXME: Refactor into SimpleConstraintManager?
261 bool isEqual(const GRState* St, SymbolRef sym, const llvm::APSInt& V) const {
262 const llvm::APSInt *i = getSymVal(St, sym);
263 return i ? *i == V : false;
264 }
Ted Kremenek45021952009-02-14 17:08:39 +0000265
Ted Kremenek45021952009-02-14 17:08:39 +0000266 const GRState* RemoveDeadBindings(const GRState* St, SymbolReaper& SymReaper);
267
Mike Stump1eb44332009-09-09 15:08:12 +0000268 void print(const GRState* St, llvm::raw_ostream& Out,
Ted Kremenek45021952009-02-14 17:08:39 +0000269 const char* nl, const char *sep);
270
271private:
Ted Kremenek9beefec2009-02-17 19:28:04 +0000272 RangeSet::Factory F;
Ted Kremenek45021952009-02-14 17:08:39 +0000273};
274
275} // end anonymous namespace
276
Ted Kremenekf1b82272009-06-18 23:20:05 +0000277ConstraintManager* clang::CreateRangeConstraintManager(GRStateManager&) {
278 return new RangeConstraintManager();
Ted Kremenek45021952009-02-14 17:08:39 +0000279}
280
Ted Kremenek45021952009-02-14 17:08:39 +0000281const llvm::APSInt* RangeConstraintManager::getSymVal(const GRState* St,
282 SymbolRef sym) const {
Ted Kremenek9beefec2009-02-17 19:28:04 +0000283 const ConstraintRangeTy::data_type *T = St->get<ConstraintRange>(sym);
284 return T ? T->getConcreteValue() : NULL;
Ted Kremenek45021952009-02-14 17:08:39 +0000285}
286
287/// Scan all symbols referenced by the constraints. If the symbol is not alive
288/// as marked in LSymbols, mark it as dead in DSymbols.
289const GRState*
Ted Kremenekc8781382009-06-17 22:28:13 +0000290RangeConstraintManager::RemoveDeadBindings(const GRState* state,
Ted Kremenek45021952009-02-14 17:08:39 +0000291 SymbolReaper& SymReaper) {
Ted Kremenek45021952009-02-14 17:08:39 +0000292
Ted Kremenekc8781382009-06-17 22:28:13 +0000293 ConstraintRangeTy CR = state->get<ConstraintRange>();
294 ConstraintRangeTy::Factory& CRFactory = state->get_context<ConstraintRange>();
Ted Kremenek45021952009-02-14 17:08:39 +0000295
Ted Kremenek9beefec2009-02-17 19:28:04 +0000296 for (ConstraintRangeTy::iterator I = CR.begin(), E = CR.end(); I != E; ++I) {
Mike Stump1eb44332009-09-09 15:08:12 +0000297 SymbolRef sym = I.getKey();
Ted Kremenek45021952009-02-14 17:08:39 +0000298 if (SymReaper.maybeDead(sym))
299 CR = CRFactory.Remove(CR, sym);
300 }
Mike Stump1eb44332009-09-09 15:08:12 +0000301
Ted Kremenekc8781382009-06-17 22:28:13 +0000302 return state->set<ConstraintRange>(CR);
Ted Kremenek45021952009-02-14 17:08:39 +0000303}
304
Ted Kremenek9beefec2009-02-17 19:28:04 +0000305//===------------------------------------------------------------------------===
306// AssumeSymX methods: public interface for RangeConstraintManager.
307//===------------------------------------------------------------------------===/
308
309RangeSet
Ted Kremenekc8781382009-06-17 22:28:13 +0000310RangeConstraintManager::GetRange(const GRState *state, SymbolRef sym) {
311 if (ConstraintRangeTy::data_type* V = state->get<ConstraintRange>(sym))
Ted Kremenek9beefec2009-02-17 19:28:04 +0000312 return *V;
Mike Stump1eb44332009-09-09 15:08:12 +0000313
Ted Kremenek9beefec2009-02-17 19:28:04 +0000314 // Lazily generate a new RangeSet representing all possible values for the
315 // given symbol type.
Ted Kremenekc8781382009-06-17 22:28:13 +0000316 QualType T = state->getSymbolManager().getType(sym);
Mike Stump1eb44332009-09-09 15:08:12 +0000317 BasicValueFactory& BV = state->getBasicVals();
Ted Kremenek9beefec2009-02-17 19:28:04 +0000318 return RangeSet(F, BV.getMinValue(T), BV.getMaxValue(T));
319}
320
321//===------------------------------------------------------------------------===
322// AssumeSymX methods: public interface for RangeConstraintManager.
323//===------------------------------------------------------------------------===/
324
325#define AssumeX(OP)\
326const GRState*\
Ted Kremenekc8781382009-06-17 22:28:13 +0000327RangeConstraintManager::AssumeSym ## OP(const GRState* state, SymbolRef sym,\
Ted Kremeneka591bc02009-06-18 22:57:13 +0000328 const llvm::APSInt& V){\
Ted Kremenekc8781382009-06-17 22:28:13 +0000329 const RangeSet& R = GetRange(state, sym).Add##OP(state->getBasicVals(), F, V);\
Ted Kremeneka591bc02009-06-18 22:57:13 +0000330 return !R.isEmpty() ? state->set<ConstraintRange>(sym, R) : NULL;\
Ted Kremenek9beefec2009-02-17 19:28:04 +0000331}
332
333AssumeX(EQ)
334AssumeX(NE)
335AssumeX(LT)
336AssumeX(GT)
337AssumeX(LE)
338AssumeX(GE)
339
340//===------------------------------------------------------------------------===
341// Pretty-printing.
342//===------------------------------------------------------------------------===/
343
Mike Stump1eb44332009-09-09 15:08:12 +0000344void RangeConstraintManager::print(const GRState* St, llvm::raw_ostream& Out,
Ted Kremenek45021952009-02-14 17:08:39 +0000345 const char* nl, const char *sep) {
Mike Stump1eb44332009-09-09 15:08:12 +0000346
Ted Kremenek9beefec2009-02-17 19:28:04 +0000347 ConstraintRangeTy Ranges = St->get<ConstraintRange>();
Mike Stump1eb44332009-09-09 15:08:12 +0000348
Ted Kremenekdd28d002009-02-16 18:42:56 +0000349 if (Ranges.isEmpty())
350 return;
Mike Stump1eb44332009-09-09 15:08:12 +0000351
Ted Kremenekdd28d002009-02-16 18:42:56 +0000352 Out << nl << sep << "ranges of symbol values:";
Mike Stump1eb44332009-09-09 15:08:12 +0000353
Ted Kremenek9beefec2009-02-17 19:28:04 +0000354 for (ConstraintRangeTy::iterator I=Ranges.begin(), E=Ranges.end(); I!=E; ++I){
Ted Kremenekec751c42009-04-21 22:37:11 +0000355 Out << nl << ' ' << I.getKey() << " : ";
Ted Kremenek53ba0b62009-06-24 23:06:47 +0000356 I.getData().print(Out);
Ted Kremenek45021952009-02-14 17:08:39 +0000357 }
Ted Kremenek45021952009-02-14 17:08:39 +0000358}