From mboxrd@z Thu Jan  1 00:00:00 1970
From: Nicolas Goaziou <mail@nicolasgoaziou.fr>
Subject: Re: [PATH] Speedups to org-table-recalculate
Date: Sat, 06 Dec 2014 00:57:37 +0100
Message-ID: <87sigt1wge.fsf@nicolasgoaziou.fr>
References: <CAPrg3HAnBUysnoeRWWU61uN5eEjp4B-mrOE5TCfNBisTi5+tbA@mail.gmail.com>
	<CALn3zogUBcy3VZPLb7zc3N9j8ibECi41Wbar6Hw_7xiOUvA5VA@mail.gmail.com>
	<CAPrg3HALf6UDzpFvoqEV2u1Z7vHqV4r7m5v0KxyTod05m_Rf7A@mail.gmail.com>
	<CALn3zoiqUfGR6G24YDqH69ZOdiAHX72yqJ+6Xdq+ELmv2n0qmg@mail.gmail.com>
	<CAPrg3HDSAH_n4GhSQpCrv19hjqCVA3TM5VJgdj1Jhzwa+JQNyA@mail.gmail.com>
	<CALn3zogsgiN9KkgkgGKH6wy6sO0g6U+MMAht252rviBDNh0YwQ@mail.gmail.com>
	<CAPrg3HCEd7VJyyo1z9860cHuqReQLsWrEWHgq7erLiEc4oTYxg@mail.gmail.com>
	<CALn3zojScmbeU8ON6r-wyYXJJNbfuj2BbvN=co9FSxHqnoqihA@mail.gmail.com>
	<CAPrg3HDm-cz=pDz21A=Cf9OnWGV07J7CSx3NzuVd=W+MeAYE3w@mail.gmail.com>
	<CALn3zoi9E5C78NOzNxAXBRi7vU8jJkSOHeYLUM_9CMABSSrRSg@mail.gmail.com>
	<CAPrg3HDKL+oJu-6XZijhUqasWa434N8U6AGFzcUVY6-+ukpU+w@mail.gmail.com>
	<CALn3zoi9aTqLN01vXSzgPAq_diSwLEHcvfTtv7CaYxLe44VVcA@mail.gmail.com>
	<CAPrg3HA0Kv-VigNdY=PnhkWEmNbGcPC3K-T1-vj3B=pgh+43ZQ@mail.gmail.com>
	<CALn3zoj3ZNSZvPob7gkT3SB_ORhN6pC51nOv-k5yXkuaU=Ennw@mail.gmail.com>
	<CAPrg3HDkyRZPo1J4OMD11r-KvJ_e7-ZXdBh0Jc4Uq1iH38s29A@mail.gmail.com>
	<CALn3zogNTthgi0xHf3ijjEENBMKiLZuOFfp_06a=7ap34GfLyA@mail.gmail.com>
	<87oas9tnlm.fsf@selenimh.mobile.lan>
	<CAPrg3HDmcfWbdWrfSg6ZWHfDnBpCjW21xqcHNp_0o4eZMZnm1A@mail.gmail.com>
	<87oas03ze0.fsf@nicolasgoaziou.fr>
	<CAPrg3HB4So=FbLnXQOOKKUg_rk2ibuv5u61fQOgJ10h6owqUTw@mail.gmail.com>
	<CAPrg3HCwgxr-=D5S1USUeG5canLDHb3XmFiBYWfT3v8YVxtLxw@mail.gmail.com>
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]:55199)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <mail@nicolasgoaziou.fr>) id 1Xx2k1-0004MC-Gd
	for emacs-orgmode@gnu.org; Fri, 05 Dec 2014 18:57:03 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <mail@nicolasgoaziou.fr>) id 1Xx2jr-0004zy-AZ
	for emacs-orgmode@gnu.org; Fri, 05 Dec 2014 18:56:53 -0500
Received: from relay5-d.mail.gandi.net ([2001:4b98:c:538::197]:46093)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <mail@nicolasgoaziou.fr>) id 1Xx2jr-0004zq-4W
	for emacs-orgmode@gnu.org; Fri, 05 Dec 2014 18:56:43 -0500
In-Reply-To: <CAPrg3HCwgxr-=D5S1USUeG5canLDHb3XmFiBYWfT3v8YVxtLxw@mail.gmail.com>
	(Nathaniel Flath's message of "Mon, 1 Dec 2014 01:15:20 -0500")
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: Nathaniel Flath <flat0103@gmail.com>
Cc: Michael Brand <michael.ch.brand@gmail.com>, org-mode List <emacs-orgmode@gnu.org>

Nathaniel Flath <flat0103@gmail.com> writes:

> Sorry, that was incorrect - real patches attached.

Thanks.

> +(defun org-table-message-once-per-second (t1 &rest args)
> +  "If there has been more than one second since T1, display message.
> +ARGS are passed as arguments to the 'message' function.  Returns
> +current time if a message is printed, otherwise returns t1..  If
> +T1 is nil, always messages."
> +  (let ((curtime (current-time)))
> +    (when (or (not t1) (< 0 (nth 1 (time-subtract curtime t1))))
> +      (apply message args)
> +      curtime))
> +    t1)

The docstring seems incorrect, as the function always returns T1, no
matter if a message is printed or not.

> +	    (setq log-last-time
> +		  (org-table-message-once-per-second
> +		   (when all log-last-time)

Nitpick: (and all log-last-time)

> +		 (when all log-last-time)

Ditto.
> +         (when all log-first-time)

Ditto.


Regards,