Merge pull request signalapp#249 from joeykrim/patch-1

moxie0 · moxie0 · commit c827f0a2a7fc · 2013-07-14T11:33:23.000-07:00
Small clarification in the comments on the types of SMS to ignore
diff --git a/src/org/thoughtcrime/securesms/service/SmsListener.java b/src/org/thoughtcrime/securesms/service/SmsListener.java
@@ -46,11 +46,10 @@ private boolean isExemption(SmsMessage message, String messageBody) {
       return true;
     }
 
-    // Sprint Visual Voicemail
     return
       message.getOriginatingAddress().length() < 7 &&
-      (messageBody.startsWith("//ANDROID:") || messageBody.startsWith("//Android:") ||
-       messageBody.startsWith("//android:") || messageBody.startsWith("//BREW:"));
+      (messageBody.toUpperCase().startsWith("//ANDROID:") || // Sprint Visual Voicemail
+       messageBody.startsWith("//BREW:")); //BREW stands for “Binary Runtime Environment for Wireless"
   }
 
   private SmsMessage getSmsMessageFromIntent(Intent intent) {

Original file line number	Diff line number	Diff line change
`@@ -46,11 +46,10 @@ private boolean isExemption(SmsMessage message, String messageBody) {`
`46`	`46`	`return true;`
`47`	`47`	`}`
`48`	`48`
`49`		`- // Sprint Visual Voicemail`
`50`	`49`	`return`
`51`	`50`	`message.getOriginatingAddress().length() < 7 &&`
`52`		`- (messageBody.startsWith("//ANDROID:") \|\| messageBody.startsWith("//Android:") \|\|`
`53`		`- messageBody.startsWith("//android:") \|\| messageBody.startsWith("//BREW:"));`
	`51`	`+ (messageBody.toUpperCase().startsWith("//ANDROID:") \|\| // Sprint Visual Voicemail`
	`52`	`+ messageBody.startsWith("//BREW:")); //BREW stands for “Binary Runtime Environment for Wireless"`
`54`	`53`	`}`
`55`	`54`
`56`	`55`	`private SmsMessage getSmsMessageFromIntent(Intent intent) {`