November 2010 - Dillo-dev - lists.mailman3.com

[patch]: fix .hgignore

by onepoint＠starurchin.org

On Sat, Nov 27, 2010 at 03:28:36PM +0000, corvid wrote: > Back in July, Jeremy wrote: > > corvid wrote: > > > Jeremy wrote: > > > > The problem was that .hgignore contains a line "dillo", which > > > > ignores every file with the string "dillo" anywhere in its > > > > path. Many other lines in .hgignore are similarly > > > > inappropriate. It looks as though these lines were written to > > > > be interpreted as glob-style patterns, but in fact mercurial > > > > interprets every line as a regexp. > > > > > > Looking in http://www.selenic.com/mercurial/hgignore.5.html it > > > seems that specifying "syntax: glob" is also a possibility. > > > > Yes, glob syntax might well be better, but either way the current > > .hgignore is not working as it should and could do with fixing. > > Just noticed this thread while cleaning out old mail, and we should > come to some kind of decision on this. Heh, I was *just* about to bump this discussion by resubmitting my patch! Is there any reason not to push this? Unless people would prefer glob syntax? Regards, Jeremy Henty

Nov. 27, 2010

[patch]: fix .hgignore

by corvid＠lavabit.com

Back in July, Jeremy wrote: > corvid wrote: > > Jeremy wrote: > > > The problem was that .hgignore contains a line "dillo", which > > > ignores every file with the string "dillo" anywhere in its path. > > > Many other lines in .hgignore are similarly inappropriate. It > > > looks as though these lines were written to be interpreted as > > > glob-style patterns, but in fact mercurial interprets every line > > > as a regexp. > > > > Looking in http://www.selenic.com/mercurial/hgignore.5.html it seems > > that specifying "syntax: glob" is also a possibility. > > Yes, glob syntax might well be better, but either way the current > .hgignore is not working as it should and could do with fixing. Just noticed this thread while cleaning out old mail, and we should come to some kind of decision on this.

Nov. 27, 2010

Dillo needs scripting

by onepoint＠starurchin.org

I thought it would be fun if I could write a DPI filter as easily as this: #!/usr/lib/dillo/bin/lua-dpi -- -*- lua -*- dpi [[ Content-type: text/html <!DOCTYPE HTML PUBLIC '-//W3C//DTD HTML 4.01 Transitional//EN'> <html> <head><title>Hello World - Lua DPI</title></head> <body> From inside a Dillo DPI, Lua says "Hello World!". </body> </html> ]] Well, now I can! http://starurchin.org/dillo/dillo_lua_dpi.png It's still just a toy at the moment, but I'm pleased with it. Regards, Jeremy Henty

Nov. 26, 2010

[patch]: @media support

by onepoint＠starurchin.org

corvid wrote: > Jeremy wrote: > > + dStrcasecmp(tval, "media") == 0) > > Should that be "screen"? <headdesk/> :-) . Good call! Updated patch attached. Regards, Jeremy Henty

Nov. 22, 2010

[patch]: @media support

by corvid＠lavabit.com

Jeremy wrote: > + dStrcasecmp(tval, "media") == 0) Should that be "screen"?

Nov. 22, 2010

[patch]: @media support

by Johannes.Hofmann＠gmx.de

On Mon, Nov 22, 2010 at 08:29:03PM +0000, Jeremy Henty wrote: > > Does what it says on the tin. Please review and comment. I'm going > to road-test this at home for a few days before pushing it. Naturally > I won't push it if people have objections or corrections. But it does > fix this site[1]! Heh, nice! After your last patch I was already tempted to ask whether you wanted to look into this :-) Feel free to commit when you think it's ready. Cheers, Johannes

Nov. 22, 2010

[patch]: @media support

by onepoint＠starurchin.org

Does what it says on the tin. Please review and comment. I'm going to road-test this at home for a few days before pushing it. Naturally I won't push it if people have objections or corrections. But it does fix this site[1]! Regards, Jeremy Henty [1]: http://www.psychologicalscience.org/onlyhuman/2009/01/paradox-of-temptation…

Nov. 22, 2010

[patch]: ignore unknown @-rules in CSS

by onepoint＠starurchin.org

On Sun, Nov 21, 2010 at 08:52:58PM +0100, Johannes Hofmann wrote: > On Sat, Nov 20, 2010 at 12:31:55PM +0000, Jeremy Henty wrote: > > > > So, here are two patches that make Dillo ignore @-rules that it > > does not recognise. > > Looks good to me. Please commit! Done! Regards, Jeremy Henty

Nov. 21, 2010

[patch]: ignore unknown @-rules in CSS

by Johannes.Hofmann＠gmx.de

Hi Jeremy, On Sat, Nov 20, 2010 at 12:31:55PM +0000, Jeremy Henty wrote: > > Currently Dillo tries to parse every @-rule in the same way (with > CssParser::parseImport()). This causes problems, particularly with > @media. Here[1] part of the style is missing so that the line size is > smaller than the text. I have also attached a small test case where a > @media rule causes the red style to go missing. > > So, here are two patches that make Dillo ignore @-rules that it does > not recognise. The first patch just moves the test for the import > keyword from CssParser::parseImport() into the main parser loop. > Because it changes indentation it looks much more radical than it is. > The second patch reworks the main parser loop so that > CssParser::parseImport() is only called for @import and other @-rules > are skipped. > > Please review and comment. I will push the patches if you think they > are good. Looks good to me. Please commit! Thanks, Johannes

Nov. 21, 2010

[patch]: ignore unknown @-rules in CSS

by onepoint＠starurchin.org

Currently Dillo tries to parse every @-rule in the same way (with CssParser::parseImport()). This causes problems, particularly with @media. Here[1] part of the style is missing so that the line size is smaller than the text. I have also attached a small test case where a @media rule causes the red style to go missing. So, here are two patches that make Dillo ignore @-rules that it does not recognise. The first patch just moves the test for the import keyword from CssParser::parseImport() into the main parser loop. Because it changes indentation it looks much more radical than it is. The second patch reworks the main parser loop so that CssParser::parseImport() is only called for @import and other @-rules are skipped. Please review and comment. I will push the patches if you think they are good. Regards, Jeremy Henty [1]http://johannhari.com//2010/11/19/we-should-not-tolerate-religious-excuse…

Nov. 20, 2010