View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0001997 | FSSCP | Platform-Engine interaction | public | 2009-09-21 04:36 | 2012-01-23 23:06 |
| Reporter | FUBAR-BDHR | Assigned To | The_E | ||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Product Version | 3.6.11 | ||||
| Summary | 0001997: WARNING: "Passed filename <name>, is too long to support an extention" problem | ||||
| Description | I know this check was added recently but there seems to be a problem with it. I am getting messages like this: WARNING: "Passed filename, 'col_viper_trainer_deb-normal', is too long to support an extension!! Maximum length, minus the extension, is 27 characters." at bmpman.cpp:792 Which would seem to be a good catch except col_viper_trainer_deb-normal.dds does not exist and none of the other file names it produces do either. There are other textures on the model with normal maps but not this one. It also shows the warning twice even though the ship is only present once in the test mission. Longest filename is 31 characters including the . and extension. | ||||
| Additional Information | Recent 3.6.11 builds. | ||||
| Tags | No tags attached. | ||||
|
|
This is getting ridiculous now it's doing it for briefing .ani files. Max length 22 character? Since when do they have normal maps and frames? If you exceed the 31 character limit then toss an error not because you might. |
|
|
Shine, normal, and other texture effects are loaded by appending suffixes to the file names. Thus, the game is going to look for col_viper_trainer_deb-normal to see if there are any normal maps for col_viper_trainer_deb. This is just how texture names work. :-/ Perhaps we should add a check for a max length of 20 characters (texture name plus "-normal") in addition to the current length check. |
|
|
So, are we going to do anything about this? Personally, I believe we shouldn't; Even if the error message is erroneous in this case, it's still valid and needed. |
|
|
Fix committed to trunk@8327. |
|
|
argh |
|
|
Fix committed to fs2_open_3_6_14@8358. |
|
fs2open: trunk r8327 2012-01-22 20:25 The_E Ported: N/A Details Diff |
Fix for Mantis 1997: "Filename too long" warning for nonexistant textures. |
Affected Issues 0001997 |
|
| mod - /trunk/fs2_open/code/model/modelinterp.cpp | Diff File | ||
|
fs2open: fs2_open_3_6_14 r8358 2012-01-23 18:06 Ported: N/A Details Diff |
Backport: Trunk 8327 + 8341; Fix for Mantis 1997: 'Filename too long' warning for nonexistant textures. && Followup to 8327: Texture names from pofs are provided without extension, so we have to make sure to account for it. |
Affected Issues 0001997 |
|
| mod - /branches/fs2_open_3_6_14/code/model/modelinterp.cpp | Diff File | ||
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2009-09-21 04:36 | FUBAR-BDHR | New Issue | |
| 2009-10-17 01:13 | FUBAR-BDHR | Note Added: 0011213 | |
| 2009-11-02 00:20 | Goober5000 | Note Added: 0011218 | |
| 2011-06-24 04:49 | Zacam | Category | --------- => Platform-Engine interaction |
| 2011-06-27 12:51 | The_E | Note Added: 0012729 | |
| 2012-01-23 01:25 | Changeset attached | => fs2open trunk r8327 | |
| 2012-01-23 01:25 | guest | Note Added: 0013114 | |
| 2012-01-23 01:25 | guest | Status | new => resolved |
| 2012-01-23 01:25 | guest | Resolution | open => fixed |
| 2012-01-23 04:14 | Goober5000 | Assigned To | => The_E |
| 2012-01-23 04:14 | Goober5000 | Status | resolved => assigned |
| 2012-01-23 04:14 | Goober5000 | Note Added: 0013116 | |
| 2012-01-23 04:14 | Goober5000 | Status | assigned => resolved |
| 2012-01-23 23:06 | chief1983 | Changeset attached | => fs2open fs2_open_3_6_14 r8358 |
| 2012-01-23 23:06 | chief1983 | Note Added: 0013136 |
