<div dir="ltr">This patch moves some member init into the class constructor initialisation lists.<div><br></div><div>Rational: somewhat cleaner, better style, and considered to produce better performance.</div><div><br></div><div>Note that I'm not exactly familiar with the patch submission project, or how to respond to mailing list threads. So, I figured I would start with something small.<br><div><br></div><div><div>diff --git a/src/amd/addrlib/core/addrobject.cpp b/src/amd/addrlib/core/addrobject.cpp</div><div>index 369be8c..a5edc14 100644</div><div>--- a/src/amd/addrlib/core/addrobject.cpp</div><div>+++ b/src/amd/addrlib/core/addrobject.cpp</div><div>@@ -59,8 +59,8 @@ AddrObject::AddrObject()</div><div> ***************************************************************************************************</div><div> */</div><div> AddrObject::AddrObject(const AddrClient* pClient)</div><div>+    : m_client(*pClient)</div><div> {</div><div>-    m_client = *pClient;</div><div> }</div><div> </div><div> /**</div><div>diff --git a/src/compiler/glsl/ir.cpp b/src/compiler/glsl/ir.cpp</div><div>index cb195b2..1101966 100644</div><div>--- a/src/compiler/glsl/ir.cpp</div><div>+++ b/src/compiler/glsl/ir.cpp</div><div>@@ -1399,10 +1399,8 @@ ir_swizzle::ir_swizzle(ir_rvalue *val, const unsigned *comp,</div><div> }</div><div> </div><div> ir_swizzle::ir_swizzle(ir_rvalue *val, ir_swizzle_mask mask)</div><div>-   : ir_rvalue(ir_type_swizzle)</div><div>+   : ir_rvalue(ir_type_swizzle), val(val), mask(mask)</div><div> {</div><div>-   this->val = val;</div><div>-   this->mask = mask;</div><div>    this->type = glsl_type::get_instance(val->type->base_type,</div><div> <span class="gmail-Apple-tab-span" style="white-space:pre">                                     </span>mask.num_components, 1);</div><div> }</div></div></div></div>