C++ Оптимизация чтения массива с пикселями

[THE LUPA]

Всем привет, пытаюсь оптимизировать чтение массива пикселей для быстрой работы.
Он читает массив пикселей в перевернутом виде, вот код.

 

[Ozaron]

А как у тебя эта функция вызывается?

 

[THE LUPA]

А как у тебя эта функция вызывается?

 

[idesync]

Всем привет, пытаюсь оптимизировать чтение массива пикселей для быстрой работы.
Он читает массив пикселей в перевернутом виде, вот код.
Посмотреть вложение 190523

Операции инкремента лучше префиксные использовать, они производительнее. Кроме этого больше ничего не могу сказать.

 

[THE LUPA]

Операции инкремента лучше префиксные использовать, они производительнее. Кроме этого больше ничего не могу сказать.

ты имеешь ввиду, так сделать?
Но это декремент. Инкремент в принципе уже везде был сделан, где идёт нагрузка.

 

[Ozaron]

А у тебя буффер вообще очищается?

 

[idesync]

ты имеешь ввиду, так сделать?
Но это декремент. Инкремент в принципе уже везде был сделан, где идёт нагрузка.
Посмотреть вложение 190530

У тебя BuffPos увеличивается постфиксными операторами инкремента, а не префиксными.
Советую сделать так: *(++BuffPos)

 

[THE LUPA]

А у тебя буффер вообще очищается?

Добавил, прибавилось +- 3 фпс :)

У тебя BuffPos увеличивается постфиксными операторами инкремента, а не префиксными.
Советую сделать так: *(++BuffPos)

щас, пробую спасибо
*(++BuffPos)
Начал получать не правильный цвет и ничего не изменилось

 

[timoxa5651]

Всем привет, пытаюсь оптимизировать чтение массива пикселей для быстрой работы.
Он читает массив пикселей в перевернутом виде, вот код.
Посмотреть вложение 190523

Во первых у тебя BitsPerPixel - константа, сравнение BitsPerPixel > 24 можно сделать один раз в самом начале. Во вторых если а) BitsPerPixel > 24 - true, и б) структура Pixel выглядит как struct Pixel { uint8_t B; uint8_t G; uint8_t R; uint8_t A; } ты можешь заменить весь for width на memcpy(&Pixels[(height - 1 - I) * width], BuffPos, 4 * width); BuffPos += 4 * width; В третьих я не очень понимаю зачем переворачивать весь огромный массив, если можно получать только тот пиксель, что тебе нужен(Pixel px = *(Pixel*)&BuffPos[(height - 1 - y) * width + x])

 

[Porches]

Насколько я понял, изучив твой код, можно сделать кэширование

 

[THE LUPA]

Во первых у тебя BitsPerPixel - константа, сравнение BitsPerPixel > 24 можно сделать один раз в самом начале. Во вторых если а) BitsPerPixel > 24 - true, и б) структура Pixel выглядит как struct Pixel { uint8_t B; uint8_t G; uint8_t R; uint8_t A; } ты можешь заменить весь for width на memcpy(&Pixels[(height - 1 - I) * width], BuffPos, 4 * width); BuffPos += 4 * width; В третьих я не очень понимаю зачем переворачивать весь огромный массив, если можно получать только тот пиксель, что тебе нужен(Pixel px = *(Pixel*)&BuffPos[(height - 1 - y) * width + x])

Спасибо огромное

 

[THE LUPA]

Во первых у тебя BitsPerPixel - константа, сравнение BitsPerPixel > 24 можно сделать один раз в самом начале. Во вторых если а) BitsPerPixel > 24 - true, и б) структура Pixel выглядит как struct Pixel { uint8_t B; uint8_t G; uint8_t R; uint8_t A; } ты можешь заменить весь for width на memcpy(&Pixels[(height - 1 - I) * width], BuffPos, 4 * width); BuffPos += 4 * width; В третьих я не очень понимаю зачем переворачивать весь огромный массив, если можно получать только тот пиксель, что тебе нужен(Pixel px = *(Pixel*)&BuffPos[(height - 1 - y) * width + x])

Я считываю весь массив для получения всех пикселей.
Чтобы потом из этого массива, получать определённую область с пикселями равные нужному мне цвету.