Skip to content
This repository has been archived by the owner on Jan 26, 2023. It is now read-only.

iOS8 対応 #96

Merged
merged 10 commits into from
Aug 16, 2014
Merged

iOS8 対応 #96

merged 10 commits into from
Aug 16, 2014

Conversation

naoya
Copy link
Owner

@naoya naoya commented Aug 16, 2014

  • SDK 8.0 に対応
  • TableView セルを自前描画してる箇所のクラッシュを fix
  • iOS 8 でも ios7? が true になるのを整理
  • コメント画面の行間が拡がっているのを fix
  • 不必要なライブラリを削除
  • WebView でアクションメニューを開くとクラッシュするのを fix

@naoya
Copy link
Owner Author

naoya commented Aug 16, 2014

ios7? の件は

  • 現状の ios7?greater than ios7? という意味のメソッドにリファクタリングし
  • ios7? は iOS7 のみ true になるようにし
  • ios8? も iOS8 のみ true になるようにし
  • (greater than ios8 という意味のメソッドを追加する)

ことで整理される。

すなわち、現状のままでも動作上は問題ないが、コードを見て後々混乱する可能性があるので注意、と

@naoya
Copy link
Owner Author

naoya commented Aug 16, 2014

PocketActivity の実装が悪いのか、UIActivity を開くと

2014-08-16 19:06:49.762 HBFav[88283:1915584] invalid value for Integer: "net.bloghackers.app.PocketActivity" (ArgumentError)
2014-08-16 19:06:49.828 HBFav[88283:1915584] *** Terminating app due to uncaught exception 'ArgumentError', reason: 'invalid value for Integer: "net.bloghackers.app.PocketActivity" (ArgumentError)

というエラーでクラッシュする。

PocketActivity を外すとちゃんと動くので、PocketActivity の実装が悪い?に1ペソ

でも、

とか見比べても何が悪いのかよくわかんないな・・・

@naoya
Copy link
Owner Author

naoya commented Aug 16, 2014

*** First throw call stack:
(
        0   CoreFoundation                      0x0527fdf6 __exceptionPreprocess + 182
        1   libobjc.A.dylib                     0x01afd837 objc_exception_throw + 44
        2   HBFav                               0x00e3aa7a _ZL10__vm_raisev + 346
        3   HBFav                               0x00e3ab62 rb_vm_raise + 226
        4   HBFav                               0x00d3d7b1 rb_exc_raise + 17
        5   HBFav                               0x00d36c72 rb_raise + 146
        6   HBFav                               0x00d3704d rb_invalid_str + 77
        7   HBFav                               0x00d1730b rb_cstr_to_inum + 1579
        8   HBFav                               0x00d1745a rb_str_to_inum + 266
        9   HBFav                               0x00d73bf4 rb_convert_to_integer + 436
        10  HBFav                               0x00d73a30 rb_Integer + 16
        11  HBFav                               0x008abbce vm_rval_to_int + 334
        12  HBFav                               0x008ad597 __unnamed_25 + 119
        13  UIKit                               0x03045b77 -[UIActivityViewController _availableActivitiesForItems:applicationExtensionActivities:] + 2264
        14  UIKit                               0x0304529a -[UIActivityViewController _availableActivitiesForItems:] + 48
        15  UIKit                               0x030467b1 -[UIActivityViewController viewDidLoad] + 187

@Watson1978
Copy link
Contributor

RubyMotion v2.29、v2.30とローカルの最新だと動くのに、v2.31だとクラッシュするので、最近直したregressionのどれかが影響していそう orz

@naoya
Copy link
Owner Author

naoya commented Aug 16, 2014

あら、まさかのそっちw

@naoya
Copy link
Owner Author

naoya commented Aug 16, 2014

コメント画面行間は pods のライブラリのアップデートで直ったみたい。

@naoya
Copy link
Owner Author

naoya commented Aug 16, 2014

2つTODOが残っているが、ブランチを整理したいのでいったん merge

@naoya naoya changed the title [WIP] iOS8 対応 iOS8 対応 Aug 16, 2014
naoya added a commit that referenced this pull request Aug 16, 2014
@naoya naoya merged commit bf6f5c4 into development/2.7 Aug 16, 2014
@naoya naoya deleted the ios8 branch August 16, 2014 23:47
@naoya naoya added this to the 2.7 milestone Aug 16, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants