Add todo comments about unicode ops

core/src/Streamly/Internal/Unicode/Array/Char.hs

@@ -6,6 +6,11 @@
 -- Maintainer  : streamly@composewell.com
 -- Stability   : experimental
 -- Portability : GHC
+
+-- XXX Remove this or move this to Unicode.Utf32 making "Array Char" as a
+-- newtype wrapper for Utf32. Is this any better than the [Char] (String) type?
+-- This provides random access and the length of the string in O(1). Also,
+-- better append performance.
 --
 module Streamly.Internal.Unicode.Array.Char
     (

core/streamly-core.cabal

@@ -327,7 +327,7 @@ library
                      -- streamly-serde
                      , Streamly.Internal.Data.Binary.Decode
 
-                    -- streamly-unicode
+                    -- streamly-unicode-core
                      , Streamly.Internal.Unicode.Stream
                      , Streamly.Internal.Unicode.String
                      , Streamly.Internal.Unicode.Char.Parser

src/Streamly/Internal/Unicode/Char.hs

@@ -6,12 +6,16 @@
 -- Maintainer  : streamly@composewell.com
 -- Stability   : experimental
 -- Portability : GHC
+
+-- XXX This module should have the reader/writer unfold/refold and read/write
+-- stream/fold routines to convert a char to/from stream.
 --
 module Streamly.Internal.Unicode.Char
     (
     -- * Predicates
-      isAsciiAlpha
+      isAsciiAlpha -- XXX Remove or move to unicode-data
 
+    -- XXX move to Unicode.Char.Case?
     -- * Unicode aware operations
     {-
       toCaseFold
@@ -20,6 +24,7 @@ module Streamly.Internal.Unicode.Char
     , toTitle
     -}
 
+    -- XXX Move to Unicode.Stream.Normalize or Unicode.Normalize?
     -- * Unicode normalization
     , NormalizationMode(..)
     , normalize

src/Streamly/Internal/Unicode/Utf8.hs

@@ -5,7 +5,12 @@
 -- Maintainer  : streamly@composewell.com
 -- Stability   : experimental
 -- Portability : GHC
---
+
+-- XXX We can move this to stream-core/streamly-unicode-core, and provide an
+-- additional module in streamly-unicode for case conversions (because it
+-- depends on unicode-data). Or just keep all of it in streamly-unicode
+-- which will have a dependency on unicode-data.
+
 module Streamly.Internal.Unicode.Utf8
     (
     -- * Type

streamly.cabal

@@ -365,7 +365,7 @@ library
                      , Streamly.Internal.Data.Stream.Time
                      , Streamly.Internal.Data.Stream.Exception.Lifted
 
-                     -- streamly-unicode
+                     -- streamly-unicode (depends on unicode-data)
                      , Streamly.Internal.Unicode.Utf8
                      , Streamly.Internal.Unicode.Char