Skip to content
This repository has been archived by the owner on Apr 17, 2022. It is now read-only.

Lazy texture loading #2304

Closed
wzdev-ci opened this issue Nov 9, 2010 · 15 comments
Closed

Lazy texture loading #2304

wzdev-ci opened this issue Nov 9, 2010 · 15 comments

Comments

@wzdev-ci
Copy link
Contributor

wzdev-ci commented Nov 9, 2010

keyword_wrf_textures_tcmask resolution_fixed type_patch (an actual patch, not a request for one) | by Per


I got really fed up modifying all those pesky wrf files whenever I needed to test something in the model world. So I wrote this patch that allows you to load textures on demand, both ordinary textures and tcmask textures. With this patch, there is really no reason to list tcmask textures in the wrf files at all. You can still put a texture in the wrf to override the usual texture loaded with a model (necessary for campaign and terrain dependent base plates).


Issue migrated from trac:2304 at 2022-04-16 06:46:10 -0700

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Nov 9, 2010

Per uploaded file dieresdie1.diff (2.4 KiB)

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Nov 9, 2010

Per uploaded file example.diff (8.0 KiB)

Example of how various texture lines can be removed and the game will still work

@wzdev-ci
Copy link
Contributor Author

Buginator commented


Doesn't this encourage bad practices ? Before, at least we had a list of textures that we used, now, it can be anything, and it would make it easier to forget to add textures.

I know the resource system is crappy, and this does help with testing things, but I am just not sure if we should allow this in anything but debug builds (for testing).

@wzdev-ci
Copy link
Contributor Author

Per commented


You mean add to the revision control system, or add to the resource system? The former is trivially spotted and fixed when it happens, and the latter is a feature by design in this patch. I do not want textures to be listed in the wrf files, unless it is to override a default texture.

@wzdev-ci
Copy link
Contributor Author

Per Inge Mathisen <per@...> commented


(In Warzone2100/warzone2100@8d41118) Add proper filenames to textures for models that are missing this. Required for lazy texture loading, see #2304

@wzdev-ci
Copy link
Contributor Author

Per Inge Mathisen <per@...> changed status from new to closed

@wzdev-ci
Copy link
Contributor Author

Per Inge Mathisen <per@...> changed resolution from `` to fixed

@wzdev-ci
Copy link
Contributor Author

Per Inge Mathisen <per@...> commented


(In Warzone2100/warzone2100@9b3cf3a) Allow you to load textures on demand, both ordinary textures and tcmask textures.There is really no reason now to list tcmask textures in the wrf files at all. You can still put a texture in the wrf to override the usual texture loaded with a model (necessary for campaign and terrain dependent base plates). Closes #2304

@wzdev-ci
Copy link
Contributor Author

Per Inge Mathisen <per@...> commented


(In Warzone2100/warzone2100@f4192aa) Remove some now unnecessary texture references to stress lazy texture loading. See #2304 for more info.

@wzdev-ci
Copy link
Contributor Author

Per Inge Mathisen <per@...> commented


In Warzone2100/warzone2100@8d41118:

#CommitTicketReference repository="" revision="8d4111886b37c7e1fc3cc3768e02655468d706ab"
Add proper filenames to textures for models that are missing this. Required for lazy texture loading, see #2304

@wzdev-ci
Copy link
Contributor Author

Per Inge Mathisen <per@...> commented


In Warzone2100/warzone2100@9b3cf3a:

#CommitTicketReference repository="" revision="9b3cf3a38212e5382e3bdbed0b5cc3d2fed6d200"
Allow you to load textures on demand, both ordinary textures and tcmask textures.There is really no reason now to list tcmask textures in the wrf files at all. You can still put a texture in the wrf to override the usual texture loaded with a model (necessary for campaign and terrain dependent base plates). Closes #2304

@wzdev-ci
Copy link
Contributor Author

Per Inge Mathisen <per@...> commented


In Warzone2100/warzone2100@f4192aa:

#CommitTicketReference repository="" revision="f4192aa4624659120c5d6f0449ce1aa915817193"
Remove some now unnecessary texture references to stress lazy texture loading. See #2304 for more info.

@wzdev-ci
Copy link
Contributor Author

Per Inge Mathisen commented


Add proper filenames to textures for models that are missing this. Required for lazy texture loading, see #2304
Changeset: 8d4111886b37c7e1fc3cc3768e02655468d706ab

@wzdev-ci
Copy link
Contributor Author

Per Inge Mathisen commented


Allow you to load textures on demand, both ordinary textures and tcmask textures.There is really no reason now to list tcmask textures in the wrf files at all. You can still put a texture in the wrf to override the usual texture loaded with a model (necessary for campaign and terrain dependent base plates). Closes #2304
Changeset: 9b3cf3a38212e5382e3bdbed0b5cc3d2fed6d200

@wzdev-ci
Copy link
Contributor Author

Per Inge Mathisen commented


Remove some now unnecessary texture references to stress lazy texture loading. See #2304 for more info.
Changeset: f4192aa4624659120c5d6f0449ce1aa915817193

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant