mirror of
https://codeberg.org/Freeyourgadget/Gadgetbridge
synced 2024-10-12 16:18:10 +02:00
emoji: only the init can be sync
I applied @cpfeiffer suggestion, then I saw it was already done :) Here only the init part is set as 'synchronized', very small optimisation but because this change is ready, why not sharing it :)
This commit is contained in:
parent
ad9b45a94d
commit
c239d1767b
@ -61,7 +61,7 @@ public class EmojiConverter {
|
|||||||
{"\u2764", "<3"}, // heart
|
{"\u2764", "<3"}, // heart
|
||||||
};
|
};
|
||||||
|
|
||||||
private static boolean isInitialised = false;
|
private static boolean isEmojiDataInitialised = false;
|
||||||
|
|
||||||
private static String convertSimpleEmojiToAscii(String text) {
|
private static String convertSimpleEmojiToAscii(String text) {
|
||||||
for (String[] emojiMap : simpleEmojiMapping) {
|
for (String[] emojiMap : simpleEmojiMapping) {
|
||||||
@ -70,12 +70,16 @@ public class EmojiConverter {
|
|||||||
return text;
|
return text;
|
||||||
}
|
}
|
||||||
|
|
||||||
private static synchronized String convertAdvancedEmojiToAscii(String text, Context context) {
|
private static synchronized void initEmojiData(Context context) {
|
||||||
// Do a lazy initialisation not to slowdown the startup and when it is needed
|
// Do a lazy initialisation not to slowdown the startup and when it is needed
|
||||||
if (!isInitialised) {
|
if (!isEmojiDataInitialised) {
|
||||||
EmojiManager.initEmojiData(context);
|
EmojiManager.initEmojiData(context);
|
||||||
isInitialised = true;
|
isEmojiDataInitialised = true;
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
private static String convertAdvancedEmojiToAscii(String text, Context context) {
|
||||||
|
initEmojiData(context);
|
||||||
|
|
||||||
return EmojiUtils.shortCodify(text);
|
return EmojiUtils.shortCodify(text);
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user