Bug 3649 - Add plugin version to debug trace information.
Add plugin version to debug trace information.
Status: RESOLVED FIXED
Product: AMX Mod X
Classification: Unclassified
Component: Core
trunk
PC All
: P3 minor
Assigned To: amxmodx-bugs@alliedmods.net
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-02-18 21:15 PST by Brad
Modified: 2013-06-25 08:03 PDT (History)
3 users (show)

See Also:


Attachments
Proposed patch (1.52 KB, text/plain)
2013-06-17 06:38 PDT, Vincent Herbet [:Arkshine]
hv.contact: review+
joropito: review+
Details

Description Brad 2009-02-18 21:15:29 PST
If it isn't too hard to implement, it would make debugging problems with plugins easier as when users provide the debug information, you would immediately know which version of your plugin they were running.

I'm referring to this type of debug information:

L 12/26/2008 - 14:09:12: Invalid cellvector handle provided (7:-102:430)
L 12/26/2008 - 14:09:12: [AMXX] Displaying debug trace (plugin "galileo.amxx")
L 12/26/2008 - 14:09:12: [AMXX] Run time error 10: native error (native "ArrayGetString")
L 12/26/2008 - 14:09:12: [AMXX]    [0] galileo.sma::map_nominate (line 1206)
L 12/26/2008 - 14:09:12: [AMXX]    [1] galileo.sma::nomination_handleMatchChoice (line 1099)

Having the version listed along with the plugin name would be awesome.
Comment 1 oliver.lutter 2009-03-17 17:20:58 PDT
Hi there, ive been dealing with the same error. The plugin is currently in debug mode, i will post more information as i gather it.

I have reason to believe that this error comes from one or both of the data files created by this plugin. One error log i have from AMXX (latest version) tells me it happens on every map the plugin randomly switched too until it finally crashed the server. I commented galileo.amxx out to get it back online with no avail. BUT after i deleted the two data files created in the AMX directory it booted right up.

Hopefully this information can help you out brad, will post more when i have it.
Comment 2 Vincent Herbet [:Arkshine] 2013-06-17 06:38:12 PDT
Created attachment 3368 [details]
Proposed patch

I'm willing to add this. 

- I see such immediate information useful in different situations, avoiding extra steps.
- It's easy to implement.

Tested, and looks good.
Comment 3 joropito 2013-06-17 08:50:01 PDT
Comment on attachment 3368 [details]
Proposed patch

I don't see any problem.
Seems useful and well implemented.
Comment 4 Vincent Herbet [:Arkshine] 2013-06-25 07:59:42 PDT
Pushed changeset: http://hg.alliedmods.net/amxmodx-central/rev/8923c858240f
Changelog:
	Add plugin version to debug trace information (bug 3649, r=joropito)

Note You need to log in before you can comment on or make changes to this bug.