Re: transMsg() (P2-php-dev 35) - p2

 >>123は可能性がないわけではないけど頭にあることが多いし
切れてもあまり困らないので今のままでいいと思います。

> '/ *<[^>]*$/i' の方が汎用に対応できるのでいいなー。

最初はタグの閉じ忘れにも対応しようとしてたんだけど
strip_tags()で<br>以外を削ってるし、なにより頭が混乱してきたので
今のかたちに落ち着きました。

> 下の '/( *<br> *)+$/i' は、別になくても良さげかもですね。

良さげですね。消しましょう。

// +------------------------------+
// | rsk <moons****@s32*****> |
// +------------------------------+

On 2005/05/11, at 7:52, aki wrote:

>> お疲れさまです。
>>
>> こちらでも僕が書いてたコードと1.5.23の差分を参考に
>> フィードバックさせてもらいました。
>> （[P2-php-dev 32]を読む前に作業してました）
>>
>> showthreadkは次で直します。ありがとうございました。
>> どうせなら「略」のときも
>> $msg = preg_replace_callback($this->str_to_link_regex, array($this,
>> 'link_callback'), $msg);
>> しようと思ってるんですが、どうでしょう？
>> ・・・と書いたところで、URLは途中で切れる可能性が高いよな、と思い直す。
>> ソースにコメント書いとこう。(笑）
>
> そうそう、URLが途中で切れると困るので、
> 簡易的な表示にしてました。
>
> でも、<br>と同じく、urlの途中で切れている場合も、
> あらかじめカットしておけばOKかなと思ったものの、
> やっぱり、 >>123 も ID:XXXX も途中で切れる可能性ある...。
>
> うーん、略があるときは、簡易プレビュー的な扱いなので、
> 今のままでもいいかな。
>
> 新しい方のコードを見て…
> $msg = preg_replace('/ *<[^>]*$/i', '', $msg);
> $msg = preg_replace('/( *<br> *)+$/i', '', $msg);
>
> '/ *<[^>]*$/i' の方が汎用に対応できるのでいいなー。
> 下の '/( *<br> *)+$/i' は、別になくても良さげかもですね。
>
> --
> aki
>

p2

[P2-php-dev 35] Re: transMsg()