#887 closed enhancement (wontfix)
[PATCH] Catch terrain patch vertex height for water rendering
| Reported by: | kromxp | Owned by: | philip |
|---|---|---|---|
| Priority: | Should Have | Milestone: | Alpha 7 |
| Component: | Core engine | Keywords: | patch |
| Cc: | Patch: |
Description
The proposed small patch catches the vertex heights of the patches used.
When rendering water if there is not any vertex below water level, the whole patch is skipped. For the patches that are shown, their height levels are cached and faster to access.
It also fixes the // TODO: stop using g_Game
An improvement by using this patch could be to only call the TerrainRenderer::RenderWater if any of the patches has been submitted to be rendered with any vertex below water level.
Attachments (1)
Change History (7)
by , 13 years ago
| Attachment: | catch-patch-vertex-height.patch added |
|---|
comment:1 by , 13 years ago
| Milestone: | Alpha 6 → Alpha 7 |
|---|
comment:2 by , 13 years ago
| Owner: | set to |
|---|
comment:3 by , 13 years ago
| Keywords: | patch added |
|---|
comment:4 by , 13 years ago
| Owner: | changed from to |
|---|
comment:5 by , 13 years ago
| Resolution: | → wontfix |
|---|---|
| Status: | new → closed |
r9814 changes the water renderer so I don't think this is needed any more.
comment:6 by , 8 years ago
| Keywords: | review removed |
|---|
Note:
See TracTickets
for help on using tickets.
Work is included in or made obsolete by eihrul_'s frakenpatch. See http://lee.fov120.com/0ad-frankenpatch.diff . Ticket will be closed once frankenpatch is reviewed and committed.