On 28 November 2023 19:56:27 UTC, Blady <blady-...@users.sf.net> wrote:
>
>
>> Le 25 nov. 2023 à 19:58, Jeffrey R. Carter via Gnoga-list 
>> <gnoga-list@lists.sourceforge.net> a écrit :
>> 
>> On 25 November 2023 18:49:46 UTC, Blady <blady-...@users.sf.net> wrote:
>>> 
>>> 
>>>> Le 18 nov. 2023 à 21:26, J Carter <jc...@ymail.com> a écrit :
>>>> 
>>>>> I've upgraded Simple Components in dev_1.8 branch: 
>>>>> https://sourceforge.net/p/gnoga/code/ci/dev_1.8/tree/
>>>>> I've tested ok with GNAT FSF 13.2 from Alire.
>>>> 
>>>> That compiles. Attached is a reproducer, data_error.adb, and corresponding 
>>>> project file (edit the path to gnoga.gpr as needed).
>>>> I've capture the javascript string:
>>>> Array.prototype.join.call (gnoga['g12'].data)
>>> < 
>>> "255,0,0,255,255,0,0,255,255,0,0,255,255,0,0,255,0,128,0,254,0,128,0,255,0,128,0,255,0,128,0,254,0,0,252,252,0,0,255,255,0,0,255,255,0,0,252,252"
>>> 
>>> The documentation says:
>>> ImageData.data Read only A Uint8ClampedArray representing a one-dimensional 
>>> array containing the data in the RGBA order, with integer values between 0 
>>> and 255 (inclusive). The order goes by rows from the top-left pixel to the 
>>> bottom-right.
>>> (https://developer.mozilla.org/en-US/docs/Web/API/ImageData)
><...>
>> Yes, that's correct. The subprogram that goes the other way also has the 
>> loops in the wrong order.
>
>Hello Jeff,
>
>I've pushed a patch on dev_1.8 branch: 
>https://sourceforge.net/p/gnoga/code/ci/70be52accab63c4866aa8dfba32e759db744a134/
>
>Is it correct for you?
>
>I've also included your reproducer, do you mind?
>
>Regards, Pascal.
>https://blady.pagesperso-orange.fr
>
>

Yes, that looks good. I don't have a problem with including the reproducer, but 
it could use some comments. Also, since you've fixed the problem, Data_Error is 
no longer an appropriate name. 
--
Jeff Carter
"Now go away or I shall taunt you a second time."
Monty Python and the Holy Grail
_______________________________________________
Gnoga-list mailing list
Gnoga-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/gnoga-list

Reply via email to