Issue #1234 - Rewrite internals of debug facility

src/plugin/PluginClass.cpp

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2015 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 612 matching lines @@
   return S_OK;
 }
 
 // This gets called whenever there's a browser event
 // ENTRY POINT
 STDMETHODIMP CPluginClass::Invoke(DISPID dispidMember, REFIID riid, LCID lcid, WORD wFlags, DISPPARAMS* pDispParams, VARIANT* pvarResult, EXCEPINFO* pExcepInfo, UINT* puArgErr)
 {
   try
   {
     WCHAR tmp[256];
-    wsprintf(tmp, L"Invoke: %d\n", dispidMember);

[Oleksandr, 2015/03/19 03:46:12]

How about switching to swprintf_s here. Seems like an appropriate changeset for
this

[Eric, 2015/03/19 12:56:35]

I need to rebase this change set. There's no analogous statement in the current
tip; it was eliminated when the present IDispatch was split up.

+    wsprintf(tmp, L"Invoke: %d", dispidMember);
     DEBUG_GENERAL(tmp);
     switch (dispidMember)
     {
     case DISPID_WINDOWSTATECHANGED:
       {
         // #1163 should validate and convert arguments here
         return OnTabChanged(pDispParams, wFlags);
       }
 
     case DISPID_HTMLDOCUMENTEVENTS2_ONBEFOREUPDATE:
@@ skipping 1242 matching lines @@
         }
       }
     }
 
     hTabWnd = ::GetWindow(hTabWnd, GW_HWNDNEXT);
   }
 
   return hTabWnd;
 
 }