Re: [O] [PATCH] Agenda: Fix org-agenda-bulk-toggle when point is at already marked item

[Bastien]

Hi Madan,

Madan Ramakrishnan <address@hidden> writes:

> * lisp/org-agenda.el (org-agenda-bulk-mark): truly make arg optional
> as advertised by the function

applied, thanks.

> This is my first patch for org so apologies for any inadvertent
> missteps

This is good, but please use plain text emails if possible.

Nick Dokos <address@hidden> writes:

> I presume arg can be negative or zero.

A value of arg below 1 doesn't make sense: arg drives the number of 
entries that will be marked, so users usually use this for arg > 1.
If (arg < 1) then the function should trigger an error, which it does.

Best,

-- 
 Bastien