<div dir="ltr">Thanks again for the input :)<div><br></div><div>So, I'll definitely change 'name' back to 'id'. However, regarding 'intensity' and 'period' I think I found the reason why these names were used in the first place. These are the names used internally in the supervisor module (for variables names and fields in the state record), but more importantly, they are used in error messages - 'invalid_intensity', 'invalid_period' (and 'reached_max_restart_intensity'... :-P ). Considering the backwards compatibility we do not want to change these error messages, so now I'm back to thinking that maybe 'intensity' and 'period' are not so bad after all... ?? Or rather, there are pros and cons for everything, but maybe this would give the most consistent feel to the complete interface.</div><div><br></div><div>/siri</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">2014-09-03 18:33 GMT+02:00 Ulf Wiger <span dir="ltr"><<a href="mailto:ulf@feuerlabs.com" target="_blank">ulf@feuerlabs.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><br><div><div>On 03 Sep 2014, at 13:50, Siri Hansen <<a href="mailto:erlangsiri@gmail.com" target="_blank">erlangsiri@gmail.com</a>> wrote:</div><br><blockquote type="cite"><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">Over the summer we have had some internal reviews of the documentation and I got some feedback on the naming. Specifically, questions are raised about the keys 'name', 'intensity' and 'period'. With the old tuple format, these three values are named Id, MaxR and MaxT respectively, and the questions are if they should be changed at all and if so, are the suggested names "the best"…</div></blockquote><br></div><div>Ah, how did I miss this?</div><div><br></div><div>Time for me to mention, again, the supervisor extension I made back in 2001:</div><div><br></div><div><a href="http://erlang.2086793.n4.nabble.com/Supervisor-shutdown-td4663655.html" target="_blank">http://erlang.2086793.n4.nabble.com/Supervisor-shutdown-td4663655.html</a></div><div><br></div><div>Just as some added input. ;-)</div><div><br></div><div>BR,</div><div>Ulf W</div><div><br></div><div>
<span style="border-collapse:separate;border-spacing:0px"><div><div>Ulf Wiger, Co-founder & Developer Advocate, Feuerlabs Inc.</div><div><a href="http://feuerlabs.com" target="_blank">http://feuerlabs.com</a></div></div><div><br></div></span><br>

</div>
<br></div></blockquote></div><br></div>