You are not logged in.
Lost Password?


Register To Post



 Bottom   Previous Topic   Next Topic

#41
Re: Updated Reality Boy
Posted on: 2010/2/18 2:52
PVB Elite
Joined 2003/7/25
USA
1507 Posts
PVBCC 1stCoderContributor#3 PosterHOTY09 EntryLong Time User (15 Years) App CoderPVBCC 2010 EntryPVBCC 2013 Entry
There could be a file name length issue... IIRC, the files are referenced by a 255 byte string, so if you've got a really long path, it may hit the limit. And I'm not sure why it doesn't output command arguments (I haven't looked at the code yet), but the debug version does output it (reality_boyD.exe). I can't say that all the arguments are verified to be correct, but I don't know that any are wrong.

DogP
Top

#42
Re: Updated Reality Boy
Posted on: 2010/2/18 3:40
PVB Elite
Joined 2003/7/26
USA
1465 Posts
PVBCC EntryCoderContributorSpecial AchievementTop10 PosterHOTY09 EntryLong Time User (15 Years) App Coder20+ Game RatingsPVBCC 2013 Entry
Quote:

DogP wrote:
There could be a file name length issue... IIRC, the files are referenced by a 255 byte string, so if you've got a really long path, it may hit the limit.


Actually, in my version of the front-end at least, the file paths (emu and ROM) are stored in 2048-byte strings. But, I kinda goofed because the command-line options (minus the emu executable path) is stored in a 512-byte string... :doh:

Microsoft's docs for ShellExecute don't say if there is a limit to the lengths of strings passed to it, so I'm going to increase it to 4K or so and test it out with a really long path.

EDIT: I increased the path variable size and also now use _snprintf() so there shouldn't be anymore buffer overruns even if the size is exceeded (it'll just cut off the last few options). I tested it on:

"C:\Documents and Settings\\My Documents\Really Big, Long, and also Huge Test Path\Which is also nested quite deeply\And is used to store only a single Virtual Boy ROM Image\Space Invaders Virtual Collection (J) [!].vb" and it worked a treat

Attached you should find a new executable to test.

Attach file:


exe rboy_front.exe Size: 96.00 KB; Hits: 890
Edited by RunnerPack on 2010/2/18 4:00
Top

#43
Re: Updated Reality Boy
Posted on: 2010/2/18 4:00
PVB Elite
Joined 2003/7/25
USA
1507 Posts
PVBCC 1stCoderContributor#3 PosterHOTY09 EntryLong Time User (15 Years) App CoderPVBCC 2010 EntryPVBCC 2013 Entry
I mean inside Reality Boy... I don't remember if the ROM path is handled differently, but I'm pretty sure the SRAM path is 255 characters (and so is my ROM map, since I just copied a lot of the SRAM code for that).

DogP
Top

#44
Re: Updated Reality Boy
Posted on: 2010/2/18 21:00
VUE(xpert)
Joined 2010/2/15
Netherlands
250 Posts
Long Time User (9 Years)
Yup, that worked, at least for the front-end side of things!

At least, I think it worked... I'm running the app through Wine on Linux right now instead of Windows 7... Not sure if Wine would've handled it properly before. Probably not.
Top

#45
Re: Updated Reality Boy
Posted on: 2010/4/11 23:58
VUE(xpert)
Joined 2010/2/15
Netherlands
250 Posts
Long Time User (9 Years)
It's been a while since this thread was updated... I'm thinking, could someone add this new release of Virtual Boy to the emulators section? Since there's some improvements in emulation and the last release of RB was in 2008, it seems significant enough to me to update the section.
Top

 Top   Previous Topic   Next Topic


Register To Post