Bug 3553 - speech.ini limited to 64 sounds
speech.ini limited to 64 sounds
Status: RESOLVED FIXED
Product: AMX Mod X
Classification: Unclassified
Component: Base Plugins
trunk
PC All
: P3 normal
Assigned To: amxmodx-bugs@alliedmods.net
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-01-08 23:41 PST by connorr
Modified: 2014-02-08 23:51 PST (History)
2 users (show)

See Also:


Attachments
Proposed patch (284 bytes, patch)
2010-12-20 11:53 PST, Reuben Morais
no flags Details | Diff | Splinter Review

Description connorr 2009-01-08 23:41:51 PST
Hello,

When you edit line 75 in cmdmenu.sma :

#define MAX_CMDS    32

You can not put a number higher than 64.

Fix :

Change line 90 :

new g_menuSelect[33][64]

With

new g_menuSelect[33][MAX_CMDS]
Comment 1 Reuben Morais 2010-12-20 11:53:09 PST
Created attachment 2480 [details] [review]
Proposed patch
Comment 2 Vincent Herbet [:Arkshine] 2010-12-21 09:28:39 PST
Such fix has been given numerous times on the forum, and of course, works as expected.
Comment 3 David Anderson [:dvander] 2011-03-25 16:14:16 PDT
Comment on attachment 2480 [details] [review]
Proposed patch

We should up MAX_CMDS to 64 at the same time, otherwise this could break config files.
Comment 4 Reuben Morais 2013-02-13 00:09:37 PST
Pushed changeset: http://hg.alliedmods.net/amxmodx-central/rev/e9e18a2b11da
Changelog:
	Increase the capacity of speech.ini (bug 3553, r=dvander).
Comment 5 David Anderson [:dvander] 2013-02-13 00:10:40 PST
Pushed changeset: http://hg.alliedmods.net/amxmodx-central/rev/b3798f1e902e
Changelog:
	bug 3553 follow-up fix.
Comment 6 AM Bugzilla Bot 2014-02-08 23:51:32 PST
Commits pushed to master at https://github.com/alliedmodders/amxmodx

https://github.com/alliedmodders/amxmodx/commit/9e5b5683afd930722eb2ad98958d82932f27c299
Increase the capacity of speech.ini (bug 3553, r=dvander).


Former-commit-id: d10ef5e49419251d3c841148093fb3210bc6d028

https://github.com/alliedmodders/amxmodx/commit/69c06545cedf2f3cf40de5af1138c4cdf8007392
bug 3553 follow-up fix.


Former-commit-id: 65d97fe5f5bca0c3a3ba8574f7c924d8ba7dc1cd

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