)]}'
{"src/com/android/phone/NotificationMgr.java":[{"author":{"_account_id":2936,"name":"Ethan Chen","email":"intervigil@gmail.com","username":"intervigil","avatars":[{"url":"https://www.gravatar.com/avatar/382d80f420fb681ba1b88c29df904f29.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/382d80f420fb681ba1b88c29df904f29.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/382d80f420fb681ba1b88c29df904f29.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/382d80f420fb681ba1b88c29df904f29.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"bd99077761857956f9803c919cbcfd98669d4a22","unresolved":false,"context_lines":[{"line_number":286,"context_line":"            }"},{"line_number":287,"context_line":"        }"},{"line_number":288,"context_line":""},{"line_number":289,"context_line":"        boolean notifProp \u003d mApp.getResources().getBoolean(R.bool.sprint_mwi_quirk);"},{"line_number":290,"context_line":"        boolean notifOption \u003d CMSettings.System.getInt(mApp.getContentResolver(),"},{"line_number":291,"context_line":"                CMSettings.System.ENABLE_MWI_NOTIFICATION, 0) \u003d\u003d 1;"},{"line_number":292,"context_line":"        if (notifProp \u0026\u0026 !notifOption) {"},{"line_number":293,"context_line":"            // sprint_mwi_quirk is true, and ENABLE_MWI_NOTIFICATION is unchecked or unset (false)"}],"source_content_type":"text/x-java","patch_set":6,"id":"64a11d71_9c3f9e7a","line":290,"range":{"start_line":289,"start_character":8,"end_line":290,"end_character":81},"updated":"2017-10-05 03:41:19.000000000","message":"Why is this checking both the overlay resource and the settings value? In this case, you don\u0027t need to check overlay resource, right?","commit_id":"502987a096bac0e999d803b51fca91bf738b9825"},{"author":{"_account_id":1690,"name":"Kelly Craft","email":"mushroomhead52e@gmail.com","username":"Kasual","avatars":[{"url":"https://www.gravatar.com/avatar/3194e998d82925c4174e9f611b5bc978.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/3194e998d82925c4174e9f611b5bc978.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/3194e998d82925c4174e9f611b5bc978.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/3194e998d82925c4174e9f611b5bc978.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"4b9b44d6c0fe6537f5d16d22546b2f049e78bc34","unresolved":false,"context_lines":[{"line_number":286,"context_line":"            }"},{"line_number":287,"context_line":"        }"},{"line_number":288,"context_line":""},{"line_number":289,"context_line":"        boolean notifProp \u003d mApp.getResources().getBoolean(R.bool.sprint_mwi_quirk);"},{"line_number":290,"context_line":"        boolean notifOption \u003d CMSettings.System.getInt(mApp.getContentResolver(),"},{"line_number":291,"context_line":"                CMSettings.System.ENABLE_MWI_NOTIFICATION, 0) \u003d\u003d 1;"},{"line_number":292,"context_line":"        if (notifProp \u0026\u0026 !notifOption) {"},{"line_number":293,"context_line":"            // sprint_mwi_quirk is true, and ENABLE_MWI_NOTIFICATION is unchecked or unset (false)"}],"source_content_type":"text/x-java","patch_set":6,"id":"74c87e2b_42430e87","line":290,"range":{"start_line":289,"start_character":8,"end_line":290,"end_character":81},"in_reply_to":"64a11d71_9c3f9e7a","updated":"2017-10-05 08:59:12.000000000","message":"It checks the overlay to see if it it\u0027s enabled and then the specific setting on each phone to see if the voicemail should be displayed.","commit_id":"502987a096bac0e999d803b51fca91bf738b9825"},{"author":{"_account_id":11801,"name":"Joseph Annareddy","email":"javelinanddart@gmail.com","username":"javelinanddart","avatars":[{"url":"https://www.gravatar.com/avatar/0df5381571b66bb61674368f2bfc3e60.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/0df5381571b66bb61674368f2bfc3e60.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/0df5381571b66bb61674368f2bfc3e60.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/0df5381571b66bb61674368f2bfc3e60.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"820649704cacd2b9ee6a59fab69eca83b8be618e","unresolved":false,"context_lines":[{"line_number":293,"context_line":"                  Log.e(LOG_TAG, \"Unable to get carrier config\");"},{"line_number":294,"context_line":"              } else {"},{"line_number":295,"context_line":"                   if (cc.getBoolean(\"sprint_mwi_quirk\")) {"},{"line_number":296,"context_line":"                       visible \u003d false;"},{"line_number":297,"context_line":"                   }"},{"line_number":298,"context_line":"              }"},{"line_number":299,"context_line":"        }"}],"source_content_type":"text/x-java","patch_set":7,"id":"df160de1_1004ba8f","line":296,"updated":"2017-10-09 12:52:14.000000000","message":"I didn\u0027t know if the return here was still necessary or not, so I omitted it for now.","commit_id":"07b8709e98628309453aa9a91ca100898f6af40a"}]}