From ba211ad9ad43e69f1bb19b00bb0de5ae446750af Mon Sep 17 00:00:00 2001
From: scrawl <scrawl@baseoftrash.de>
Date: Thu, 12 Nov 2015 23:27:34 +0100
Subject: [PATCH] Read NiPointLight (Fixes #3011)

---
 components/nif/effect.cpp  | 27 ++++++++++++--------------
 components/nif/effect.hpp  | 39 ++++++++++++++++++++++++--------------
 components/nif/niffile.cpp |  1 +
 3 files changed, 38 insertions(+), 29 deletions(-)

diff --git a/components/nif/effect.cpp b/components/nif/effect.cpp
index 41dcb09de..78feeea94 100644
--- a/components/nif/effect.cpp
+++ b/components/nif/effect.cpp
@@ -5,29 +5,19 @@
 namespace Nif
 {
 
-void NiLight::SLight::read(NIFStream *nif)
+void NiLight::read(NIFStream *nif)
 {
+    NiDynamicEffect::read(nif);
+
     dimmer = nif->getFloat();
     ambient = nif->getVector3();
     diffuse = nif->getVector3();
     specular = nif->getVector3();
 }
 
-void NiLight::read(NIFStream *nif)
-{
-    Effect::read(nif);
-
-    nif->getInt(); // 1
-    nif->getInt(); // 1?
-    light.read(nif);
-}
-
 void NiTextureEffect::read(NIFStream *nif)
 {
-    Effect::read(nif);
-
-    int tmp = nif->getInt();
-    if(tmp) nif->getInt(); // always 1?
+    NiDynamicEffect::read(nif);
 
     /*
            3 x Vector4 = [1,0,0,0]
@@ -52,10 +42,17 @@ void NiTextureEffect::read(NIFStream *nif)
 
 void NiTextureEffect::post(NIFFile *nif)
 {
-    Effect::post(nif);
+    NiDynamicEffect::post(nif);
     texture.post(nif);
 }
 
+void NiPointLight::read(NIFStream *nif)
+{
+    NiLight::read(nif);
 
+    constantAttenuation = nif->getFloat();
+    linearAttenuation = nif->getFloat();
+    quadraticAttenuation = nif->getFloat();
+}
 
 }
diff --git a/components/nif/effect.hpp b/components/nif/effect.hpp
index fae1cd7f5..75de0a9ab 100644
--- a/components/nif/effect.hpp
+++ b/components/nif/effect.hpp
@@ -29,27 +29,38 @@
 namespace Nif
 {
 
-typedef Node Effect;
-
-// Used for NiAmbientLight and NiDirectionalLight. Might also work for
-// NiPointLight and NiSpotLight?
-struct NiLight : Effect
+struct NiDynamicEffect : public Node
 {
-    struct SLight
+    void read(NIFStream *nif)
     {
-        float dimmer;
-        osg::Vec3f ambient;
-        osg::Vec3f diffuse;
-        osg::Vec3f specular;
+        Node::read(nif);
+        unsigned int numAffectedNodes = nif->getUInt();
+        for (unsigned int i=0; i<numAffectedNodes; ++i)
+            nif->getUInt(); // ref to another Node
+    }
+};
+
+// Used as base for NiAmbientLight, NiDirectionalLight, NiPointLight and NiSpotLight.
+struct NiLight : NiDynamicEffect
+{
+    float dimmer;
+    osg::Vec3f ambient;
+    osg::Vec3f diffuse;
+    osg::Vec3f specular;
 
-        void read(NIFStream *nif);
-    };
-    SLight light;
+    void read(NIFStream *nif);
+};
+
+struct NiPointLight : public NiLight
+{
+    float constantAttenuation;
+    float linearAttenuation;
+    float quadraticAttenuation;
 
     void read(NIFStream *nif);
 };
 
-struct NiTextureEffect : Effect
+struct NiTextureEffect : NiDynamicEffect
 {
     NiSourceTexturePtr texture;
 
diff --git a/components/nif/niffile.cpp b/components/nif/niffile.cpp
index 1387a97b0..ab0af8f99 100644
--- a/components/nif/niffile.cpp
+++ b/components/nif/niffile.cpp
@@ -82,6 +82,7 @@ static std::map<std::string,RecordFactoryEntry> makeFactory()
     newFactory.insert(makeEntry("NiFlipController",           &construct <NiFlipController>            , RC_NiFlipController              ));
     newFactory.insert(makeEntry("NiAmbientLight",             &construct <NiLight>                     , RC_NiLight                       ));
     newFactory.insert(makeEntry("NiDirectionalLight",         &construct <NiLight>                     , RC_NiLight                       ));
+    newFactory.insert(makeEntry("NiPointLight",               &construct <NiPointLight>                , RC_NiLight                       ));
     newFactory.insert(makeEntry("NiTextureEffect",            &construct <NiTextureEffect>             , RC_NiTextureEffect               ));
     newFactory.insert(makeEntry("NiVertWeightsExtraData",     &construct <NiVertWeightsExtraData>      , RC_NiVertWeightsExtraData        ));
     newFactory.insert(makeEntry("NiTextKeyExtraData",         &construct <NiTextKeyExtraData>          , RC_NiTextKeyExtraData            ));