search

davidbien / lexang

Lexical Analyzer Generator Template Library
Boost Software License 1.0
0 stars 0 forks source link

Change _l_transport_fd_ctxt to have just a simple pointer and length instead of a segarray #9

Closed davidbien closed 3 years ago

davidbien commented 3 years ago

Just overkill to have the segmented array there - and large to have with each token - and unnecessary given that we know the size when we create it. We'll have to store the offset in it, however. If we rid the pointer back to the transport object we could do it with 3 u64s - which would make it the same size as the fixed memory ctxt and that's nice. We should also rid the pointer back to the transport for the fixed memory context since this just f's things up if we want to (and we could) use a token after the transport object has gone away.

davidbien commented 3 years ago

Fixed as described in the issue.