zsh-workers
 help / color / mirror / code / Atom feed
* incomplete sentence in manual
@ 2015-09-28 18:12 Daniel Shahaf
  2015-09-29  1:30 ` Bart Schaefer
  0 siblings, 1 reply; 2+ messages in thread
From: Daniel Shahaf @ 2015-09-28 18:12 UTC (permalink / raw)
  To: zsh-workers

Commit 18d82a6c401758e8bd49fef9478168c68d7d2652 contains:

    @@ -170,7 +174,8 @@ pattern are loaded.
     With the tt(-w) flag, the var(name)s are taken as names of files compiled
     with the tt(zcompile) builtin, and all functions defined in them are
    -marked for autoloading.
    +marked for autoloading.  Note this does not otherwise change the search
    +order for
    
     The flags tt(-z) and tt(-k) mark the function to be autoloaded using the
     zsh or ksh style, as if the option tt(KSH_AUTOLOAD) were unset or were

Could the end of the sentence be committed too, please?


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: incomplete sentence in manual
  2015-09-28 18:12 incomplete sentence in manual Daniel Shahaf
@ 2015-09-29  1:30 ` Bart Schaefer
  0 siblings, 0 replies; 2+ messages in thread
From: Bart Schaefer @ 2015-09-29  1:30 UTC (permalink / raw)
  To: zsh-workers

On Sep 28,  6:12pm, Daniel Shahaf wrote:
} Subject: incomplete sentence in manual
}
} Commit 18d82a6c401758e8bd49fef9478168c68d7d2652 contains:
} 
}     @@ -170,7 +174,8 @@ pattern are loaded.
}      With the tt(-w) flag, the var(name)s are taken as names of files compiled
}      with the tt(zcompile) builtin, and all functions defined in them are
}     -marked for autoloading.
}     +marked for autoloading.  Note this does not otherwise change the search
}     +order for
}     
} Could the end of the sentence be committed too, please?

Oops.  I went looking for where $fpath processing was discussed in order to
decide whether it was OK to call it out here, and then got distracted with
other edits and never came back to that thought.

So either the entire fragment should be deleted (it's now covered by the
first paragraph under autoload which was added by that same commit), or
if you don't feel that's sufficient then:


diff --git a/Doc/Zsh/builtins.yo b/Doc/Zsh/builtins.yo
index 97c3370..691aaeb 100644
--- a/Doc/Zsh/builtins.yo
+++ b/Doc/Zsh/builtins.yo
@@ -175,7 +175,7 @@ pattern are loaded.
 With the tt(-w) flag, the var(name)s are taken as names of files compiled
 with the tt(zcompile) builtin, and all functions defined in them are
 marked for autoloading.  Note this does not otherwise change the search
-order for 
+order via tt(fpath) when the function is first called.
 
 The flags tt(-z) and tt(-k) mark the function to be autoloaded using the
 zsh or ksh style, as if the option tt(KSH_AUTOLOAD) were unset or were


I can go either way.


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-09-29  1:30 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-28 18:12 incomplete sentence in manual Daniel Shahaf
2015-09-29  1:30 ` Bart Schaefer

Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).