Changeset [bcb2c98d3c5464030d27ef083475eacc0adaed88] by Jocelyn Legault

December 6th, 2009 @ 04:49 AM

Debug output: improve output performance and design

  • Replace the outputF function with the DebugOutput object that allows to bundle up the outputs and flush them to the debug console only when ready.
  • It also improves the testability of debug tools that need to output as you now only have to overload the output part of the class.
  • DebugOutput comes with its set of tests.
  • The existing function guards tests have been updated to reflect the new output method.

  • Future improvements:

    • Instead of an extern variable to the global DebugOutput (yuk) we could implement a DebugOutput stack where we could push and pop DebugOutput objects. This stack would live in the application object.

[#8 state:needs_ack responsible:npp-community]

Signed-off-by: Jocelyn Legault jocelynlegault@gmail.com
http://github.com/npp-community/npp-community/commit/bcb2c98d3c5464...

Committed by Jocelyn Legault

  • M PowerEditor/src/MISC/Debug/npp_debug.cpp
  • M PowerEditor/src/MISC/Debug/npp_debug.h
  • M PowerEditor/tests/testNppDebug.cpp
New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

Notepad++ Community Release

Shared Ticket Bins