Skip to content

Commit

Permalink
ContentCAO: Update light of all attached entities (#9975)
Browse files Browse the repository at this point in the history
  • Loading branch information
SmallJoker authored Jun 1, 2020
1 parent f849917 commit a08d18a
Show file tree
Hide file tree
Showing 4 changed files with 22 additions and 60 deletions.
30 changes: 3 additions & 27 deletions src/client/clientenvironment.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -320,21 +320,8 @@ void ClientEnvironment::step(float dtime)
// Step object
cao->step(dtime, this);

if (update_lighting) {
// Update lighting
u8 light = 0;
bool pos_ok;

// Get node at head
v3s16 p = cao->getLightPosition();
MapNode n = this->m_map->getNode(p, &pos_ok);
if (pos_ok)
light = n.getLightBlend(day_night_ratio, m_client->ndef());
else
light = blend_light(day_night_ratio, LIGHT_SUN, 0);

cao->updateLight(light);
}
if (update_lighting)
cao->updateLight(day_night_ratio);
};

m_ao_manager.step(dtime, cb_state);
Expand Down Expand Up @@ -402,18 +389,7 @@ u16 ClientEnvironment::addActiveObject(ClientActiveObject *object)
object->addToScene(m_texturesource);

// Update lighting immediately
u8 light = 0;
bool pos_ok;

// Get node at head
v3s16 p = object->getLightPosition();
MapNode n = m_map->getNode(p, &pos_ok);
if (pos_ok)
light = n.getLightBlend(getDayNightRatio(), m_client->ndef());
else
light = blend_light(getDayNightRatio(), LIGHT_SUN, 0);

object->updateLight(light);
object->updateLight(getDayNightRatio());
return object->getId();
}

Expand Down
6 changes: 3 additions & 3 deletions src/client/clientobject.h
Original file line number Diff line number Diff line change
Expand Up @@ -41,10 +41,10 @@ class ClientActiveObject : public ActiveObject

virtual void addToScene(ITextureSource *tsrc) {}
virtual void removeFromScene(bool permanent) {}
// 0 <= light_at_pos <= LIGHT_SUN
virtual void updateLight(u8 light_at_pos) {}
virtual void updateLightNoCheck(u8 light_at_pos) {}

virtual void updateLight(u32 day_night_ratio) {}
virtual v3s16 getLightPosition() { return v3s16(0, 0, 0); }

virtual bool getCollisionBox(aabb3f *toset) const { return false; }
virtual bool getSelectionBox(aabb3f *toset) const { return false; }
virtual bool collideWithObjects() const { return false; }
Expand Down
42 changes: 15 additions & 27 deletions src/client/content_cao.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ class TestCAO : public ClientActiveObject

void addToScene(ITextureSource *tsrc);
void removeFromScene(bool permanent);
void updateLight(u8 light_at_pos);
void updateLight(u32 day_night_ratio);
v3s16 getLightPosition();
void updateNodePos();

Expand Down Expand Up @@ -254,7 +254,7 @@ void TestCAO::removeFromScene(bool permanent)
m_node = NULL;
}

void TestCAO::updateLight(u8 light_at_pos)
void TestCAO::updateLight(u32 day_night_ratio)
{
}

Expand Down Expand Up @@ -784,34 +784,22 @@ void GenericCAO::addToScene(ITextureSource *tsrc)
setNodeLight(m_last_light);
}

void GenericCAO::updateLight(u8 light_at_pos)
void GenericCAO::updateLight(u32 day_night_ratio)
{
// Don't update light of attached one
if (getParent() != NULL) {
return;
}

updateLightNoCheck(light_at_pos);

// Update light of all children
for (u16 i : m_attachment_child_ids) {
ClientActiveObject *obj = m_env->getActiveObject(i);
if (obj) {
obj->updateLightNoCheck(light_at_pos);
}
}
}

void GenericCAO::updateLightNoCheck(u8 light_at_pos)
{
if (m_glow < 0)
return;
u8 light_at_pos = 0;
bool pos_ok;

u8 li = decode_light(light_at_pos + m_glow);
v3s16 p = getLightPosition();
MapNode n = m_env->getMap().getNode(p, &pos_ok);
if (pos_ok)
light_at_pos = n.getLightBlend(day_night_ratio, m_client->ndef());
else
light_at_pos = blend_light(day_night_ratio, LIGHT_SUN, 0);

if (li != m_last_light) {
m_last_light = li;
setNodeLight(li);
u8 light = decode_light(light_at_pos);
if (light != m_last_light) {
m_last_light = light;
setNodeLight(light);
}
}

Expand Down
4 changes: 1 addition & 3 deletions src/client/content_cao.h
Original file line number Diff line number Diff line change
Expand Up @@ -236,9 +236,7 @@ class GenericCAO : public ClientActiveObject
m_visuals_expired = true;
}

void updateLight(u8 light_at_pos);

void updateLightNoCheck(u8 light_at_pos);
void updateLight(u32 day_night_ratio);

void setNodeLight(u8 light);

Expand Down

0 comments on commit a08d18a

Please sign in to comment.