Project

General

Profile

Actions

最適化 #755

closed

memoize deprecated

Added by dai lxr over 9 years ago. Updated over 8 years ago.

Status:
終了
Priority:
低め
Target version:
Start date:
2015-05-19
Due date:
% Done:

0%

プラグイン名:

Description

https://github.com/djberg96/memoize

= DEPRECATION WARNING
Please do not use this gem. It has long been surpassed by other, better
memoization gems like memoizable and memoist.

https://github.com/dkubb/memoizable
https://github.com/matthewrudy/memoist

どちらもメソッド名が memoize と memoize と同じなので、
miquireとincludeを置き換えてやるだけかと思ったのですが、
他にもコードの書き換えが必要のようです。

Actions #1

Updated by dai lxr over 9 years ago

添付のパッチでmemoizeをmemoistに置き換えたところ、動いているようです。

同様に、単純にmemoizeをmemoizableに置き換えたところ、

/usr/lib/ruby/2.1.0/monitor.rb:241: stack level too deep (SystemStackError)

というエラーが出て起動しませんでした。

Actions #2

Updated by dai lxr over 9 years ago

Internal Server Error でファイル添付ができていなかったのでパッチをベタ貼りします。

diff --git a/Gemfile b/Gemfile
index cbf5607..70da7dc 100644
--- a/Gemfile
+++ b/Gemfile
@@ -11,7 +11,7 @@ group :default do
   gem 'oauth', '~> 0.4'
   gem 'json_pure', '~> 1.8'
   gem 'addressable', '~> 2.3'
-  gem 'memoize', '~> 1.3'
+  gem 'memoist', '~> 0.12'
   gem 'ruby-hmac', '~> 0.4'
   gem 'typed-array', '~> 0.1'
   gem 'delayer', '~> 0.0'
diff --git a/core/mui/gtk_web_image_loader.rb b/core/mui/gtk_web_image_loader.rb
index ede1d6b..aaab5f6 100644
--- a/core/mui/gtk_web_image_loader.rb
+++ b/core/mui/gtk_web_image_loader.rb
@@ -3,7 +3,7 @@

 miquire :core, 'serialthread', 'skin'
 miquire :mui, 'web_image_loader_image_cache'
-miquire :lib, 'memoize', 'addressable/uri'
+miquire :lib, 'addressable/uri'
 require 'net/http'
 require 'uri'
 require 'thread'
diff --git a/core/utils.rb b/core/utils.rb
index 99522f2..8f191b0 100644
--- a/core/utils.rb
+++ b/core/utils.rb
@@ -34,8 +34,8 @@ require File.join(File::dirname(__FILE__), 'miquire')
 miquire :lib, 'lazy'

 # すべてのクラスにメモ化機能を
-miquire :lib, 'memoize'
-include Memoize
+miquire :lib, 'memoist'
+include Memoist

 # Environment::CONFROOT内のファイル名を得る。
 #   confroot(*path)
Actions #3

Updated by toshi_a 初音 over 9 years ago

  • Assignee set to toshi_a 初音

どうするか考えてみます。
どのバージョンで切り替えるかまだ決めてませんが、使用するライブラリも変わるのでマイナーアップデート(3.3)の方向で検討しています

Actions #4

Updated by toshi_a 初音 over 8 years ago

  • Status changed from 新規 to 実装待ち
  • Target version set to 3.3
Actions #5

Updated by toshi_a 初音 over 8 years ago

  • Status changed from 実装待ち to 終了

mikutter 3.3で実装しました

Actions

Also available in: Atom PDF