From: ueno Date: Wed, 11 Jun 2003 06:56:47 +0000 (+0000) Subject: * riece-identity.el (riece-completing-read-identity): Remove nil X-Git-Tag: strict-naming-mergepoint~21 X-Git-Url: http://git.chise.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=9f267e6270733e03e63834690dd7fc52b6d04470;p=elisp%2Friece.git * riece-identity.el (riece-completing-read-identity): Remove nil from channels before completing-read. * riece-message.el (riece-message-make-name): Fix condition for priv. (riece-message-make-global-name): Ditto. (riece-message-buffer): Ditto. * riece-misc.el (riece-current-nickname): Use riece-with-identity-buffer. --- diff --git a/lisp/ChangeLog b/lisp/ChangeLog index 322f883..3be7d79 100644 --- a/lisp/ChangeLog +++ b/lisp/ChangeLog @@ -1,5 +1,17 @@ 2003-06-11 Daiki Ueno + * riece-identity.el (riece-completing-read-identity): Remove nil + from channels before completing-read. + + * riece-message.el (riece-message-make-name): Fix condition for priv. + (riece-message-make-global-name): Ditto. + (riece-message-buffer): Ditto. + + * riece-misc.el (riece-current-nickname): Use + riece-with-identity-buffer. + +2003-06-11 Daiki Ueno + * riece-message.el (riece-message-parent-buffers): Regard message's speaker as target when priv mode. diff --git a/lisp/riece-identity.el b/lisp/riece-identity.el index 9ec6475..3f7e2a1 100644 --- a/lisp/riece-identity.el +++ b/lisp/riece-identity.el @@ -155,14 +155,14 @@ take server names into account." (riece-with-server-buffer server (riece-make-identity (riece-encode-coding-string prefix) server)))) -(defun riece-completing-read-identity (prompt table +(defun riece-completing-read-identity (prompt channels &optional predicate must-match) (let* ((decoded (completing-read prompt - (mapcar (lambda (channel) - (list (riece-decode-identity channel))) - table) + (delq nil (mapcar (lambda (channel) + (list (riece-decode-identity channel))) + (or channels riece-current-channels))) predicate must-match)) (encoded (riece-encode-identity decoded))) diff --git a/lisp/riece-message.el b/lisp/riece-message.el index cb5f3ef..82be833 100644 --- a/lisp/riece-message.el +++ b/lisp/riece-message.el @@ -60,7 +60,7 @@ :group 'riece-message) (defun riece-message-make-open-bracket (message) - "Makes `open-bracket' string for MESSAGE." + "Make `open-bracket' string for MESSAGE." (if (riece-message-own-p message) ">" (if (eq (riece-message-type message) 'notice) @@ -72,7 +72,7 @@ "<"))))) (defun riece-message-make-close-bracket (message) - "Makes `close-bracket' string for MESSAGE." + "Make `close-bracket' string for MESSAGE." (if (riece-message-own-p message) "<" (if (eq (riece-message-type message) 'notice) @@ -84,14 +84,15 @@ ">"))))) (defun riece-message-make-name (message) - "Makes local identity for MESSAGE." - (if (and (riece-message-private-p message) - (riece-message-own-p message)) - (riece-identity-prefix (riece-message-target message)) - (riece-identity-prefix (riece-message-speaker message)))) + "Make local identity for MESSAGE." + (if (riece-message-private-p message) + (if (riece-message-own-p message) + (riece-identity-prefix (riece-message-target message)) + (riece-identity-prefix (riece-message-speaker message))) + (riece-identity-prefix (riece-message-target message)))) (defun riece-message-make-global-name (message) - "Makes global identity for MESSAGE." + "Make global identity for MESSAGE." (if (riece-message-private-p message) (if (riece-message-own-p message) (riece-identity-prefix (riece-message-target message)) @@ -101,13 +102,13 @@ (defun riece-message-buffer (message) "Return the buffer where MESSAGE should appear." - (let* ((target (if (riece-identity-equal - (riece-message-target message) - (riece-current-nickname)) - (riece-message-speaker message) + (let* ((target (if (riece-message-private-p message) + (if (riece-message-own-p message) + (riece-message-target message) + (riece-message-speaker message)) (riece-message-target message))) (buffer (riece-channel-buffer-name target))) - (unless (get-buffer buffer) + (unless (riece-identity-member target riece-current-channels) (riece-join-channel target) ;; If you are not joined any channel, ;; switch to the target immediately. @@ -119,17 +120,22 @@ (defun riece-message-parent-buffers (message buffer) "Return the parents of BUFFER where MESSAGE should appear. Normally they are *Dialogue* and/or *Others*." - (if (or (and buffer (riece-frozen buffer)) - (and riece-current-channel + (if (riece-message-own-p message) + riece-dialogue-buffer + (if (and buffer (riece-frozen buffer)) ;the message might not be + ;visible in buffer's window + (list riece-dialogue-buffer riece-others-buffer) + (if (and riece-current-channel ;the message is not sent to + ;the current channel (if (riece-message-private-p message) (not (riece-identity-equal (riece-message-speaker message) riece-current-channel)) (not (riece-identity-equal (riece-message-target message) - riece-current-channel))))) - (list riece-dialogue-buffer riece-others-buffer) - riece-dialogue-buffer)) + riece-current-channel)))) + (list riece-dialogue-buffer riece-others-buffer) + riece-dialogue-buffer)))) (defun riece-display-message (message) "Display MESSAGE object." @@ -142,6 +148,7 @@ Normally they are *Dialogue* and/or *Others*." (global-name (funcall riece-message-make-global-name-function message)) (buffer (riece-message-buffer message)) + (server-name (riece-identity-server (riece-message-speaker message))) parent-buffers) (when (and buffer (riece-message-own-p message) @@ -154,11 +161,12 @@ Normally they are *Dialogue* and/or *Others*." (concat open-bracket name close-bracket " " (riece-message-text message) "\n")) (riece-insert parent-buffers - (concat - (riece-concat-server-name - (concat open-bracket global-name close-bracket - " " (riece-message-text message))) - "\n")) + (if (equal server-name "") + (concat open-bracket global-name close-bracket + " " (riece-message-text message) "\n") + (concat open-bracket global-name close-bracket + " " (riece-message-text message) + " (from " server-name ")\n"))) (run-hook-with-args 'riece-after-display-message-functions message))) (defun riece-make-message (speaker target text &optional type own-p) @@ -194,12 +202,10 @@ Currently possible values are `action' and `notice'." (defun riece-message-private-p (message) "Return t if MESSAGE is a private message." - (if (riece-message-own-p message) - (not (riece-channel-p (riece-identity-prefix - (riece-message-target message)))) - (riece-identity-equal - (riece-message-target message) - (riece-current-nickname)))) + (not (or (riece-channel-p (riece-identity-prefix + (riece-message-speaker message))) + (riece-channel-p (riece-identity-prefix + (riece-message-target message)))))) (defun riece-message-external-p (message) "Return t if MESSAGE is from outside the channel." @@ -209,7 +215,7 @@ Currently possible values are `action' and `notice'." (riece-with-identity-buffer target (mapcar (lambda (user) - (riece-make-identity user (riece-identity-server target))) + (riece-make-identity user riece-server-name)) (riece-channel-get-users (riece-identity-prefix target)))))))) (defun riece-own-channel-message (message &optional channel type) diff --git a/lisp/riece-misc.el b/lisp/riece-misc.el index 6b04b3e..9f02c66 100644 --- a/lisp/riece-misc.el +++ b/lisp/riece-misc.el @@ -81,7 +81,7 @@ (defun riece-current-nickname () "Return the current nickname." - (riece-with-server-buffer (riece-identity-server riece-current-channel) + (riece-with-identity-buffer riece-current-channel (if riece-real-nickname (riece-make-identity riece-real-nickname riece-server-name))))