diff options
author | Vas Crabb <vas@vastheman.com> | 2020-04-16 05:05:16 +1000 |
---|---|---|
committer | Vas Crabb <vas@vastheman.com> | 2020-04-16 14:04:06 +1000 |
commit | 1a5c013c330fd1a6b3d21d9bbbaf512539ae4503 (patch) | |
tree | b3ff6b9f247bdb2f721346a6adae7157d0a66c90 /hash/msx1_cart.xml | |
parent | 94ddd2d427700c5802e00adda0d3373941d06bba (diff) |
(nw) softlist_dev.cpp: validate ROM labels
The free-for-all on labels in software lists is not working. There's no
consistency, labels are getting excessively long, people are starting to
use non-ASCII characters in labels making it harder for others to type
them when manipulating files on the command line, and there's too much
markup being put in labels.
The length limit is 127 characters, same as for labels in MAME itself.
This should be long enough to be descriptive. Remember that the Win32
path limit is 260 characters, and many applications and frameworks have
issues with longer paths, including Windows Explorer and the .NET
framework. Labels are used as filenames, so concessions need to be
made for this.
I have not abbreviated excessively long labels myself - they're
currently causing 135 validity errors. Someone else can fix them.
Printable ASCII characters are allowed, with a few exceptions. The
exceptions are limited to characters most likely to cause issues for
interactive shells and scripts:
* ! - csh event substitution (very difficult to escape properly)
* $ - sh varibale expansion
* % - csh job control, cmd variable expansion
* / - UNIX directory separator
* : - sh path separator, Windows drive qualifier
* \ - sh escape, Windows directory separator
Most of the labels that had to be edited were using ! for markup, or
using ! and % for titles in labels. Strangely, titles in labels are
often forced to lower case, despite this never being enforced for
software lists. There are also various other edits to titles used for
labels, such as moving articles to the end (with or without a comma),
or replacing spaces with underscores. As I already said, there's no
consistency at all.
There is far too much markup in labels. They're even being used for
notes in some cases (e.g. at least one case where a dumper's name is in
the label). The XML schema supports metadata - use it. For example,
you can use part_id for an unrestricted display name for a software
part. You can also use XML comments for notes.
And while on the topic of metadata, vgmplay.xml is putting the same
thing in the part_id as well as the label. The part_id should have
the actual title, not the title mangled to make it more suitable for
use as a filename. Addressing this would be a lot of work, given how
large the file is.
For now, empty data areas in software lists cause a verbose message
rather than a validation warning. There are thousands of software
lists using empty data areas to indicate the size/width of cartridge
RAM/EEPROM/etc.
Diffstat (limited to 'hash/msx1_cart.xml')
-rw-r--r-- | hash/msx1_cart.xml | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/hash/msx1_cart.xml b/hash/msx1_cart.xml index 81c03a80040..42255a56f87 100644 --- a/hash/msx1_cart.xml +++ b/hash/msx1_cart.xml @@ -1346,7 +1346,7 @@ kept for now until finding out what those bytes affect... <info name="alt_title" value="アドベン・チュー太" /> <part name="cart" interface="msx_cart"> <dataarea name="rom" size="16384"> - <rom name="adven' chuta! (japan).rom" size="16384" crc="96e02df3" sha1="ae0c0b0c9468e137eba5b52f96241acd55d9746a" offset="0" /> + <rom name="adven' chuta (japan).rom" size="16384" crc="96e02df3" sha1="ae0c0b0c9468e137eba5b52f96241acd55d9746a" offset="0" /> </dataarea> </part> </software> @@ -2794,7 +2794,7 @@ kept for now until finding out what those bytes affect... <info name="alt_title" value="シーソー" /> <part name="cart" interface="msx_cart"> <dataarea name="rom" size="16384"> - <rom name="c-so! (japan).rom" size="16384" crc="df8bfc89" sha1="cd69ce70747bc0deebba9fd307a07f5dfb926cc3" offset="0" /> + <rom name="c-so (japan).rom" size="16384" crc="df8bfc89" sha1="cd69ce70747bc0deebba9fd307a07f5dfb926cc3" offset="0" /> </dataarea> </part> </software> @@ -3478,7 +3478,7 @@ kept for now until finding out what those bytes affect... <info name="alt_title" value="カモン!ピコ" /> <part name="cart" interface="msx_cart"> <dataarea name="rom" size="32768"> - <rom name="come on! picot (japan).rom" size="32768" crc="98b1cc99" sha1="1cf39e12313ff3064353c136262f176a56ab20a6" offset="0" /> + <rom name="come on picot (japan).rom" size="32768" crc="98b1cc99" sha1="1cf39e12313ff3064353c136262f176a56ab20a6" offset="0" /> </dataarea> </part> </software> @@ -6167,7 +6167,7 @@ kept for now until finding out what those bytes affect... <info name="usage" value="Requires a machine with RAM in a main slot" /> <part name="cart" interface="msx_cart"> <dataarea name="rom" size="16384"> - <rom name="hustle! chumy (japan).rom" size="16384" crc="99518a12" sha1="e06ed9b0b3e1fbb9f27be65c37b4d9b548889b4f" offset="0" /> + <rom name="hustle chumy (japan).rom" size="16384" crc="99518a12" sha1="e06ed9b0b3e1fbb9f27be65c37b4d9b548889b4f" offset="0" /> </dataarea> </part> </software> @@ -6179,7 +6179,7 @@ kept for now until finding out what those bytes affect... <info name="alt_title" value="ハッスルチュミー" /> <part name="cart" interface="msx_cart"> <dataarea name="rom" size="16384"> - <rom name="hustle! chumy (japan) (alt 1).rom" size="16384" crc="22fd4780" sha1="86e688cea0f0c8bcd31b6bcd19fd72ac108646bb" offset="0" /> + <rom name="hustle chumy (japan) (alt 1).rom" size="16384" crc="22fd4780" sha1="86e688cea0f0c8bcd31b6bcd19fd72ac108646bb" offset="0" /> </dataarea> </part> </software> @@ -9723,7 +9723,7 @@ kept for now until finding out what those bytes affect... <info name="alt_title" value="ピットフォール" /> <part name="cart" interface="msx_cart"> <dataarea name="rom" size="65536"> - <rom name="pitfall! (japan).rom" size="16384" crc="5a009c55" sha1="b88e9c548873dcfd190e0e38f7b279344eea41ec" offset="0" /> + <rom name="pitfall (japan).rom" size="16384" crc="5a009c55" sha1="b88e9c548873dcfd190e0e38f7b279344eea41ec" offset="0" /> <rom size="16384" offset="0x4000" loadflag="reload" /> <rom size="16384" offset="0x8000" loadflag="reload" /> <rom size="16384" offset="0xc000" loadflag="reload" /> @@ -9739,7 +9739,7 @@ kept for now until finding out what those bytes affect... <info name="alt_title" value="ピットフォール" /> <part name="cart" interface="msx_cart"> <dataarea name="rom" size="65536"> - <rom name="pitfall! (japan) (alt 1).rom" size="16384" crc="930aeb2c" sha1="5fb4b6c3735e4d9415565a856bb69f9fb4857161" offset="0" /> + <rom name="pitfall (japan) (alt 1).rom" size="16384" crc="930aeb2c" sha1="5fb4b6c3735e4d9415565a856bb69f9fb4857161" offset="0" /> <rom size="16384" offset="0x4000" loadflag="reload" /> <rom size="16384" offset="0x8000" loadflag="reload" /> <rom size="16384" offset="0xc000" loadflag="reload" /> @@ -9755,7 +9755,7 @@ kept for now until finding out what those bytes affect... <info name="alt_title" value="ピットフォール" /> <part name="cart" interface="msx_cart"> <dataarea name="rom" size="65536"> - <rom name="pitfall! (japan) (alt 2).rom" size="16384" crc="2cb24473" sha1="2fa9c0f016efc2d1752a272c632393f5063ea06c" offset="0" /> + <rom name="pitfall (japan) (alt 2).rom" size="16384" crc="2cb24473" sha1="2fa9c0f016efc2d1752a272c632393f5063ea06c" offset="0" /> <rom size="16384" offset="0x4000" loadflag="reload" /> <rom size="16384" offset="0x8000" loadflag="reload" /> <rom size="16384" offset="0xc000" loadflag="reload" /> @@ -12014,7 +12014,7 @@ kept for now until finding out what those bytes affect... <info name="alt_title" value="띠띠!빵빵!" /> <part name="cart" interface="msx_cart"> <dataarea name="rom" size="32768"> - <rom name="ti ti! pang pang!.rom" size="32768" crc="2808a3bc" sha1="00ffd7adfc0a7a342912344ad64902e226b3ad41" offset="0x0000" /> + <rom name="ti ti pang pang.rom" size="32768" crc="2808a3bc" sha1="00ffd7adfc0a7a342912344ad64902e226b3ad41" offset="0x0000" /> </dataarea> </part> </software> @@ -12025,7 +12025,7 @@ kept for now until finding out what those bytes affect... <publisher>Jaleco</publisher> <part name="cart" interface="msx_cart"> <dataarea name="rom" size="16384"> - <rom name="top roller! (japan).rom" size="16384" crc="fc609730" sha1="37c85041128ee21300327726b7300a20ef6b84dd" offset="0" /> + <rom name="top roller (japan).rom" size="16384" crc="fc609730" sha1="37c85041128ee21300327726b7300a20ef6b84dd" offset="0" /> </dataarea> </part> </software> |