diff options
author | Paul Keith <javelinanddart@gmail.com> | 2019-02-17 05:20:04 +0100 |
---|---|---|
committer | Paul Keith <javelinanddart@gmail.com> | 2019-02-17 23:31:00 +0100 |
commit | c9f1d6744fa58796382d369579f891107d59bd67 (patch) | |
tree | c87df116c70aa8a34b2a61f52a88b1c1edb9ebe0 | |
parent | d6a273df7424921c3c777ab7dc6b364f5001b805 (diff) | |
download | lineage-sdk-c9f1d6744fa58796382d369579f891107d59bd67.tar.gz lineage-sdk-c9f1d6744fa58796382d369579f891107d59bd67.tar.bz2 lineage-sdk-c9f1d6744fa58796382d369579f891107d59bd67.zip |
sdk: Cleanup usage of List.toArray(T[] a)
* The parameter here only has to have the type you want it to return,
so creating an array of size > 0 is unnecessary and wasteful
Change-Id: I28e490fb6fa3703d7edca21b29d640105072947b
10 files changed, 13 insertions, 15 deletions
diff --git a/host/migration/src/AdbCommand.java b/host/migration/src/AdbCommand.java index 54cdb7b7..bdc54640 100644 --- a/host/migration/src/AdbCommand.java +++ b/host/migration/src/AdbCommand.java @@ -54,7 +54,7 @@ public class AdbCommand extends Command { List<String> commandList = new ArrayList<String>( baseCommand.length + 1); commandList.addAll(Arrays.asList(baseCommand)); - String[] commands = commandList.toArray(new String[commandList.size()]); + String[] commands = commandList.toArray(new String[0]); if (MigrationTest.DEBUG) { System.out.println("Using commands: " + Arrays.toString(commands)); diff --git a/host/migration/src/InsertCommand.java b/host/migration/src/InsertCommand.java index 0e435570..db2acd09 100644 --- a/host/migration/src/InsertCommand.java +++ b/host/migration/src/InsertCommand.java @@ -59,7 +59,7 @@ public class InsertCommand extends Command { commandList.add("--bind name:" + setting.getKeyType() + ":" + setting.getKey()); commandList.add("--bind value:" + setting.getValueType() + ":" + "\"" + setting.getValue() + "\""); - commands = commandList.toArray(new String[commandList.size()]); + commands = commandList.toArray(new String[0]); if (MigrationTest.DEBUG) { System.out.println("Using commands: " + Arrays.toString(commands)); } diff --git a/host/migration/src/QueryCommand.java b/host/migration/src/QueryCommand.java index 6d5c1dc8..69be0b21 100644 --- a/host/migration/src/QueryCommand.java +++ b/host/migration/src/QueryCommand.java @@ -63,7 +63,7 @@ public class QueryCommand extends Command { commandList.add(PROJECTION); commandList.add("--show-type"); //this is totally awesomely lineage specific commandList.add("true"); - commands = commandList.toArray(new String[commandList.size()]); + commands = commandList.toArray(new String[0]); if (MigrationTest.DEBUG) { System.out.println("Using commands: " + Arrays.toString(commands)); } diff --git a/lineage/lib/main/java/org/lineageos/platform/internal/LineageWeatherManagerService.java b/lineage/lib/main/java/org/lineageos/platform/internal/LineageWeatherManagerService.java index 6515fb8f..54ba4079 100644 --- a/lineage/lib/main/java/org/lineageos/platform/internal/LineageWeatherManagerService.java +++ b/lineage/lib/main/java/org/lineageos/platform/internal/LineageWeatherManagerService.java @@ -375,10 +375,9 @@ public class LineageWeatherManagerService extends LineageSystemService { contentValuesList.add(contentValues); } - ContentValues[] updateValues = new ContentValues[contentValuesList.size()]; if (size != getContext().getContentResolver().bulkInsert( WeatherColumns.CURRENT_AND_FORECAST_WEATHER_URI, - contentValuesList.toArray(updateValues))) { + contentValuesList.toArray(new ContentValues[0]))) { Slog.w(TAG, "Failed to update the weather content provider"); return false; } diff --git a/lineage/lib/main/java/org/lineageos/platform/internal/PerformanceManagerService.java b/lineage/lib/main/java/org/lineageos/platform/internal/PerformanceManagerService.java index 9545dc39..5c17d92a 100644 --- a/lineage/lib/main/java/org/lineageos/platform/internal/PerformanceManagerService.java +++ b/lineage/lib/main/java/org/lineageos/platform/internal/PerformanceManagerService.java @@ -378,8 +378,7 @@ public class PerformanceManagerService extends LineageSystemService { @Override public PerformanceProfile[] getPowerProfiles() throws RemoteException { synchronized (mLock) { - return mProfiles.values().toArray( - new PerformanceProfile[mProfiles.size()]); + return mProfiles.values().toArray(new PerformanceProfile[0]); } } diff --git a/lineage/lib/main/java/org/lineageos/platform/internal/ProfileManagerService.java b/lineage/lib/main/java/org/lineageos/platform/internal/ProfileManagerService.java index a46c783a..d2ac06fb 100644 --- a/lineage/lib/main/java/org/lineageos/platform/internal/ProfileManagerService.java +++ b/lineage/lib/main/java/org/lineageos/platform/internal/ProfileManagerService.java @@ -410,7 +410,7 @@ public class ProfileManagerService extends LineageSystemService { @Override public Profile[] getProfiles() { - Profile[] profiles = getProfileList().toArray(new Profile[mProfiles.size()]); + Profile[] profiles = getProfileList().toArray(new Profile[0]); Arrays.sort(profiles); return profiles; } @@ -492,7 +492,7 @@ public class ProfileManagerService extends LineageSystemService { @Override public NotificationGroup[] getNotificationGroups() { - return mGroups.values().toArray(new NotificationGroup[mGroups.size()]); + return mGroups.values().toArray(new NotificationGroup[0]); } @Override diff --git a/lineage/lib/main/java/org/lineageos/platform/internal/display/DisplayHardwareController.java b/lineage/lib/main/java/org/lineageos/platform/internal/display/DisplayHardwareController.java index 85fe9a3f..3f4bd452 100644 --- a/lineage/lib/main/java/org/lineageos/platform/internal/display/DisplayHardwareController.java +++ b/lineage/lib/main/java/org/lineageos/platform/internal/display/DisplayHardwareController.java @@ -136,7 +136,7 @@ public class DisplayHardwareController extends LiveDisplayFeature { return; } - registerSettings(settings.toArray(new Uri[settings.size()])); + registerSettings(settings.toArray(new Uri[0])); } @Override diff --git a/sdk/src/java/lineageos/app/Profile.java b/sdk/src/java/lineageos/app/Profile.java index 9b02b8d8..5ca5fcf7 100755 --- a/sdk/src/java/lineageos/app/Profile.java +++ b/sdk/src/java/lineageos/app/Profile.java @@ -507,7 +507,7 @@ public final class Profile implements Parcelable, Comparable { * @hide */ public ProfileGroup[] getProfileGroups() { - return profileGroups.values().toArray(new ProfileGroup[profileGroups.size()]); + return profileGroups.values().toArray(new ProfileGroup[0]); } /** @@ -566,7 +566,7 @@ public final class Profile implements Parcelable, Comparable { uuids.add(new ParcelUuid(u)); } dest.writeInt(1); - dest.writeParcelableArray(uuids.toArray(new Parcelable[uuids.size()]), flags); + dest.writeParcelableArray(uuids.toArray(new Parcelable[0]), flags); } else { dest.writeInt(0); } @@ -766,7 +766,7 @@ public final class Profile implements Parcelable, Comparable { * @return the secondary uuids for the Profile */ public UUID[] getSecondaryUuids() { - return mSecondaryUuids.toArray(new UUID[mSecondaryUuids.size()]); + return mSecondaryUuids.toArray(new UUID[0]); } /** diff --git a/sdk/src/java/lineageos/hardware/LineageHardwareManager.java b/sdk/src/java/lineageos/hardware/LineageHardwareManager.java index 33a62cd2..4543ad90 100644 --- a/sdk/src/java/lineageos/hardware/LineageHardwareManager.java +++ b/sdk/src/java/lineageos/hardware/LineageHardwareManager.java @@ -679,7 +679,7 @@ public final class LineageHardwareManager { remapped.add(r); } } - return remapped.toArray(new DisplayMode[remapped.size()]); + return remapped.toArray(new DisplayMode[0]); } } diff --git a/sdk/src/java/lineageos/util/palette/Palette.java b/sdk/src/java/lineageos/util/palette/Palette.java index cf960dda..ca788891 100644 --- a/sdk/src/java/lineageos/util/palette/Palette.java +++ b/sdk/src/java/lineageos/util/palette/Palette.java @@ -567,7 +567,7 @@ public final class Palette { scaledBitmap.getPixels(pixels, 0, width, 0, 0, width, height); final ColorCutQuantizer quantizer = new ColorCutQuantizer(pixels, mMaxColors, - mFilters.isEmpty() ? null : mFilters.toArray(new Filter[mFilters.size()])); + mFilters.isEmpty() ? null : mFilters.toArray(new Filter[0])); // If created a new bitmap, recycle it if (scaledBitmap != mBitmap) { |