dMZX Forums: String comparison broken for strings of different length -> Archived MegaZeux Bugs -> Tracker

Jump to content

Report ID 714 Title String comparison broken for strings of different length
Product Archived MegaZeux Bugs Status Fixed (Severity 3 - Medium)
Version 2.91c Fixed in 2.91e
Introduced In Version 2.80XOperating System All platforms

Page 1 of 1
  • Cannot start a new Issue
  • Closed Issue This issue is locked

Issue Timeline Issue Timeline
Report ID #714: String comparison broken for strings of different length

#1 User is offline  
Lachesis 

  • the pinnacle of human emotion
  • Group: DigiStaff
  • Posts: 3,943
  • Joined: 17-July 04
  • Gender:Female
  • Location:Sealand

Posted 25 February 2018 - 10:40 PM

MegaZeux has used very sketchy string comparison for a long time. While endianness issues were fixed in 2.91b, it still favors string length as a comparing factor over string contents. This means that built-in string comparison can not be used for e.g. sorting algorithms unless every string being sorted is the same length.

This probably needs to a version check, so a fix will have to wait until 2.92.
"Let's just say I'm a GOOD hacker, AND virus maker. I'm sure you wouldn't like to pay for another PC would you?"

xx̊y (OST) - HELLQUEST (OST) - Zeux I: Labyrinth of Zeux (OST) (DOS OST)
w/ Lancer-X and/or asgromo: Pandora's Gate - Thanatos Insignia - no True(n) - For Elise OST
MegaZeux: Online Help File - Keycode Guide - Joystick Guide - Official GIT Repository

Page 1 of 1  
  • Cannot start a new Issue
  • Closed Issue This issue is locked

Replies (1 - 2)

#2 User is offline  
Lachesis 

  • the pinnacle of human emotion
  • Group: DigiStaff
  • Posts: 3,943
  • Joined: 17-July 04
  • Gender:Female
  • Location:Sealand

Posted 03 September 2018 - 10:02 AM

Updating status to: Fixed
Issue fixed in: 2.91e

This was fixed by GreaseMonkey in GIT 107a822e. I went ahead and pulled it for 2.91e.
"Let's just say I'm a GOOD hacker, AND virus maker. I'm sure you wouldn't like to pay for another PC would you?"

xx̊y (OST) - HELLQUEST (OST) - Zeux I: Labyrinth of Zeux (OST) (DOS OST)
w/ Lancer-X and/or asgromo: Pandora's Gate - Thanatos Insignia - no True(n) - For Elise OST
MegaZeux: Online Help File - Keycode Guide - Joystick Guide - Official GIT Repository

#3 User is offline  
Terryn 

  • ******
  • Group: DigiStaff
  • Posts: 2,966
  • Joined: 12-October 00
  • Gender:Male

Posted 17 September 2018 - 04:59 AM

Moving to: Archived MegaZeux Bugs

Page 1 of 1
  • Cannot start a new Issue
  • Closed Issue This issue is locked

0 User(s) are reading this issue
0 Guests and 0 Anonymous Users


Powered by IP.Tracker 1.3.2 © 2025  IPS, Inc.