Fix a regression in reverse_lookup_filter introduced in #699

[ksqsf]

Pull request

Issue tracker

Reported by @eagleoflqj ; Fix a regression introduced by #699

#699 mistakenly assumed at least one of append_comment and overwrite_comment is true, and will break the old behavior. This PR fixes this.

Sorry for my mistake :(

Feature

Unit test

• Done

Manual test

• Done

Code Review

1. Unit and manual test pass
2. GitHub Action CI pass
3. At least one contributor reviews and votes
4. Can be merged clean without conflicts
5. PR will be merged by rebase upstream base

Additional Info

[eagleoflqj]

Changed the logic a bit.

seq	overwrite	append	empty	e825f7d	fd1a6dc
0	0	0	0	return early	return early
1	0	0	1	overwrite	overwrite
2	0	1	0	append	append
3	0	1	1	append	overwrite
4-7	1	?	?	overwrite	overwrite

The only difference with your commit is when !overwrite && append && empty, I think in this case better not adding a extraneous space.