Hi Madan,
Madan Ramakrishnan 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.
Madan Ramakrishnan wrote:
> Hi Nick,
> Ah, of course, you're right; it would result in a change in behavior.
> While I can certainly tweak the patch to maintain the old behavior
> in the case of a negative or zero arg, stepping back, I don't quite
> understand the reasoning for the current
Hi Nick,
Ah, of course, you're right; it would result in a change in behavior.
While I can certainly tweak the patch to maintain the old behavior in the
case of a negative or zero arg, stepping back, I don't quite understand the
reasoning for the current behavior. If a user has gone to the tro
Madan Ramakrishnan wrote:
> * lisp/org-agenda.el (org-agenda-bulk-mark): truly make arg optional
> as advertised by the function
>
> Problem here was that org-agenda-bulk-toggle calls org-agenda-bulk-mark
> with no parameters; however, the (max arg 1) call inside
> org-agenda-bulk-mark
> will fa
* lisp/org-agenda.el (org-agenda-bulk-mark): truly make arg optional
as advertised by the function
Problem here was that org-agenda-bulk-toggle calls org-agenda-bulk-mark
with no parameters; however, the (max arg 1) call inside
org-agenda-bulk-mark
will fail with no parameter. Change the max to a