[KevinH]

Hi,

Your link_pattern used if FILEPOS_ON_ALL_ANCHORS is True seems to be a bit broken:

For example: here is what the rawml says for one link:

<a filepos=0000006414 >M<span><font size="2">APS</font></span></a>

but this link is never properly detected or processed by your link pattern:

link_pattern = re.compile(r'''<a\s*(></a>|/>)''', re.IGNORECASE)

So you might want to take another look at your link patterns to make sure rawml of this type gets processed properly.

Hope this helps,

KevinH

Originally Posted by KevinH

Hi,

Thanks for posting! I grabbed it and tried it on a bunch of mobis I had and unfortunately, the internal links anchors from many of the internal links in the document no longer work. I tested it with mobiunpack version 31 without your changes and all internal links worked.

So somehow your changes have broken some of the internal links.
I will try to track this down.

I did get some form of NCX file but it was incomplete and there were error messages:

Write html
ERROR: last byte not 0x80
ERROR: text not found 1354424
Wite ncx
Write opf

I will keep playing with it to see if I get get the internal links working again.

Thanks for getting this ncx stuff going!

KevinH