[GitHub] [incubator-pegasus] neverchanje commented on pull request #1240: refactor: use state machine to improve split_args for strings

[GitBox <gi...@apache.org>]

neverchanje commented on PR #1240:
URL: https://github.com/apache/incubator-pegasus/pull/1240#issuecomment-1313293245

   I believe that we shouldn't invest too much effort in writing commonly-used utilities on our own. Instead, we should find if there's a trustworthy library providing this functionality, such as https://abseil.io/docs/cpp/guides/strings#abslstrsplit-for-splitting-strings or https://www.boost.org/doc/libs/1_80_0/doc/html/string_algo/usage.html.
   
   I feel that C++ coders rarely consider relying on the community. Programmers using other languages would never think of writing their string split unless there are some special requirements. Therefore, if possible, please try out the above two libraries and see if they can address your problem.


-- 
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: dev-unsubscribe@pegasus.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@pegasus.apache.org
For additional commands, e-mail: dev-help@pegasus.apache.org