Remove additional calendars in the ICU data

android/patch_locale.sh

 #!/bin/sh
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # Keep only the currencies used by the larget 60 economies in terms of GDP
 # with several more added in.
 # TODO(jshin): Use ucurr_isAvailable in ICU to drop more currencies.
 # See also http://en.wikipedia.org/wiki/List_of_circulating_currencies
 # Copied from scripts/trim_data.sh. Need to refactor.
@@ skipping 146 matching lines @@
       ;;
     ar)
       EXTRA_CAL='arabic'
       ;;
     fa)
       EXTRA_CAL='persian'
       ;;
     ja)
       EXTRA_CAL='japanese'
       ;;
+    *)
+      EXTRA_CAL=''
+      ;;
   esac
 
   # Add 'roc' calendar to zh_Hant*.
-  [[ "$(basename $i .txt)" =~ 'zh_Hant' ]] && { EXTRA_CAL="$EXTRA_CAL|roc"; }
+  [[ "$(basename $i .txt)" =~ 'zh_Hant' ]] && { EXTRA_CAL="${EXTRA_CAL}|roc"; }
 
-  CAL_PATTERN="(${COMMON_CALENDARS}|${EXTRA_CAL})"
+  CAL_PATTERN="(${COMMON_CALENDARS})"

[Mark Mentovai, 2015/04/03 12:51:09]

You can write all of this in one line:

  CAL_PATTERN="${COMMON_CALENDARS}${EXTRA_CAL:+|${EXTRA_CAL}}"

${var:+text} expands to nothing if var is empty and to text if var is nonempty.

+  [[ -n "${EXTRA_CAL}" ]] && \
+    { CAL_PATTERN="(${COMMON_CALENDARS}|${EXTRA_CAL})"; }
   echo $CAL_PATTERN
 
   echo Overwriting $i...
   sed -r '/^    calendar\{$/,/^    \}$/ {
             /^    calendar\{$/p
             /^        '${CAL_PATTERN}'\{$/, /^        \}$/p
             /^    \}$/p
             d
           }' -i $i
 done
 
 echo DONE.