<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><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">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 class="HOEnZb"><div class="h5">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>