Flooding this mailing list again to say I've renamed history.erl as group_history.erl. this should significantly decrease the chances of name clashes, respects the kernel application's naming convention and shows the dependency on the group module.<br>

<br><div class="gmail_quote">On Mon, Nov 21, 2011 at 6:11 PM, Fred Hebert <span dir="ltr"><<a href="mailto:mononcqc@gmail.com">mononcqc@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<div>I thought of this. I can definitely change it for kernel_history, but 'shell_history' wouldn't make sense given: a) the module is in the 'kernel' application, b) the 'shell' module is actually in the 'stdlib' application. This one operates on 'group', which runs within kernel.<br>


</div><div><br></div><div>My decision was made from the idea that whatever usually is in OTP ends up having no prefix, although user-created applications tend to prefix themselves. I thus relied on the vague (and pretty much guaranteed to be wrong) assumption that people would name their own libraries reasonably using prefixes fitting their own application names.</div>


<div><br></div><div>I can definitely rename the module if people feel it's too much of a danger, though.</div><div class="HOEnZb"><div class="h5"><br><div class="gmail_quote">On Mon, Nov 21, 2011 at 6:05 PM, Kostis Sagonas <span dir="ltr"><<a href="mailto:kostis@cs.ntua.gr" target="_blank">kostis@cs.ntua.gr</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>On 11/21/11 20:31, Fred Hebert wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Oh yeah, in case anyone is wondering how invasive of a patch this is,<br>
this is the diff between the default R14B04 group.erl file and the new one:<br>
<br>
35c35<br>
< put(line_buffer, proplists:get_value(line_buffer, Options,<br>
history:load())),<br>
---<br>
 > put(line_buffer, proplists:get_value(line_buffer, Options, [])),<br>
647d646<br>
< history:add(Line),<br>
<br>
Literally two lines changed, and it should not harm the current<br>
behaviour of the shell other than loading history.<br>
</blockquote>
<br></div></div>
The real evasiveness is of course not in these two lines but in that the name of the new module ('history') which may clash with similarly named files in many user applications.<br>
<br>
Perhaps a name like 'shell_history' is more appropriate for the new module?<br>
<br>
Kostis<br>
_______________________________________________<br>
erlang-questions mailing list<br>
<a href="mailto:erlang-questions@erlang.org" target="_blank">erlang-questions@erlang.org</a><br>
<a href="http://erlang.org/mailman/listinfo/erlang-questions" target="_blank">http://erlang.org/mailman/listinfo/erlang-questions</a><br>
</blockquote></div><br>
</div></div></blockquote></div><br>