uiop/stream:call-with-temporary-file docstring incorrect

Bug #1604008 reported by pipping
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ASDF
Fix Released
Undecided
Unassigned

Bug Description

The docstrings for

  uiop/stream:call-with-temporary-file and
  uiop/stream:stream:with-temporary-file

both mention that `prefix` defaults to "uiop". This does not seem to be the case. In fact, `prefix` does not have a default according to the signature of `uiop/stream:call-with-temporary-file`.

In practice, however (and this is what the documentation should probably say instead), the function behaves almost as if the default was "tmp" since the expression `(or prefix "tmp")` is used to construct the variable `prefix-pn`, which is the only value influenced by prefix. The key difference between an actual default value of `prefix` and this behaviour is that the current implementation turns even an explicitly passed `nil` into "tmp".

Revision history for this message
Faré (fahree) wrote :

Thanks! Fixed in 176ab37.

Changed in asdf:
status: New → Fix Committed
pipping (pipping)
Changed in asdf:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.