<div dir="ltr"><div class="gmail_extra"></div><div class="gmail_extra"><div class="gmail_extra">The code looks ok, but a bit stange.</div><div class="gmail_extra"><br></div><div class="gmail_extra"><table class="" style="border-collapse:collapse;border-spacing:0px;color:rgb(51,51,51);font-family:Helvetica,arial,nimbussansl,liberationsans,freesans,clean,sans-serif,'Segoe UI Emoji','Segoe UI Symbol';font-size:13px;line-height:18.2000007629395px"><tbody><tr><td id="LC4471" class="" style="font-family:Consolas,'Liberation Mono',Menlo,Courier,monospace;padding:0px 10px;vertical-align:top;font-size:12px;white-space:pre;overflow:visible;word-wrap:normal">            <span class="" style="color:rgb(167,29,93)">if</span> (sp == <span class="" style="color:rgb(0,134,179)">null</span>)
            <span style="line-height:18.2000007629395px">m_log.WarnFormat(
            </span><span class="" style="line-height:18.2000007629395px;color:rgb(24,54,145)"><span class="">"</span>[SCENE PRESENCE]: Did not find presence with id {0} in {1} before timeout<span class="">",
            <span style="color:rgb(51,51,51);line-height:18.2000007629395px">agentID, RegionInfo.RegionName);

I would rather write this on one line:

</span><div class="gmail_extra" style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:small;line-height:normal;white-space:normal"><table class="" style="border-collapse:collapse;border-spacing:0px;color:rgb(51,51,51);font-family:Helvetica,arial,nimbussansl,liberationsans,freesans,clean,sans-serif,'Segoe UI Emoji','Segoe UI Symbol';font-size:13px;line-height:18.2000007629395px"><tbody><tr><td id="LC4471" class="" style="font-family:Consolas,'Liberation Mono',Menlo,Courier,monospace;padding:0px 10px;vertical-align:top;font-size:12px;white-space:pre;overflow:visible;word-wrap:normal">            <span class="" style="color:rgb(167,29,93)">if</span> (sp == <span class="" style="color:rgb(0,134,179)">null</span>) <span style="line-height:18.2000007629395px">m_log.WarnFormat("</span><span class="" style="line-height:18.2000007629395px;color:rgb(24,54,145)">[SCENE PRESENCE]: Did not find presence with id {0} in {1} before timeout<span class="">", <span style="color:rgb(51,51,51);line-height:18.2000007629395px">agentID, RegionInfo.RegionName);

Or break it up in multiple lines like this: (with on every line an other parameter if it makes the code better readable)<br></span><div class="gmail_extra" style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:small;line-height:normal;white-space:normal"><br></div><div class="gmail_extra" style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:small;line-height:normal;white-space:normal"><table class="" style="border-collapse:collapse;border-spacing:0px;color:rgb(51,51,51);font-family:Helvetica,arial,nimbussansl,liberationsans,freesans,clean,sans-serif,'Segoe UI Emoji','Segoe UI Symbol';font-size:13px;line-height:18.2000007629395px"><tbody><tr><td id="LC4471" class="" style="font-family:Consolas,'Liberation Mono',Menlo,Courier,monospace;padding:0px 10px;vertical-align:top;font-size:12px;white-space:pre;overflow:visible;word-wrap:normal">            <span class="" style="color:rgb(167,29,93)">if</span> (sp == <span class="" style="color:rgb(0,134,179)">null</span>) {
                   <span style="line-height:18.2000007629395px">m_log.WarnFormat(
                          </span><span class="" style="line-height:18.2000007629395px;color:rgb(24,54,145)"><span class="">"</span>[SCENE PRESENCE]: Did not find presence with id {0} in {1} before timeout<span class="">",
                          <span style="color:rgb(51,51,51);line-height:18.2000007629395px">agentID, 
                          RegionInfo.RegionName);
            }


</span></span></span></td></tr></tbody></table></div><span style="color:rgb(51,51,51);line-height:18.2000007629395px">
</span></span></span></td></tr></tbody></table></div></span></span></td></tr></tbody></table></div></div></div>