From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Kelvin White Newsgroups: gmane.emacs.bugs Subject: bug#17755: 24.3; ERC user mode support Date: Wed, 18 Jun 2014 14:51:22 -0400 Message-ID: <87wqcef3v9.wl%kelvin.white77@gmail.com> References: <87wqcnag57.fsf@localhost.i-did-not-set--mail-host-address--so-tickle-me> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=US-ASCII X-Trace: ger.gmane.org 1403126793 13154 80.91.229.3 (18 Jun 2014 21:26:33 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Wed, 18 Jun 2014 21:26:33 +0000 (UTC) Cc: Lawrence Mitchell , Michael Olson , 17755@debbugs.gnu.org, Mario Lang , Diane Murray , Alex Schroeder , Kelvin White , Julien Danjou , Francis Litterio , Jorgen Schaefer To: Stefan Monnier Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Wed Jun 18 23:26:23 2014 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1WxNN8-0003bh-Ph for geb-bug-gnu-emacs@m.gmane.org; Wed, 18 Jun 2014 23:26:23 +0200 Original-Received: from localhost ([::1]:60798 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WxNN8-0002jL-CW for geb-bug-gnu-emacs@m.gmane.org; Wed, 18 Jun 2014 17:26:22 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:52902) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WxKxs-0001p2-9S for bug-gnu-emacs@gnu.org; Wed, 18 Jun 2014 14:52:13 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WxKxn-0004yL-9i for bug-gnu-emacs@gnu.org; Wed, 18 Jun 2014 14:52:08 -0400 Original-Received: from debbugs.gnu.org ([140.186.70.43]:60953) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WxKxm-0004yD-Ur for bug-gnu-emacs@gnu.org; Wed, 18 Jun 2014 14:52:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.80) (envelope-from ) id 1WxKxm-0002oi-Me for bug-gnu-emacs@gnu.org; Wed, 18 Jun 2014 14:52:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Kelvin White Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Wed, 18 Jun 2014 18:52:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 17755 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 17755-submit@debbugs.gnu.org id=B17755.140311749910793 (code B ref 17755); Wed, 18 Jun 2014 18:52:02 +0000 Original-Received: (at 17755) by debbugs.gnu.org; 18 Jun 2014 18:51:39 +0000 Original-Received: from localhost ([127.0.0.1]:52103 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1WxKxO-0002nx-KS for submit@debbugs.gnu.org; Wed, 18 Jun 2014 14:51:39 -0400 Original-Received: from mail-yk0-f174.google.com ([209.85.160.174]:61801) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1WxKxL-0002ni-Rj for 17755@debbugs.gnu.org; Wed, 18 Jun 2014 14:51:36 -0400 Original-Received: by mail-yk0-f174.google.com with SMTP id 19so902995ykq.33 for <17755@debbugs.gnu.org>; Wed, 18 Jun 2014 11:51:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=date:message-id:from:to:cc:subject:in-reply-to:references :user-agent:mime-version:content-type; bh=TaUS0FMscCwP0gujOD81E0pmuObd2o3IEqGZ9ko0z4U=; b=v78zP8AlrzVy+KL/Ul4hRGwoyRjjUj3UuSHM8j5JvYd2eMj4EfLP4kKe/5dzVB7HpD TGET4OLoSqeAXNSc5Ig0lS3VSwtfOk6kqL/ql4bcIfLfD3/ZJL1R9Nu4bVcyQ2+mbBK8 Nwj4hTLCcPlcQzZRhz+NuUHKUoESZoOaKlXUhjEyhjbpG5rnJYsRpogpYpBAqFU5iH5C uWvOF3cv6ON6+HJJKpiZvckEhZ1muu/AcKCWZhtwzFa+UmPmno49B5su03txFt4S/1Xi dlz3y5SYxYUJfpMOlo8XgP7+AzRkcS93iIINNmhYYnorRbr5zpdlWZtNhjUUX99+0lZ9 g0gg== X-Received: by 10.236.18.195 with SMTP id l43mr918387yhl.150.1403117490219; Wed, 18 Jun 2014 11:51:30 -0700 (PDT) Original-Received: from localhost.gmail.com ([2600:3c02::f03c:91ff:fe73:3fa6]) by mx.google.com with ESMTPSA id 49sm2166418yhu.26.2014.06.18.11.51.28 for (version=TLSv1.2 cipher=RC4-SHA bits=128/128); Wed, 18 Jun 2014 11:51:29 -0700 (PDT) In-Reply-To: User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM/1.14.9 (=?UTF-8?Q?Goj=C5=8D?=) APEL/10.8 EasyPG/1.0.0 Emacs/24.3 (x86_64-pc-linux-gnu) MULE/6.0 (HANACHIRUSATO) X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.15 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x X-Received-From: 140.186.70.43 X-Mailman-Approved-At: Wed, 18 Jun 2014 17:26:14 -0400 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.bugs:90524 Archived-At: At Wed, 18 Jun 2014 14:32:09 -0400, Stefan Monnier wrote: > > >> > + (defsubst erc-channel-user-owner-p (nick) > >> > + "Return t if NICK is an owner of the current channel." > >> Usually we say "non-nil" rather than "t", unless the callers need to > >> rely on the return value being t rather than some other non-nil value. > > Indeed, the callers do rely on the value being t in this case. rather than > > just non-nil > > Hmm? I see only one call to erc-channel-user-owner-p and the result is > passed to `cond' so any non-nil value will work as well (which is the > way it should be for functions named `-p'). > You are right, this could be non-nil. I was looking at `erc-channel-user-owner' not the predicate form `erc-channel-user-owner-p'. > > + 2014-06-11 Kelvin White > > + > > + * erc-backend.el: Handle user modes in relevent server responses > > + * erc.el: Better user mode support. > > + (erc-channel-user): Add members for new modes. > > + (erc-channel-member-halfop-p, erc-channel-user-admin-p, erc-channel-user-owner-p): Use new struct members. > > + (erc-format-nick, erc-format-@nick): Display user modes as nick prefix. > > + (erc-nick-prefix-face, erc-my-nick-prefix-face): Add new faces to separate colors if desired. > > + (erc-get-user-mode-prefix): Return symbol for mode prefix. > > + (erc-update-channel-member, erc-update-current-channel-member, erc-channel-receive-names): Update channel users. > > + (erc-nick-at-point): Return correct user info. > > + > Ah yes, I did change the rest of the code to under 80 columns, but it looks like I missed the changelog. Sorry, it's been one of those days. > Please try a bit harder to stay within 80 columns. M-q is your friend. > > > ! ;; Hacker: l3thal@smashthestack.org > > ! ;; Created: 2004-05-07 > > ! ;; Hacked: 2014-06-08 > This and the next comment slipped my me on mistake. This was for my own use to track revisions as It wasn't in version control. It should not have been included in the patch > If you want to add your name, add it to the "Author:" part (one per > line, see in erc.el for an example). And don't add entries like > timestamps of when it was last changed. This just causes spurious > conflicts when merging branches. This info is readily available via > "bzr log". I tolerate things like "Created" (tho find it a waste of > perfectly good bits and screen real estate) because, by nature these > should never change and hence don't cause such spurious conflicts. > > > ! ;;; hacked > > (define-erc-response-handler (JOIN) > > ! "HACKED: Handle join messages." > > What does this "hacked" mean here? Looks like left-over annotations you > used temporarily to keep track of what you changed. This was a mistake, please see above. > > Anyway, the patch looks OK to me. As soon as your copyright paperwork > comes through I can give you write access and you can install it. > > > Stefan