<br><br><div class="gmail_quote">On Fri, Jun 18, 2010 at 10:53 PM, Jacek Antonelli <span dir="ltr"><<a href="mailto:jacek.antonelli@gmail.com">jacek.antonelli@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
I've been looking over the commits on the weeklies, in order to<br>
backport the good ones for beta 5. I have questions about some of the<br>
commits, hopefully you guys can provide some insight:<br>
<br>
What are the reasons for these commits being reverted later? Should I<br>
keep them or omit them?<br>
<br>
  * <a href="http://github.com/mccabe/imprudence/commit/8fe81ea" target="_blank">http://github.com/mccabe/imprudence/commit/8fe81ea</a> : Changed the<br>
default minimize position to the top left so it doesn't block chat<br>
anymore<br></blockquote><div><br>Can be removed from history if you want, it was reverted because it ended up getting in the way of people's mini-maps. <br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

  * <a href="http://github.com/mccabe/imprudence/commit/cbcb2a2" target="_blank">http://github.com/mccabe/imprudence/commit/cbcb2a2</a> : Applied patch<br>
by Robin Cornelius for SNOW-693: Corrupt assets crash viewer with<br>
openjpeg<br>
<br></blockquote><div><br>Reverted because it caused a memory leak. Might be useful to keep around to compare later if the cause of the leak can be found. Seems to be sporadic (at least I didn't get it on Windows). Armin did, and other people confirmed it on the forums. <br>
 <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
What is the reason for this being reverted later, and then un-reverted<br>
again? Should I keep it?<br>
<br>
  * <a href="http://github.com/mccabe/imprudence/commit/cdd1c1b" target="_blank">http://github.com/mccabe/imprudence/commit/cdd1c1b</a> : Automatically<br>
try logging in again if 'Login Failed. You appear to be already logged<br>
in' error happens on OpenSim<br></blockquote><div><br>Reverted because opensim also has an issue that causes the viewer to "mysteriously quit" on login (what <a href="http://github.com/mccabe/imprudence/commit/5bb9fbc" target="_blank">5bb9fbc</a> was part of fixing). Unreverted when the other issue was fixed. <br>
 </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
What are the statuses of these commits? Do they help? Should I keep them?<br>
<br>
  * <a href="http://github.com/mccabe/imprudence/commit/b9798d1" target="_blank">http://github.com/mccabe/imprudence/commit/b9798d1</a> : Stops an<br>
avatar's texture from flickering on OS but leaves behind the corrupt<br>
texture until they rebake. A half-fix at best. Maybe. Revert if it<br>
causes problems<br></blockquote><div><br>Necessary until a better fix can be found. <br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

  * <a href="http://github.com/mccabe/imprudence/commit/f19d234" target="_blank">http://github.com/mccabe/imprudence/commit/f19d234</a> : Test commit:<br>
check to see what happens if we comment out the flashing fix<br>
<br>
What's the reason for this being UDPBlackListed? Is this something LL changed?<br>
<br></blockquote><div><br>It's something LL changed after 1.23 for security reasons (see Snowglobe). <br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

  * <a href="http://github.com/mccabe/imprudence/commit/8edc8b4" target="_blank">http://github.com/mccabe/imprudence/commit/8edc8b4</a> : OpenCircuit<br>
is now UDPBlackListed<br>
<br>
What's the rationale for making "Show Look At" and "Private Look At"<br>
mutually exclusive?<br>
 <br></blockquote><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
  * <a href="http://github.com/mccabe/imprudence/commit/90bd91f" target="_blank">http://github.com/mccabe/imprudence/commit/90bd91f</a> : Added menu<br>
entry for Private Look At, made it so you can't have Show Look At and<br>
Private Look At enabled at the same time <br></blockquote><div><br>The rationale comes from combat sims, where people use show look at to
cheat while masking their own show look at beacon, and because people
get creeped out by the idea that someone could be tracking where you're
looking while you can't them. For the menu entry, it's that nobody's
going to find the debug setting for PrivateLookAtTarget, let
alone even remember what it was called from the release notes. We got
positive feedback after adding the easy option to opt-out.<br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
What's the purpose of this commit? Is it ready for the mainline?<br>
<br>
  * <a href="http://github.com/mccabe/imprudence/commit/5bb9fbc" target="_blank">http://github.com/mccabe/imprudence/commit/5bb9fbc</a> : wip forcing<br>
region disconnects to log you out<br>
<br></blockquote><div><br>Can squash it with the other changes that finished this branch if you want, but this change keeps the viewer from "crashing" (forceDisconnecting) when an opensim screws up presence data after a user crashes/logs out or teleports. I'd say it's ready, it went through a lot of testing. We might want to make it so it resets the progress bar and perhaps figure out a more intuitive notification later, but it takes multiple failed login attempts before you really notice the progress bar doesn't reset back to 0 after being sent to the login screen, so I don't see that as a blocker. <br>
 </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
- Jacek<br>
_______________________________________________<br>
ImpDev mailing list<br>
<a href="mailto:ImpDev@lists.imprudenceviewer.org">ImpDev@lists.imprudenceviewer.org</a><br>
<a href="http://lists.imprudenceviewer.org/listinfo.cgi/impdev-imprudenceviewer.org" target="_blank">http://lists.imprudenceviewer.org/listinfo.cgi/impdev-imprudenceviewer.org</a><br>
</blockquote></div><br><br>-- MC<br clear="all"><br>-- <br>"Work is love made visible." — Kahlil Gibran<br><br>"We will not walk in fear, one of another. We will not be driven by fear into an age of unreason if we dig deep in our history and doctrine and remember that we are not descended from fearful men, not from men who feared to write, to speak, to associate and to defend causes which were for the moment unpopular. We can deny our heritage and our history, but we cannot escape responsibility for the result. There is no way for a citizen of the Republic to abdicate his responsibility." -- Edward R. Murrow, March 9, 1954<br>