[haiku-commits] Re: BRANCH axeld-github.imap [91f396f] in src: add-ons/mail_daemon/inbound_protocols/imap/imap_lib tests/add-ons/mail/imap add-ons/mail_daemon/inbound_protocols/imap

  • From: John Scipione <jscipione@xxxxxxxxx>
  • To: "haiku-commits@xxxxxxxxxxxxx" <haiku-commits@xxxxxxxxxxxxx>
  • Date: Wed, 29 May 2013 16:47:30 -0400

On Wed, May 29, 2013 at 12:15 PM, axeld-github.imap
<community@xxxxxxxxxxxx> wrote:
> +                       bool                            fUIDs;

Definitely a nitpick, but, shouldn't the variable name be a verb since
it's a bool, something like fFetchUIDs or fShouldFetchUIDs rather than
just UIDs? To some it might look it is fUIDs references a list of UIDs
rather than a flag.

Other related posts: