mikutter

photo_support は、依存関係としてはgtk pluginとの関係を持っていませんが、実装上はgdk_pixbuf2 gemに依存しており実質的にgtkへの依存と見做せる状態です。
gemのAPIを使っている場所は1行だけなんですが、こんな風に決め打ってしまっても十分そうだと考えています。意見いただけませんか？

拡張子は GitHubのヘルプページ から拾って決め打ちました。
もう大したことないのでインラインにしています。
それによって無意味になった定数のほうは……保守的にするならこうですが、多分誰も使ってないんじゃないかなあという気も。

diff --git a/plugin/photo_support/photo_support.rb b/plugin/photo_support/photo_support.rb
index 7cacd2b6..8489082b 100644
--- a/plugin/photo_support/photo_support.rb
+++ b/plugin/photo_support/photo_support.rb
@@ -5,9 +5,11 @@ require 'httpclient'
 require 'json'

 module Plugin::PhotoSupport
-  SUPPORTED_IMAGE_FORMATS = GdkPixbuf::Pixbuf.formats.flat_map{|f| f.extensions }.freeze
+  SUPPORTED_IMAGE_FORMATS = %w[jpeg jpg png gif].freeze
+  deprecate_constant :SUPPORTED_IMAGE_FORMATS
+
   INSTAGRAM_PATTERN = %r{\Ahttps?://(?:instagr\.am|(?:www\.)?instagram\.com)/p/([a-zA-Z0-9_\-]+)/}
-  GITHUB_IMAGE_PATTERN = %r<\Ahttps://github\.com/(\w+/\w+)/blob/(.*\.(?:#{SUPPORTED_IMAGE_FORMATS.join('|')}))\z>
+  GITHUB_IMAGE_PATTERN = %r<\Ahttps://github\.com/(\w+/\w+)/blob/(.*\.(?:jpe?g|png|gif))\z>
   YOUTUBE_PATTERN = %r<\Ahttps?://(?:www\.youtube\.com/watch\?v=|youtu\.be/)(\w+)>
   NICOVIDEO_PATTERN = %r<\Ahttps?://(?:(?:www|sp)\.nicovideo\.jp/watch|nico\.ms)/([sn][mo][1-9]\d*)>

SUPPORTED_IMAGE_FORMATS の扱いは、コメントに書いてくれたパッチでOKです。多分誰も使ってないですが、消すのはメジャーバージョンアップのときにしたいです。

#note-5, #note-6 対応しました。

とりあえず現時点までの作業成果で、Mastodonをsubscribeするmikutterdをだいぶ簡単に作れるくらいまで進みました。
あと残ってるものはこんな感じかと。

リストアップできてないものも考慮すると、この作業って #1380 と被ってきますね。

やれそう/やりたい¶

• settings
• shortcutkey (rel. 機能 #1386: Shortcut Keyの設定に、FormDSLのlistviewを使う)

一応やれそうだが、あまり重要でない¶

• aspectframe
• bugreport

放置¶

• console
• intent_selector
• modelviewer
• quickstep
• search
• skin_setting_gtk

settingsについては、以下のチケットに切り出し
提案 #1490: settingsプラグインのGtk依存をやめる