)]}'
{"service/java/com/android/server/wifi/ActiveModeWarden.java":[{"author":{"_account_id":11883,"name":"Luca Stefani","email":"luca.stefani.ge1@gmail.com","username":"luca020400","avatars":[{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/e0d7fe321be897bf85cf3d240cbd3be9.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"28ef43307214018e7ea7928fce8354ddbf4dd51a","unresolved":true,"context_lines":[{"line_number":655,"context_line":"        }"},{"line_number":656,"context_line":"    };"},{"line_number":657,"context_line":""},{"line_number":658,"context_line":"    public void registerStaEventCallback() {}"},{"line_number":659,"context_line":"    public void unregisterStaEventCallback() {}"},{"line_number":660,"context_line":"}"}],"source_content_type":"text/x-java","patch_set":1,"id":"dd5178f4_d03d1f52","line":659,"range":{"start_line":658,"start_character":0,"end_line":659,"end_character":47},"updated":"2020-08-02 16:35:33.000000000","message":"so it\u0027s just stubs?","commit_id":"55040e5955c211817bc9dfbba3cc59d297b47e76"},{"author":{"_account_id":14437,"name":"Kshitij Gupta","email":"kshitijgm@gmail.com","username":"AgentFabulous","avatars":[{"url":"https://www.gravatar.com/avatar/26fd44de6e856da7dd34b7c1d1c31d08.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/26fd44de6e856da7dd34b7c1d1c31d08.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/26fd44de6e856da7dd34b7c1d1c31d08.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/26fd44de6e856da7dd34b7c1d1c31d08.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"205f2bb297fcfeaef1a5bb4d4ae2aaa11c6ac563","unresolved":true,"context_lines":[{"line_number":655,"context_line":"        }"},{"line_number":656,"context_line":"    };"},{"line_number":657,"context_line":""},{"line_number":658,"context_line":"    public void registerStaEventCallback() {}"},{"line_number":659,"context_line":"    public void unregisterStaEventCallback() {}"},{"line_number":660,"context_line":"}"}],"source_content_type":"text/x-java","patch_set":1,"id":"8e380e08_cbce4b0f","line":659,"range":{"start_line":658,"start_character":0,"end_line":659,"end_character":47},"in_reply_to":"dd5178f4_d03d1f52","updated":"2020-08-05 19:39:33.000000000","message":"Well yes, but actually no. MTK does not use this class for ActiveModeWarden, they extend it into another class and that is another rabbit hole of classes and changes to other repos. This simply allows ImsService to register a callback that doesn\u0027t otherwise exist. I\u0027m not sure what issues an empty callback in place of the real ones would cause here, but in the last 1-2 months of use, it hasn\u0027t caused any trouble on my builds; it fixes crash of ImsService on StaState change (WiFi connect, disconnect, etc). Without the callback, the crash breaks ImsService until next reboot (constant FC dialogs and no IMS).","commit_id":"55040e5955c211817bc9dfbba3cc59d297b47e76"}]}
