Anonymous | Login | Signup for a new account | 2024-04-24 23:55 UTC |
My View | View Issues | Change Log | Roadmap | Zandronum Issue Support Ranking | Rules | My Account |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0002303 | Zandronum | [All Projects] Suggestion | public | 2015-06-11 19:22 | 2024-04-09 13:55 | ||||
Reporter | Leonard | ||||||||
Assigned To | Leonard | ||||||||
Priority | normal | Severity | feature | Reproducibility | N/A | ||||
Status | resolved | Resolution | fixed | ||||||
Platform | Microsoft | OS | Windows | OS Version | XP/Vista/7 | ||||
Product Version | 3.0-beta | ||||||||
Target Version | 3.1 | Fixed in Version | 3.1 | ||||||
Summary | 0002303: Omit color codes in name's length limit. | ||||||||
Description | <Leonard> I was wondering and just to be sure Here is a link to the referenced ticket. Since the new userinfo is in 3.0 I think this is doable. I checked in the latest GZDoom and I can change my name to a very long length. The only limit is when writing the DEM_UINFCHANGED command which zandronum doesn't even use and the name is cleaned/truncated before it's written in anyways. MAXPLAYERNAME can also be set to be actually 32 since the "\c-" color code appended at the end won't be counted. | ||||||||
Attached Files | |||||||||
Notes | |
(0012653) carpathia (reporter) 2015-06-11 20:10 |
THIS! How has it taken so long for someone to consider colour code not being counted towards name length... |
(0014413) Dusk (developer) 2016-02-10 14:52 |
This one is actually more difficult than in ZDoom since existing code inherited from Skulltag still uses fixed-size char arrays that rely on the length limit. And in any case an absolute length limit must exist because this string gets sent over the network. What I guess we could do is set a fixed name length limit (say 24 chars for instance) and an absolute length limit of 3x that (72) for color codes that is harder to actually reach, even with longer form colors such as the ones provided by NewTextColors.pk3. The former limit would thus be the actual restricting name length and the 72 char limit as a technical maximum. |
(0014415) Leonard (developer) 2016-02-10 15:41 edited on: 2016-02-10 15:47 |
I totally forgot about this ticket. I actually did this a few months ago but I forgot about it because I started working on other stuff. Basically I did exactly what you said dusk, I had the MAXPLAYERNAME define be the actual character limit and a new define "MAXPLAYERNAMECHAR" which was the string limit. I also realised the char arrays zandronum uses were problematic so I actually resized all of the ones that used the player's name in any way (by searching for each GetName() instance). I'll attach the original patch I made if that helps but it may need some testing. EDIT: Couldn't figure out how to attach on the tracker so here's a link. |
(0014424) Dusk (developer) 2016-02-11 17:41 |
Hmm, interesting. Though I wonder if we could instead expend the effort to using FStrings in the affected code. |
(0014487) Dusk (developer) 2016-02-23 22:37 |
You know what, let's just add it now 'https://bitbucket.org/Torr_Samaho/zandronum/pull-requests/71 [^]' |
(0018848) Leonard (developer) 2017-11-09 22:52 |
I'll revive this sometime in the near future. |
(0021681) Kaminsky (developer) 2021-08-08 02:06 |
In 3.1, colour codes are no longer included in the length of a player's name, meaning players can use more complex colour schemes without hitting the 32-character limit, even when using the new text colours. Ask Dusk suggested before, there's an absolute limit for how long a name can actually be, including the colour codes (96 characters long). However, this limit should be hard to reach under normal use, and primarily exists so that names aren't too big when sent across the network. |
(0023579) Ru5tK1ng (updater) 2024-04-09 13:55 |
Verified that this works correctly online. |
This issue is already marked as resolved. If you feel that is not the case, please reopen it and explain why. |
|
Supporters: | Leonard DrinkyBird carpathia unknownna ZzZombo Fused Monsterovich Combinebobnt Klofkac Marcaek Korshun The Toxic Avenger djskaarj JC Tenton MaxRideWizardLord FascistCat Catastrophe Frits |
Opponents: | Argentum |
Issue History | |||
Date Modified | Username | Field | Change |
2015-06-11 19:22 | Leonard | New Issue | |
2015-06-11 20:10 | carpathia | Note Added: 0012653 | |
2015-06-12 15:12 | Dusk | Relationship added | child of 0002268 |
2015-07-04 22:54 | MaxRideWizardLord | Note Added: 0012829 | |
2015-07-05 00:39 | Dusk | Note Deleted: 0012829 | |
2016-02-10 14:52 | Dusk | Note Added: 0014413 | |
2016-02-10 15:41 | Leonard | Note Added: 0014415 | |
2016-02-10 15:47 | Leonard | Note Edited: 0014415 | View Revisions |
2016-02-11 17:41 | Dusk | Note Added: 0014424 | |
2016-02-23 22:37 | Dusk | Note Added: 0014487 | |
2016-02-23 22:37 | Dusk | Assigned To | => Dusk |
2016-02-23 22:37 | Dusk | Status | new => needs review |
2017-11-09 22:52 | Leonard | Note Added: 0018848 | |
2017-11-09 22:52 | Leonard | Assigned To | Dusk => Leonard |
2017-11-09 22:52 | Leonard | Status | needs review => assigned |
2021-08-08 02:06 | Kaminsky | Note Added: 0021681 | |
2021-08-08 02:06 | Kaminsky | Status | assigned => needs testing |
2021-08-08 02:06 | Kaminsky | Target Version | => 3.1 |
2024-04-09 13:55 | Ru5tK1ng | Note Added: 0023579 | |
2024-04-09 13:55 | Ru5tK1ng | Status | needs testing => resolved |
2024-04-09 13:55 | Ru5tK1ng | Resolution | open => fixed |
2024-04-09 13:55 | Ru5tK1ng | Fixed in Version | => 3.1 |
Copyright © 2000 - 2024 MantisBT Team |