From mboxrd@z Thu Jan  1 00:00:00 1970
From: Nicolas Goaziou <n.goaziou@gmail.com>
Subject: Re: org-review-schedule
Date: Fri, 25 Apr 2014 10:02:58 +0200
Message-ID: <87bnvpltu5.fsf@gmail.com>
References: <m2ppke1izt.fsf@polytechnique.org> <877g6leltk.fsf@bzg.ath.cx>
	<m2iopzdjyx.fsf@polytechnique.org> <87fvl1lx50.fsf@gmail.com>
	<m2sip1am6l.fsf@polytechnique.org>
Mime-Version: 1.0
Content-Type: text/plain
Return-path: <emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org>
Received: from eggs.gnu.org ([2001:4830:134:3::10]:33043)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <n.goaziou@gmail.com>) id 1Wdb5f-0004rc-80
	for emacs-orgmode@gnu.org; Fri, 25 Apr 2014 04:02:40 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <n.goaziou@gmail.com>) id 1Wdb5Z-0002DD-DS
	for emacs-orgmode@gnu.org; Fri, 25 Apr 2014 04:02:35 -0400
In-Reply-To: <m2sip1am6l.fsf@polytechnique.org> (Alan Schmitt's message of
	"Fri, 25 Apr 2014 09:43:46 +0200")
List-Id: "General discussions about Org-mode." <emacs-orgmode.gnu.org>
List-Unsubscribe: <https://lists.gnu.org/mailman/options/emacs-orgmode>,
	<mailto:emacs-orgmode-request@gnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/archive/html/emacs-orgmode>
List-Post: <mailto:emacs-orgmode@gnu.org>
List-Help: <mailto:emacs-orgmode-request@gnu.org?subject=help>
List-Subscribe: <https://lists.gnu.org/mailman/listinfo/emacs-orgmode>,
	<mailto:emacs-orgmode-request@gnu.org?subject=subscribe>
Errors-To: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org
Sender: emacs-orgmode-bounces+geo-emacs-orgmode=m.gmane.org@gnu.org
To: Alan Schmitt <alan.schmitt@polytechnique.org>
Cc: Bastien <bzg@gnu.org>, emacs-orgmode <emacs-orgmode@gnu.org>

Alan Schmitt <alan.schmitt@polytechnique.org> writes:

> I guess I should have asked: who decides what goes in contrib?

The Org maintainer. Another option is to turn it into an ELPA package.

> Is this mature/useful enough to be included? If so, I'll edit these as
> you suggest.

It seems useful. Maturity is not a problem since you have write access
to the repo. Anyway, Bastien will give you the definitive answer.

> After this many violations, I've added
>
> #+begin_src emacs-lisp
>   (setq-default show-trailing-whitespace t)
> #+end_src
>
> to my configuration ;-)

FWIW, I use `whitespace-mode' instead.

> It works, so I guess it's not supposed to move the point. It's to be
> used with `org-agenda-skip-function', which says:
>
> ,-----------------------------------------------------------------------
> | Function to be called at each match during agenda construction.       
> | If this function returns nil, the current match should not be skipped.
> | Otherwise, the function must return a position from where the search  
> | should be continued.                                                  
> `-----------------------------------------------------------------------

Then I suggest to explain it in the docstring (both the return value and
that it will be used as `org-agenda-skip-function' value).


Regards,

-- 
Nicolas Goaziou