Кирпичные обои для Android

Выход за пределы границ массива

private int[] getHistogram(Bitmap grayscale) {
   int width = grayscale.getWidth();
   int height = grayscale.getHeight();
   // TODO: Fine tune the performance here, tracking on b/123615079.
   int[] histogram = new int[256];
   for (int row = 0; row < height; row++) {
      for (int col = 0; col < width; col++) {
         int pixel = grayscale.getPixel(col, row);
         int y = Color.red(pixel) + Color.green(pixel) + Color.blue(pixel);
         histogram[y]++;
      }
   }

Ну это ещё ладно…

А как вот с таким подходом потом памятью жонглировать?

А почему grayscale value получается вне диапозона 0…255?

Ниработает

Да, действительно, туплю. Осталось понять, почему ошибка не воспроизводилась ранее, ведь найти картинку, у которой сумма компонент будет больше 255, должно быть совсем несложно. Это любой совсем белый пиксел, например. Или серый, который ярче 34% серого, что должно встречаться очень часто.

Ну так вылет недалеко обычно ничего страшного не представляет, а чем дальше тем хуже.

И выделение памяти опасное.

Обои ставят раком Android смартфоны

А если просто открыть картинку в фуллскрине тоже сработает? Почему не ломает другие девайсы, на андроиде решили написать свою парсилку изображений?

Интересно, это никаких патентов эпола не нарушает? Или там только про юникод?

ЗЫ: на картинках нет копытных

Но почему?!

Так ведь в Java в отличии от c++ всегда проверяются границы массивов, не? Если вылет будет, то в 100% случаев будет exception и если его никто не поймает, то приложение упадёт (и если это что-то системное, то может быть плохо). Так чтобы вылетел за границу и иногда падал иногда нет не должно быть.

Блин, я не заметил что это Java.

Ну так ещё проще.

на картинках нет копытных

А как хотелось…

skia на С++

Фрагмент кода, который цитировали выше - на Java.