Don't inject stuff into the text #13

New Issue

tastytea · 2021-06-06T16:33:56+02:00

tastytea commented

2021-06-06 16:33:56 +02:00

Right now we replace headlines with <H>headline</H> and page numbers with <PAGE 22>. It would be better to record the positions of headlines and pages and not inject anything. This way, headlines and page numbers would be accessible for --raw searches too.

Right now we replace headlines with `<H>headline</H>` and page numbers with `<PAGE 22>`. It would be better to record the positions of headlines and pages and not inject anything. This way, headlines and page numbers would be accessible for `--raw` searches too.

tastytea added the

enhancement

label 2021-06-06 16:33:56 +02:00

tastytea commented

2021-06-06 21:29:59 +02:00

In order to have page numbers and headlines with --raw, I would have to duplicate the code or make it far more complicated. I don't think it is worth the effort at the moment.

In order to have page numbers and headlines with `--raw`, I would have to duplicate the code or make it far more complicated. I don't think it is worth the effort at the moment.

tastytea referenced this issue from a commit

2021-06-06 21:30:04 +02:00

Don't inject page numbers and headline-markers into the text.

tastytea closed this issue

2021-06-06 21:30:04 +02:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: tastytea/epubgrep#13