Re: [PATCH 1/3] test: add functions to count how much times notmuch was called

Subject: Re: [PATCH 1/3] test: add functions to count how much times notmuch was called

Date: Tue, 29 Nov 2011 09:40:16 -0500

To: Dmitry Kurochkin, Tomi Ollila, notmuch@notmuchmail.org

Cc:

From: Jameson Graef Rollins

On Tue, 29 Nov 2011 01:26:39 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:
> > echo \$((count + 1)) > "$notmuch_counter_state_path"
> 
> I do not think this is really an optimization.  And I find expr more
> clear than using $(()).  I always have troubles remembering "random
> special char syntax" (yeah, not a Perl fan :)), prefer human friendly
> words.

This allows the shell to do the math itself, without calling a
subprocess.  I agree with Tomi that $(()) is a little cleaner.

jamie.

• Previous message (by thread): Re: [PATCH v4 0/3] emacs: do not call notmuch show for non-inlinable parts

Thread:

• Dmitry Kurochkin—[PATCH 1/3] test: add functions to count how much times notmuch was called [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Dmitry Kurochkin—[PATCH 2/3] test: check that Emacs UI does not call notmuch for non-inlinable parts [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Austin Clements—Re: [PATCH 2/3] test: check that Emacs UI does not call notmuch for non-inlinable parts [inbox, notmuch::review, unread]
  • Dmitry Kurochkin—[PATCH 3/3] emacs: do not call notmuch show for non-inlinable parts [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Austin Clements—Re: [PATCH 3/3] emacs: do not call notmuch show for non-inlinable parts [inbox, notmuch::review, unread]
  • Austin Clements—Re: [PATCH 1/3] test: add functions to count how much times notmuch was called [inbox, notmuch::review, unread]
    • Dmitry Kurochkin—Re: [PATCH 1/3] test: add functions to count how much times notmuch was called [inbox, unread]
  • Dmitry Kurochkin—[PATCH 0/3] emacs: do not call notmuch show for non-inlinable parts [inbox, unread]
    • Dmitry Kurochkin—[PATCH 1/3] test: add functions to count how much times notmuch was called [inbox, notmuch::obsolete, notmuch::patch, notmuch::test, unread]
      • Tomi Ollila—Re: [PATCH 1/3] test: add functions to count how much times notmuch was called [inbox, unread]
        • Dmitry Kurochkin—Re: [PATCH 1/3] test: add functions to count how much times notmuch was called [inbox, unread]
          • Tomi Ollila—Re: [PATCH 1/3] test: add functions to count how much times notmuch was called [inbox, unread]
            • Dmitry Kurochkin—Re: [PATCH 1/3] test: add functions to count how much times notmuch was called [inbox, unread]
              • Tomi Ollila—Re: [PATCH 1/3] test: add functions to count how much times notmuch was called [inbox, unread]
          • Jameson Graef Rollins—Re: [PATCH 1/3] test: add functions to count how much times notmuch was called [inbox, signed, unread]
    • Dmitry Kurochkin—[PATCH 2/3] test: check that Emacs UI does not call notmuch for non-inlinable parts [inbox, notmuch::obsolete, notmuch::patch, notmuch::test, unread]
    • Dmitry Kurochkin—[PATCH 3/3] emacs: do not call notmuch show for non-inlinable parts [inbox, notmuch::obsolete, notmuch::patch, unread]
    • Austin Clements—Re: [PATCH 0/3] emacs: do not call notmuch show for non-inlinable parts [inbox, notmuch::review, unread]
      • Jameson Graef Rollins—Re: [PATCH 0/3] emacs: do not call notmuch show for non-inlinable parts [inbox, notmuch::review, signed, unread]
  • Dmitry Kurochkin—[PATCH v3 0/3] emacs: do not call notmuch show for non-inlinable parts [inbox, unread]
    • Dmitry Kurochkin—[PATCH v3 1/3] test: add functions to count how much times notmuch was called [inbox, notmuch::obsolete, notmuch::patch, notmuch::test, unread]
    • Dmitry Kurochkin—[PATCH v3 2/3] test: check that Emacs UI does not call notmuch for non-inlinable parts [inbox, notmuch::obsolete, notmuch::patch, notmuch::test, unread]
    • Dmitry Kurochkin—[PATCH v3 3/3] emacs: do not call notmuch show for non-inlinable parts [inbox, notmuch::obsolete, notmuch::patch, unread]
  • Dmitry Kurochkin—[PATCH v4 0/3] emacs: do not call notmuch show for non-inlinable parts [inbox, unread]
    • Dmitry Kurochkin—[PATCH v4 1/3] test: add functions to count how much times notmuch was called [inbox, notmuch::patch, notmuch::pushed, notmuch::test, unread]
      • Tomi Ollila—Re: [PATCH v4 1/3] test: add functions to count how much times notmuch was called [inbox, unread]
    • Dmitry Kurochkin—[PATCH v4 2/3] test: check that Emacs UI does not call notmuch for non-inlinable parts [inbox, notmuch::patch, notmuch::pushed, notmuch::test, unread]
    • Dmitry Kurochkin—[PATCH v4 3/3] emacs: do not call notmuch show for non-inlinable parts [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
    • Dmitry Kurochkin—Re: [PATCH v4 0/3] emacs: do not call notmuch show for non-inlinable parts [inbox, unread]
    • David Bremner—Re: [PATCH v4 0/3] emacs: do not call notmuch show for non-inlinable parts [inbox, unread]