search

QW-Group / mvdsv

MVDSV: a QuakeWorld server
GNU General Public License v2.0
59 stars 56 forks source link

[BUG] sv_demoremove *token doesn't work properly #134

Open tarbugre opened 5 months ago

tarbugre commented 5 months ago

Describe the bug using sv_demoremove with token doesn't remove all demos with that token

To Reproduce Steps to reproduce the behavior:

1- have a big list of demos (3000+) 2- type rcon sv_demoremove ]240315 to remove demos from that date 3- type rcon sv_demoremove ]240315 to remove demos from that date again

Expected behavior all demos from that date are removed on the first command

What actually happens The command removes a few demos, type again, and it removes a few more, and a few more get left behind

Environment(please complete the following information):

Operation System: [e.g. Windows, Linux, ...]: Ubuntu 20.04.3 LTS" Processor Architecture: [e.g. x86, x86_64, arm7, ...] x86-64 MVDSV Version: mvdsv 0.35 antilag r115 KTX Version: 1.41 beta antilag r115

Additional context

ciscon commented 4 months ago

please check after this commit: https://github.com/QW-Group/mvdsv/commit/4bdd739eacf00d739713d09696ecd11b85d95a92 if you have more than 4096 demos it still may take multiple tries to get all of them, but it shouldn't be as many, it can only deal with that many in a go.