diff options
| author | Tatsuhiro Tsujikawa <tatsuhiro.t@gmail.com> | 2008-04-27 02:22:14 +0000 |
|---|---|---|
| committer | Tatsuhiro Tsujikawa <tatsuhiro.t@gmail.com> | 2008-04-27 02:22:14 +0000 |
| commit | 1ef99931e131343f069a4b74edfc06196d72d85b (patch) | |
| tree | d640f061a66a193d5f4d92b61836161208ccff14 /test/DefaultBtMessageFactoryTest.cc | |
| parent | a7952cce05a41c6e20d6eddc569a8bffe974aade (diff) | |
2008-04-27 Tatsuhiro Tsujikawa <tujikawa at rednoah dot com>
Rewritten Exception class. Throw exception object, not its pointer and
catch by reference, so that remove problematic delete operator for
catched exception.
* src/Exception.cc
* src/Exception.h
* test/ExceptionTest.cc
* src/*: All files throwing/catching exception.
* test/*: All files throwing/catching exception.
Diffstat (limited to 'test/DefaultBtMessageFactoryTest.cc')
| -rw-r--r-- | test/DefaultBtMessageFactoryTest.cc | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/test/DefaultBtMessageFactoryTest.cc b/test/DefaultBtMessageFactoryTest.cc index c9259724..3de44c06 100644 --- a/test/DefaultBtMessageFactoryTest.cc +++ b/test/DefaultBtMessageFactoryTest.cc @@ -88,9 +88,8 @@ void DefaultBtMessageFactoryTest::testCreateBtMessage_BtExtendedMessage() _peer->setExtendedMessagingEnabled(false); factory.createBtMessage((const unsigned char*)msg+4, sizeof(msg)); CPPUNIT_FAIL("exception must be thrown."); - } catch(Exception* e) { - std::cerr << *e << std::endl; - delete e; + } catch(Exception& e) { + std::cerr << e.stackTrace() << std::endl; } } @@ -110,11 +109,8 @@ void DefaultBtMessageFactoryTest::testCreatePortMessage() (factory.createBtMessage(&data[4], sizeof(data)-4))); CPPUNIT_ASSERT(!m.isNull()); CPPUNIT_ASSERT_EQUAL((uint16_t)6881, m->getPort()); - } catch(Exception* e) { - std::cerr << *e << std::endl; - std::string msg = e->getMsg(); - delete e; - CPPUNIT_FAIL(msg); + } catch(Exception& e) { + CPPUNIT_FAIL(e.stackTrace()); } } { |
