[hleV]

It's more of a request, which, if granted, would increase the performance of the callback function passed in ArraySort().

An example plugin:

#include <amxmodx>

public plugin_init()
{
    new Array:array = ArrayCreate(32);
    FillArrayWithRandomValues(array); // Irrelevant
    ArraySort(array, "OnSortArray");
    ArrayDestroy(array);
}

public OnSortArray(Array:array, item1, item2)
{
    static value1[32], value2[32];
    ArrayGetArray(array, item1, value1);
    ArrayGetArray(array, item2, value2);
    
    return GetSortReturnValue(value1, value2); // Irrelevant
}

Now everytime OnSortArray() is called, ArrayGetArray() is called twice. It becomes a very expensive operation when there are thousands of entries in the array to sort.

What I want to suggest is to directly pass the values of currently sorted array items rather than their positions in the array. So it would become:

public OnSortArray(Array:array, const value1[], const value2[])
{
    return GetSortReturnValue(value1, value2); // Irrelevant
}

That should be possible, right? One array can only hold either cells, strings or arrays, so there wouldn't be problems with deciding what kind of parameters to pass into the callback.

This would be useful for both AMX Mod X & SourceMod.

[YamiKaitou]

Then why not go submit this as a Feature Request in the Bug Tracker?

[hleV]

Fair enough.

Though I can't seem to find this Feature Request. Everything there seems to imply that I'm reporting a bug.

[YamiKaitou]

In some sense you are reporting a bug, as the feature is missing.