You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "sgilmore10 (via GitHub)" <gi...@apache.org> on 2023/06/20 13:28:22 UTC

[GitHub] [arrow] sgilmore10 opened a new issue, #36173: [C++] Add lone surrogate pair test case for `UTF8StringToUTF16`

sgilmore10 opened a new issue, #36173:
URL: https://github.com/apache/arrow/issues/36173

   ### Describe the enhancement requested
   
   Add lone lone surrogate pair test case for `UTF8StringToUTF16`.
   
   ### Component(s)
   
   C++


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] sgilmore10 commented on issue #36173: [C++] Add lone surrogate pair test case for `UTF8StringToUTF16`

Posted by "sgilmore10 (via GitHub)" <gi...@apache.org>.
sgilmore10 commented on issue #36173:
URL: https://github.com/apache/arrow/issues/36173#issuecomment-1613174507

   take


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] westonpace commented on issue #36173: [C++] Add lone surrogate pair test case for `UTF8StringToUTF16`

Posted by "westonpace (via GitHub)" <gi...@apache.org>.
westonpace commented on issue #36173:
URL: https://github.com/apache/arrow/issues/36173#issuecomment-1613155122

   Is this something you are interested in contributing?  Or is there a particular reason why you believe this is needed?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] pitrou closed issue #36173: [C++] Add lone high and low code-point test case for `UTF8StringToUTF16`

Posted by "pitrou (via GitHub)" <gi...@apache.org>.
pitrou closed issue #36173: [C++] Add lone high and low code-point test case for `UTF8StringToUTF16` 
URL: https://github.com/apache/arrow/issues/36173


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [arrow] sgilmore10 commented on issue #36173: [C++] Add lone surrogate pair test case for `UTF8StringToUTF16`

Posted by "sgilmore10 (via GitHub)" <gi...@apache.org>.
sgilmore10 commented on issue #36173:
URL: https://github.com/apache/arrow/issues/36173#issuecomment-1613174280

   Hi @westonpace,
   
   Apologies, I made this as a followup task for #36167. One reviewer [commented](https://github.com/apache/arrow/pull/36167#discussion_r1234856883) that it would be nice to include a test point for this case, but the PR was already merged. 
   
   I meant to submit a PR for this last week, but it slipped my mind. I just pushed my branch with the changes [here](https://github.com/mathworks/arrow/tree/GH-36173). I'll create a pull request for this issue today.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org