Improves readability in HeightMapRenderObjClass::Update_Center #926
Conversation
bobtista
force-pushed
the
heightmapUpdateCenter
branch
from
2f4bc96
to
98cd3e5
Compare
|
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## develop #926 +/- ##
==========================================
- Coverage 2.97% 2.97% -0.01%
==========================================
Files 803 803
Lines 80343 80343
Branches 13764 13764
==========================================
- Hits 2390 2387 -3
- Misses 77610 77612 +2
- Partials 343 344 +1 ☔ View full report in Codecov by Sentry. |
bobtista
force-pushed
the
heightmapUpdateCenter
branch
from
98cd3e5
to
709f055
Compare
| min_x = m_map->Border_Size() + min_x; | ||
| max_x = m_map->Border_Size() + max_x; | ||
| min_y = m_map->Border_Size() + min_y; | ||
| max_y = m_map->Border_Size() + max_y; |
| min_x = min_x / 10.0f; | ||
| max_x = max_x / 10.0f; | ||
| min_y = min_y / 10.0f; | ||
| max_y = max_y / 10.0f; |
| } else { | ||
| i6 = i6 + m_x - 1; | ||
| x_start_tile = x_start_tile + m_x - 1; |
| i6 -= i1; | ||
| i7 += i1; | ||
| x_start_tile -= increment; | ||
| x_end_tile += increment; | ||
|
|
||
| if (m_originX < 0) { | ||
| m_originX = m_originX + m_x - 1; |
| } else { | ||
| i4 = i4 + m_y - 1; | ||
| y_start_tile = y_start_tile + m_y - 1; |
| float x_dist = viewport_max.X - viewport_min.X; | ||
| float y_dist = viewport_max.Y - viewport_min.Y; | ||
| float scale_factor = -1.0f; | ||
| float min_x = 200000.0f; |
There was a problem hiding this comment.
Right now there is
x_dist
min_x
among others.
Perhaps it makes sense to name those x,y variables in consistent order?
Perhaps x,y always at the end of the name?
|
|
||
| float z = max_height; | ||
| Vector2 viewport_min; | ||
| Vector2 viewport_max; |
There was a problem hiding this comment.
Perhaps just view_max for short.
| Vector2 viewport_max; | ||
| camera->Get_View_Plane(viewport_min, viewport_max); | ||
| float x_dist = viewport_max.X - viewport_min.X; | ||
| float y_dist = viewport_max.Y - viewport_min.Y; |
There was a problem hiding this comment.
Perhaps this variable can still retain view in it.
No description provided.