2 mo. ago • 98%

`getStorageList()`

15 comments

No usages

78
- Well its public so who knows in what other package is using it. And it's static, so who know what kind if monster is calling it from some inapropriate place. I have seen static methods being called from translation/localization files, because "it works".
  
  40
  
  IntelliJ finds most uses in my experience unless you're doing something weird with reflection or similar. And if it's a public facing API only used by the library's consumers..– it should be used in tests at the very least! Especially if it's prone to regressions like the comment suggests
  
  15
- Somebody should tell Albert and the others, they can let this method go.
  
  30
  
  I already did so an hour ago
  
  UPDATE: It has been merged
  
  30
Any context?

35
- Found it while refactoring KDE Connect Android.
  
  59
  
  Found the masterpiece: https://github.com/KDE/kdeconnect-android/blob/d41ed4d911959beef923d0d9a9c74072f0dd920a/src/org/kde/kdeconnect/Helpers/StorageHelper.java#L56
  
  40
Ah yes, that's Android for ya.

21
I keep telling myself that in the ideal world, phones would be programmed in Forth.

6

You've viewed 15 comments.