Read-Only ByteBuffer & Error Codes #3
Conversation
| image.height, | ||
| false | ||
| ) | ||
| val source = try { |
There was a problem hiding this comment.
Die Lösung bekämpft im Prinzip die Symptome und nicht die Ursache oder? Ist für mich ok, aber könnte man nicht verhindern, dass es überhaupt zu der Exception kommt?
There was a problem hiding this comment.
Jein. Das Problem das zur BufferUnderflowException führte, war wenn zwei ImageDecoders (also GlobalHistogram und Hybrid) parallel auf den gleichen Plane Buffer im ImageProxy zugegriffen haben. ByteBuffer ist nicht wirklich Threadsafe weil intern die Leseposition als State gespeichert wird. Die wichtige Zeile ist also auf Zeile 45 wo mit .asReadOnlyBuffer() eine Kopie des ByteBuffers (mit gleichem Inhalt, aber separiertem State) erstellt wird bevor diese an die LuminanceSource mitgegeben wird. Das Try-Catch hab ich sicherheitshalber eingebaut, damit dem Client bei einem einzelnen fehlerhaften Frame keine Exception sondern ein DecodingState.Error geliefert wird.
There was a problem hiding this comment.
Ah ok, den change hab ich übersehen. Dann passts. Danke!
No description provided.