LDraw.org Discussion Forums
LDView 4.2 Beta 1 Released - Printable Version

+- LDraw.org Discussion Forums (https://forums.ldraw.org)
+-- Forum: LDraw Programs (https://forums.ldraw.org/forum-7.html)
+--- Forum: LDraw Editors and Viewers (https://forums.ldraw.org/forum-11.html)
+--- Thread: LDView 4.2 Beta 1 Released (/thread-4278.html)

Pages: 1 2 3 4 5 6 7 8 9 10 11 12


Re: LDView 4.2 Beta 1 Released - Tim Gould - 2012-05-08

Maybe keep it for files with .dat extension but strip it for models. I believe parts should still be whitespace free.

Tim


Re: LDView 4.2 Beta 1 Released - Travis Cobbs - 2012-05-08

It looks like I had enough left on my computer from old installs (I upgraded it from Tiger to Snow Leopard, I think) to possibly make this work. Please try the build here:

http://www.halibut.com/~tcobbs/ldraw/private/LDViewU-20120507.tar.bz2


Re: LDView 4.2 Beta 1 Released - Travis Cobbs - 2012-05-08

I think I like that. Plus, it makes my life easier, since it would take a fair bit of extra code to prevent the warning from showing up in the list of selected errors and warnings. (I can't just remove it from the list, because that would change the selected ones after it.) But do you think .dat is the best switch? Or should it be:
  • In LDraw/Parts, LDraw/Unofficial, or LDraw/p directory, or any subdirectory of the same OR
  • Contains part-specifying meta (like !LDRAW_ORG OFFICIAL_PART) OR
  • ? (I'm open for other suggestions.)



Re: LDView 4.2 Beta 1 Released - Tim Gould - 2012-05-08

My suggestion was based purely on ease of implementation not robustness. Choosing by directory would probably be the best. If it's in any of those directories it really oughtn't have whitespace.

Tim


Re: LDView 4.2 Beta 1 Released - Daniel Goerner - 2012-05-08

What's a whitespace?


Re: LDView 4.2 Beta 1 Released - Philippe Hurbain - 2012-05-08

Ascii code 0x20, character you get when you hit space bar on your keyboard, blank space between words, etc Wink


Re: LDView 4.2 Beta 1 Released - Daniel Goerner - 2012-05-08

U-hu, and when is that considered to be an error?


Re: LDView 4.2 Beta 1 Released - Travis Cobbs - 2012-05-08

Currently, LDView generates a warning when it encounters white space in the filename of an LDraw file in a type 1 line. When LDView 4.1 was released, this was perfectly reasonable, because even though LDView could load such files (using the same algorithm as MLCAD), the LDraw spec didn't mention how that should be done.

Since then, the LDraw spec has been updated to standardize the algorithm MLCAD uses, so warning about white spaces in filenames is no longer really a good idea. However, DAT files in the official library are not allowed to use spaces in their filenames, so I plan to go with Tim's suggestion, and leave the warning in, but only when processing a part, sub-part, or primitive.


Re: LDView 4.2 Beta 1 Released - Travis Cobbs - 2012-05-09

Can someone with a PPC Mac please give the above build a try and let me know if it works? If I don't get any feedback on it, I probably won't include PPC in the official release.


Re: LDView 4.2 Beta 1 Released - MoonMan - 2012-05-11

I'm sorry I didn't write back sooner. I didn't check the forums because I wasn't expecting a response. It works great. Thank you very much for taking the time to accomodate me!