andrews05

Results 100 comments of andrews05

I'm on the older codebase but the 2.0 branch appears to work the same, just from looking at the code. There's no reference to 'data-role' besides the default value for...

A quick and easy way (though not necessarily the best way) that I've been dealing with this is just by using CSS overrides, e.g.: ``` css #my-editor font[size='3'] { font-size:...

Good question. I don't really know the answer but my gut feeling would be no, not tangibly. According to [pngcrush](https://pmt.sourceforge.io/pngcrush/ChangeLog.html) the Average and Paeth filters are slower to decode, and...

Hi @aleksusklim This seems quite strange, I ran that same command on oxipng 9 on aarch64-apple-darwin and it came to 6468192, the same as you showed for v8. @AlexTMjugador Any...

Weird, I just ran that exact command (now on x86_64-apple-darwin, 1.66.0-nightly) and got the smaller 6468192 result. [edit] Aha! Upgrading to latest rust now gives me the larger result. pngcheck...

> Do you remember what Rust version did not exhibit this problem? Both me and CI used nightly Rust to compile binaries, so that may explain why I was not...

@AlexTMjugador Any ideas on how we can track down the cause of this? I found a smaller (faster) test case: issue-29.png also shows a small increase between 1.72 and 1.74....

@AlexTMjugador Narrowed it down to this commit: https://github.com/zopfli-rs/zopfli/commit/90cfdd4c085bb87d55d3cb54b0ed0bdd21dde7cd On rust 1.72 it doesn't have any impact on output. But on rust 1.74 it's somehow different.

Nothing has changed in v9.1.x to address this, sorry.

@AlexTMjugador Afaik, zopflipng doesn't alter the ZOPFLI_MASTER_BLOCK_SIZE - it's a compile time constant which defaults to 1MB. So surely that wouldn't explain the other issue?