Support lines of length more than 4096 bytes

[masahide]

Merged funkygao/tail@c5abc6d to correct the problem that occurs in the line of more than 4096 bytes.
And add test code.

[srid]

it may be best to make readLine return a slice of lines by splitting read lines per Config.MaxLineSize, as otherwise there is no limit on how much buf can grow.

[srid]

@masahide sorry - it looks like i overlooked something. two months ago, i noted above that the buf variable can grow unbounded, as the for loop doesn't split it per the MaxLineSize setting and return a slice of lines. did you make any code changes for this? i don't see it in the current diff.

[masahide]

Please try View full changes.

    if !isPrefix || tail.MaxLineSize > 0 {
        return line, err
    }

for loop is not performed if MaxLineSize is greater than or equal to zero.
Does this have a problem?

[masahide]

It seems that "the diff display" is wrong by "git push -f."

[masahide]

👍
Indeed!

[masahide]

Because it contains the buffer newlines code, add 2 bytes of CR/LF.

[masahide]

Please confirm it, tried to change to be set MaxLineSize to the bufio.reader.

[srid]

@masahide looks good. please rebase your branch, and then i'll merge it.

[masahide]

The rebasing.
May this be sufficient?
Or is it better to collect more?

[srid]

you need to rebase the master branch. your branch is outdated,
https://github.com/masahide/tail/commits/add-test-over4096byte-line

compare to,
https://github.com/ActiveState/tail/commits/master

try:

git remote add upstream git@github.com:ActiveState/tail.git
git fetch upstream
git rebase upstream/master

[masahide]

I am sorry.
I was mistaken.
I've rebase the master.
Please confirm.

[srid]

@masahide - see my line note abvove.

[srid]

all good, thanks for the work @masahide

[masahide]

thanks!