Skip to content

Commit edc5a5f

Browse files
committed
[Release] Changed Version to 8 (version name 1.5.1)
1.Fixed a bug while labelling screenshot, added a null check to fix it Signed-off-by: pavan142 <[email protected]>
1 parent c3eb9ec commit edc5a5f

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

app/build.gradle

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,8 @@ android {
1111
applicationId "com.frankenstein.screenx"
1212
minSdkVersion 21
1313
targetSdkVersion 29
14-
versionCode 7
15-
versionName "1.4.1"
14+
versionCode 8
15+
versionName "1.5.1"
1616
testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
1717
}
1818
buildTypes {

app/src/main/java/com/frankenstein/screenx/helper/AppHelper.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,6 +115,8 @@ private static String getSourceApp(Context context, String filename) {
115115
}
116116
String packageId = filename.substring(startIndex, endIndex);
117117
String appName = getAppName(pm, packageId);
118+
if (appName == null)
119+
return null;
118120
appName = (appName.compareTo("") == 0) ? null : appName;
119121
return appName;
120122
}
@@ -144,7 +146,7 @@ public static Screenshot processNewScreen(Context context, File file) {
144146
Screenshot screen = new Screenshot(fileName, file.getAbsolutePath(), appName);
145147
if (appName == null) {
146148
// TODO: In the name of reusability of code, we have an ugly tmp single element array here
147-
// Find a better way for it. but I guess we should leave it is it,
149+
// Find a better way for it. but I guess we should leave it as it is,
148150
// Following the old adage of don't fix it if it's not broken
149151
ArrayList<Screenshot> tmp = new ArrayList<>();
150152
tmp.add(screen);

0 commit comments

Comments
 (0)