<div dir="ltr">notify_user sounds good to me :)</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Mar 26, 2014 at 5:36 PM, Manuel Quiñones <span dir="ltr"><<a href="mailto:manuq@laptop.org" target="_blank">manuq@laptop.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">2014-03-26 18:12 GMT-03:00 James Cameron <<a href="mailto:quozl@laptop.org">quozl@laptop.org</a>>:<br>
<div class="">> On Wed, Mar 26, 2014 at 09:40:41AM -0300, Martin Abente wrote:<br>
>> His work is almost done [2,3,4], but it introduces a new method to<br>
>> the base Activity class, therefore I wanted your opinions regarding<br>
>> the name of this method, the current options are [4]:<br>
><br>
>> * send_notification<br>
>> * add_notification<br>
>> * shell_notify<br>
>> * sugar_notify<br>
><br>
> "shell_" and "sugar_" prefixes are superfluous given that the base<br>
> Activity class is already a part of Sugar, and the Shell isn't the<br>
> eventual target of the notification; the user is.<br>
><br>
> "_notify" and "_notification" are ambiguous, because they don't say<br>
> what is being notified; the system, the class instance, or the user.<br>
><br>
> "add_" prefix exposes an implementation detail; that notifications are<br>
> a list.  This exposure is unnecessary.<br>
><br>
> Of the ones you listed, "send_notification" fits best.<br>
<br>
</div>I agree.  Thanks for the detailed comments on each proposal.<br>
<div class="im HOEnZb"><br>
><br>
> Of the ones you didn't list, "notify_user" would be best.<br>
><br>
> "sugar_shell_add_notification_and_send_to_user" would be the worst.<br>
><br>
> --<br>
> James Cameron<br>
> <a href="http://quozl.linux.org.au/" target="_blank">http://quozl.linux.org.au/</a><br>
</div><div class="HOEnZb"><div class="h5">> _______________________________________________<br>
> Sugar-devel mailing list<br>
> <a href="mailto:Sugar-devel@lists.sugarlabs.org">Sugar-devel@lists.sugarlabs.org</a><br>
> <a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
<br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
.. manuq ..<br>
</font></span></blockquote></div><br></div>