src/io/utf8.c: Does the amount read in Parrot_io_read_utf8() need to be checked?

[jkeenan]

This ticket moves into Trac discussion of an issue originally tracked in RT at  RT #46413.

In that RT, Paul Cochrane cited lines in the file now known as src/io/utf8.c:

 46 size_t
 47 Parrot_io_read_utf8(PARROT_INTERP, ARGMOD(PMC *filehandle),
 48         ARGMOD(STRING **buf))
 49 {
...
 83                 /* RT#46413 need to check the amount read here? */
 84                 read         = Parrot_io_read_buffer(interp, filehandle, &s2    );          
 85                 UNUSED(read);

Paul commented, "At present this variable is marked UNUSED(read). However, it might be a good idea to actually check the amount read in this function call. After which the UNUSED() needs to disappear."

Can someone follow up?

[jkeenan]

Reviewing this old ticket today leads me to ask, "Can anyone suggest what we might do with the amount read here?"

If not, I will remove the TODO item and close this ticket within 7 days.

Thank you very much.

kid51

[jkeenan]

No one spoke up to suggest "something we might do with the amount read" there, so I eliminated the comment in r48777. Closing ticket.

Thank you very much.

kid51