Re: [twmode-users] twittering-completing-read

2010-01-14 スレッド表示 Tadashi MATSUO
松尾です。

> emacs-major-versionで場合分けをするのは、最小限に
> しませんか。以下を例にして言うと、
> 
> (if (fboundp 'delete-dups)
> ;; delete-dupsをつかったコード
>   ;; 自前で実装したコード
> 
> にしたほうが、保守する際に前提となる知識が少なくて
> 済むからです。

機能の違いが直接分かる場合には極力emacs-major-versionを
使わない方が良いですね。twittering-remove-duplicates()を
そのように修正しました。

> もっとも、今回のケースでは、completing-read()には
> 常にalistを渡すようにしてもよいと思いますが。。

こちらはどうしましょうか。
completing-read()に丸投げできるときはそうした方が安心の
ような気もします。

---
松尾 直志 

--
Throughout its 18-year history, RSA Conference consistently attracts the
world's best and brightest in the field, creating opportunities for Conference
attendees to learn about information security's most important issues through
interactions with peers, luminaries and emerging and established companies.
http://p.sf.net/sfu/rsaconf-dev2dev
___
twmode-users mailing list
twmode-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/twmode-users


Re: [twmode-users] twittering-completing-read

2010-01-14 スレッド表示 Yuto Hayamizu

はやみずです


> emacs-major-versionで場合分けをするのは、最小限に
> しませんか。以下を例にして言うと、
> 
> (if (fboundp 'delete-dups)
> ;; delete-dupsをつかったコード
>   ;; 自前で実装したコード
> 
> にしたほうが、保守する際に前提となる知識が少なくて
> 済むからです。

賛成です



At Thu, 14 Jan 2010 12:30:29 +0900 (JST),
yata_git...@y.haun.org wrote:
> 
> 好みの問題もあると思うのですが、
> 
> emacs-major-versionで場合分けをするのは、最小限に
> しませんか。以下を例にして言うと、
> 
> (if (fboundp 'delete-dups)
> ;; delete-dupsをつかったコード
>   ;; 自前で実装したコード
> 
> にしたほうが、保守する際に前提となる知識が少なくて
> 済むからです。
> 
> もっとも、今回のケースでは、completing-read()には
> 常にalistを渡すようにしてもよいと思いますが。。
> 
> 
> > Emacs22以降ではcompleting-read()に与える補完候補として
> > stringのlistを使えますが、Emacs21以前では使うことが
> > できません(alistにする必要があります)。
> 
> > completing-read()を呼び出す場面で一々対処するのは見通しも
> > よくないのでtwittering-completing-read()という関数を追加
> > してそちらで対処するようにしてみました。
> 
> > また、補完候補から重複を除くのに使っていたdelete-dups()は
> > Emacs21以前では定義されていません。
> > こちらもtwittering-remove-duplicates()という関数を作って
> > Emacs21以前の環境では自前で実装したものを使うようにして
> > います。
> > delete-dups()は引数のlistに対して破壊的に働きますが、
> > twittering-remove-duplicates()は引数のオブジェクトを
> > 変更しないようにしてあります。
> 
> > ---
> > 松尾 直志 
> 
> --
> Throughout its 18-year history, RSA Conference consistently attracts the
> world's best and brightest in the field, creating opportunities for Conference
> attendees to learn about information security's most important issues through
> interactions with peers, luminaries and emerging and established companies.
> http://p.sf.net/sfu/rsaconf-dev2dev
> ___
> twmode-users mailing list
> twmode-users@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/twmode-users

--
Throughout its 18-year history, RSA Conference consistently attracts the
world's best and brightest in the field, creating opportunities for Conference
attendees to learn about information security's most important issues through
interactions with peers, luminaries and emerging and established companies.
http://p.sf.net/sfu/rsaconf-dev2dev
___
twmode-users mailing list
twmode-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/twmode-users


Re: [twmode-users] twittering-completing-read

2010-01-13 スレッド表示 yata_github
好みの問題もあると思うのですが、

emacs-major-versionで場合分けをするのは、最小限に
しませんか。以下を例にして言うと、

(if (fboundp 'delete-dups)
;; delete-dupsをつかったコード
  ;; 自前で実装したコード

にしたほうが、保守する際に前提となる知識が少なくて
済むからです。

もっとも、今回のケースでは、completing-read()には
常にalistを渡すようにしてもよいと思いますが。。


> Emacs22以降ではcompleting-read()に与える補完候補として
> stringのlistを使えますが、Emacs21以前では使うことが
> できません(alistにする必要があります)。

> completing-read()を呼び出す場面で一々対処するのは見通しも
> よくないのでtwittering-completing-read()という関数を追加
> してそちらで対処するようにしてみました。

> また、補完候補から重複を除くのに使っていたdelete-dups()は
> Emacs21以前では定義されていません。
> こちらもtwittering-remove-duplicates()という関数を作って
> Emacs21以前の環境では自前で実装したものを使うようにして
> います。
> delete-dups()は引数のlistに対して破壊的に働きますが、
> twittering-remove-duplicates()は引数のオブジェクトを
> 変更しないようにしてあります。

> ---
> 松尾 直志 

--
Throughout its 18-year history, RSA Conference consistently attracts the
world's best and brightest in the field, creating opportunities for Conference
attendees to learn about information security's most important issues through
interactions with peers, luminaries and emerging and established companies.
http://p.sf.net/sfu/rsaconf-dev2dev
___
twmode-users mailing list
twmode-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/twmode-users


Re: [twmode-users] twittering-completing-read

2010-01-10 スレッド表示 Yuto Hayamizu

はやみずです

対応ありがとうございます。前にこのバグを踏んだときに直そうと思ってすっ
かりわすれていました。


At Sun, 10 Jan 2010 20:19:00 +0900 (JST),
Tadashi MATSUO wrote:
> 
> 松尾です。
> 
> Emacs22以降ではcompleting-read()に与える補完候補として
> stringのlistを使えますが、Emacs21以前では使うことが
> できません(alistにする必要があります)。
> 
> completing-read()を呼び出す場面で一々対処するのは見通しも
> よくないのでtwittering-completing-read()という関数を追加
> してそちらで対処するようにしてみました。
> 
> また、補完候補から重複を除くのに使っていたdelete-dups()は
> Emacs21以前では定義されていません。
> こちらもtwittering-remove-duplicates()という関数を作って
> Emacs21以前の環境では自前で実装したものを使うようにして
> います。
> delete-dups()は引数のlistに対して破壊的に働きますが、
> twittering-remove-duplicates()は引数のオブジェクトを
> 変更しないようにしてあります。
> 
> ---
> 松尾 直志 
> 
> --
> This SF.Net email is sponsored by the Verizon Developer Community
> Take advantage of Verizon's best-in-class app development support
> A streamlined, 14 day to market process makes app distribution fast and easy
> Join now and get one step closer to millions of Verizon customers
> http://p.sf.net/sfu/verizon-dev2dev 
> ___
> twmode-users mailing list
> twmode-users@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/twmode-users

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
___
twmode-users mailing list
twmode-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/twmode-users


[twmode-users] twittering-completing-read

2010-01-10 スレッド表示 Tadashi MATSUO
松尾です。

Emacs22以降ではcompleting-read()に与える補完候補として
stringのlistを使えますが、Emacs21以前では使うことが
できません(alistにする必要があります)。

completing-read()を呼び出す場面で一々対処するのは見通しも
よくないのでtwittering-completing-read()という関数を追加
してそちらで対処するようにしてみました。

また、補完候補から重複を除くのに使っていたdelete-dups()は
Emacs21以前では定義されていません。
こちらもtwittering-remove-duplicates()という関数を作って
Emacs21以前の環境では自前で実装したものを使うようにして
います。
delete-dups()は引数のlistに対して破壊的に働きますが、
twittering-remove-duplicates()は引数のオブジェクトを
変更しないようにしてあります。

---
松尾 直志 

--
This SF.Net email is sponsored by the Verizon Developer Community
Take advantage of Verizon's best-in-class app development support
A streamlined, 14 day to market process makes app distribution fast and easy
Join now and get one step closer to millions of Verizon customers
http://p.sf.net/sfu/verizon-dev2dev 
___
twmode-users mailing list
twmode-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/twmode-users