Remove default separator from itemize()

[mih]

This addresses an issues brought up in
#539 (comment)

This changeset removes the default argument to avoid the impression that "line-processing" is the main target. The code does not imply that, and the existing usage also not.

The possibility to do line-splitting is not touched (or removed). The documentation needs no adjustment.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (f2a6fe4) 92.66% compared to head (03e9500) 92.66%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #545   +/-   ##
=======================================
  Coverage   92.66%   92.66%           
=======================================
  Files         137      137           
  Lines       10031    10031           
  Branches     1085     1085           
=======================================
  Hits         9295     9295           
  Misses        712      712           
  Partials       24       24           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[christian-monch]

I think we are actually mostly line-based, even if the line terminator is often '\0'. For example, the git ls-files manpage says:

-z
    \0 line termination on output and do not quote filenames. See OUTPUT below for more information.

But I also think it makes sense to let the user provide a separator, especially because not providing a separator will reduce performance. But we should probably highlight the fact that passing None as a separator will result in using all standard line splitting patterns that are built into python.

[mih]

Thanks for the feedback!