<br><br><div class="gmail_quote">On Sun, May 16, 2010 at 6:28 PM,  <span dir="ltr"><<a href="mailto:burban@opopop.net">burban@opopop.net</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;">
<div class="im">Anton Rizov <<a href="mailto:anton.rizov@gmail.com">anton.rizov@gmail.com</a>> writes:<br>
<br>
> Hi,<br>
><br>
> Before anything else I have a "management" question. As I'll be doing<br>
> (mostly trivial for now) changes so it would be useful to know about the<br>
> policy here. What do you prefer to ask/discuss on this list before<br>
> changing anything or directly send patches?<br>
><br>
> This time I'll ask:<br>
><br>
> * Is there any reason in-application is not exported?<br>
><br>
>   In protocol.lisp there is a macro in-application but it is not<br>
>   exported. IMHO it could be useful in client code as it will<br>
>   eliminate the need to specify :application argument in<br>
>   defentry-point.<br>
><br>
>   Any objections about exporting it?<br>
><br>
> * Is *ucw-applications-directory* in vars.lisp needed?<br>
><br>
>   I don't think this variable is actually used. Any reason to keep it?<br>
><br>
> * Is *ucw-systems* in vars.lisp needed?<br>
><br>
>   Same as *ucw-applications-directory*<br>
<br>
</div>IMHO, in-application, *ucw-applications-directory* and *ucw-systems*<br>
are not used elsewhere, so could be suppressed. *ucw-applications* is<br>
used in create-server, so should be kept to not break existing code.<br></blockquote><div><br>OK. Won't touch *ucw-applications*.  <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

<div class="im"><br>
> * How about changing default value fo create-server :application argument?<br>
><br>
> - (applications *ucw-applications*)<br>
> + (applications (or *ucw-applications* '(*default-application*))<br>
<br>
</div>What if *default-application* stays nil?<br>
<div class="im"><br></div></blockquote><div> </div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="im">
>   So than one could write:<br>
><br>
>   (in-application (make-instance 'demo-application))<br>
><br>
>   (create-server)<br>
><br>
>   rather than<br>
><br>
>   (create-server :applications '(*default-application*))<br>
<br>
</div>I prefer the later, as it clearly shows what applications are handled<br>
by what server.<br>
<div class="im"><br>
>   Or maybe changing in-application<br>
>   (pushnew *default-application* *ucw-applications*)<br>
<br>
</div>Another concern I have with 1 line defmacro such as in-application is<br>
that in most cases they don't bring clarity and simplification to the<br>
code; at the contrary, in-application hides that there is a side effect.<br>
<br></blockquote><div>I guess you're right. It may be a personal preference but it looks more clear to me to <br>write (in-application some-application) than (setf *default-application* some-application). <br>In addition I assosiate it with in-package macro.<br>
<br>So what's the verdict: Export or Remove?<br><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;">
Sincerely.<br>
<font color="#888888"><br>
--<br>
<br>
B. Urban<br></font></blockquote><div><br><br>Kind regards,<br>Anton <br></div></div><br>