On Mon, Aug 17, 2009 at 5:45 PM, Matthew Mondor<mm_lists@...> wrote:
> Also, I tested slime with CVS HEAD ECL with the attached slime diff and
> it works fine again for me. However this diff would probably have to
> check against ext:+ecl-version-number+ ideally if it was to be part of
> official slime so that it keeps working with old releases?
I actually modified your patch for ECL precisely because it was not
backwards compatible in the naming of buffering modes -- it is a bad
idea to introduce just new features for the sake of it without
preserving the little code that is nowadays ECL-specific.
My concern right now is more towards people that do not download the
latest slime, or use default software distributions and do not need /
want to apply patches. I am pretty sure that many ECL users are
already bothered enough by incompatible changes so that we introduce
even more of them.
Setting the default buffering to NIL (:NONE, synonyms actually) will
only break code that does not supply a default value for the buffering
-- which is in any case a bad idea if your code _does_ rely on a
certain buffering type -- and would keep current Slime users happy.
In the mean time your patch can be committed and once we make sure
this funny clashing of expectations is no longer a problem I can redo
the patch to the default buffer type.
Instituto de Física Fundamental, CSIC
c/ Serrano, 113b, Madrid 28006 (Spain)