Project

General

Profile

Actions

バグ #1515

closed

機能 #887: gtk3

gtk3: mui 関連の deprecated 警告

Added by Izumi Tsutsui 7 months ago. Updated 7 months ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
ブランチ:
topic/1515-mui-avoid-deprecated
クラッシュする:
No

Description

twitter plugin の gtk3 対応をやっていると
それらからしか呼ばれてなさそうなところでちらほら deprecated の warning が出ていて、
本題のデバッグ中にメッセージが出てめんどくさいのでとりあえず修正したい、という案件です。

パッチは追って。

その1
source:core/mui/gtk_extension.rb@0b99c704#L246

warning: /usr/pkg/lib/ruby/gems/2.7.0/gems/glib2-3.4.9/lib/glib2/deprecatable.rb:177:in `block in __define_deprecated_method__': {MIKUTTER_DIR}/core/mui/gtk_extension.rb:246:in `get_range': 'Gtk::TextBuffer#get_iter_at_offset' has been deprecated. Use 'Gtk::TextBuffer#get_iter_at(:offset => nil)'.

その2
source:core/mui/gtk_inneruserlist.rb@0b99c704#L20

warning: /usr/pkg/lib/ruby/gems/2.7.0/gems/glib2-3.4.9/lib/glib2/deprecatable.rb:123:in `const_missing': {MIKUTTER_DIR}/core/mui/gtk_inneruserlist.rb:20:in `initialize': 'Gtk::SORT_DESCENDING' has been deprecated. Use 'Gtk::SortType::DESCENDING' or ':descending'.

その3
source:core/mui/gtk_userlist.rb@0b99c704#L28

warning: /usr/pkg/lib/ruby/gems/2.7.0/gems/glib2-3.4.9/lib/glib2/deprecatable.rb:177:in `block in __define_deprecated_method__': {MIKUTTER_DIR}/core/mui/gtk_userlist.rb:28:in `initialize': 'Gtk::VScrollbar.new' has been deprecated. Use 'Gtk::Scrollbar.new(:vertical, adjustment)'.
warning: /usr/pkg/lib/ruby/2.7.0/rubygems/deprecate.rb:62:in `block (2 levels) in deprecate': NOTE: Gtk::Box#closeup is deprecated with no replacement. It will be removed on or after 2018-09-01.
Gtk::Box#closeup called from {MIKUTTER_DIR}/core/mui/gtk_userlist.rb:29.

source:core/mui/gtk_extension.rb@0b99c704#L235Gtk::Box#closeup を deprecated にするのか?
というところは議論の余地があるかもしれませんが、
手元ではだいぶ消してしまったので修正する方針のままでいいかなあ、と思っています。


Files


Related issues

Related to 提案 #1525: closeupメソッドの利用をやめる終了toshi_a 初音Actions
Actions #2

Updated by toshi_a 初音 7 months ago

  • Status changed from 分類待ち to パッチ適用待ち
  • Assignee set to Izumi Tsutsui

ありがとうございます。影響少ないので先に取り込んじゃいましょう。

closeupはどうでもいいんですが(残しときたいという思いも薄いし、消したいとも思わない)、みんなもう対応しちゃってそうなので、mikutter本体では利用やめましょうか。
でもあんまりサードパーティープラグインでこの警告出まくるようならdeprecated外したほうが良いかも。対応個数結構多いので、issueは作っときます。

Actions #3

Updated by toshi_a 初音 7 months ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • ブランチ set to topic/1515-mui-avoid-deprecated
Actions #4

Updated by Izumi Tsutsui 7 months ago

  • Status changed from レビュー待ち to マージ待ち

toshi_a 初音 さんは #note-2 で書きました:

ありがとうございます。影響少ないので先に取り込んじゃいましょう。

差分としてはOKです。
が、こっちのチケットが先だったせいで
source:plugin/mastodon_account_viewer/relational_container.rbHBox の修正は
#1520 で当てちゃってました。これだけ手動コンフリクト解決必要です。

closeupはどうでもいいんですが(残しときたいという思いも薄いし、消したいとも思わない)、みんなもう対応しちゃってそうなので、mikutter本体では利用やめましょうか。
でもあんまりサードパーティープラグインでこの警告出まくるようならdeprecated外したほうが良いかも。対応個数結構多いので、issueは作っときます。

本体側で消す方針なら作業は機械的にできるので、プラグイン的にもそういう方針が決まればやりやすいかなとは思います。
個人的に読む側の立場としては、単なる alias 的なやつは探すのにちょっと手間がかかるのでリファレンスで見つかるほうがいい、
というのはあるかもしれません。(GTK3は公式ドキュメントもいまいちですが……)

Actions #5

Updated by toshi_a 初音 7 months ago

  • Status changed from マージ待ち to 終了
Actions #6

Updated by Izumi Tsutsui 6 months ago

  • Related to 提案 #1525: closeupメソッドの利用をやめる added
Actions

Also available in: Atom PDF