Sign up now! · Forgot password?
RSS/Atom feed identi.ca Twitter

Bug 312800: Selective gaussian blur should be able to read delta and blur from different sources

This discussion is connected to the gimp-developer-list.gnome.org mailing list which is provided by the GIMP developers and not related to gimpusers.com.

This is a read-only list on gimpusers.com so this discussion thread is read-only, too.

Karthikeyan S
2011-12-15 16:46:14 UTC (almost 3 years ago)

Bug 312800: Selective gaussian blur should be able to read delta and blur from different sources

I took the patch given earlier by Luis de* and made the changes suggested above along with code for updating preview.

Attaching the patch. This currently does not work on mmx since we totally ignore the delta layer when calling mmx routine. But doesn;t break mmx.

Working on mmx code changes...

-karthik

Karthikeyan S
2011-12-15 16:48:09 UTC (almost 3 years ago)

Bug 312800: Selective gaussian blur should be able to read delta and blur from different sources

By "suggested above" I mean @
https://bugzilla.gnome.org/show_bug.cgi?id=312800

On Thu, Dec 15, 2011 at 10:16 PM, Karthikeyan S wrote:

I took the patch given earlier by Luis de* and made the changes suggested above along with code for updating preview.

Attaching the patch. This currently does not work on mmx since we totally

ignore the delta layer when calling mmx routine. But doesn;t break mmx.

Working on mmx code changes...

-karthik

Karthikeyan S
2011-12-19 16:38:42 UTC (almost 3 years ago)

Bug 312800: Selective gaussian blur should be able to read delta and blur from different sources

Final Patch.
Changed the mmx code to use the another layer than the current one for getting delta.

Please review.
If this is a useless patch since the request was too long ago, let me know.

-Karthik

On Thu, Dec 15, 2011 at 10:18 PM, Karthikeyan S wrote:

By "suggested above" I mean @
https://bugzilla.gnome.org/show_bug.cgi?id=312800

On Thu, Dec 15, 2011 at 10:16 PM, Karthikeyan S wrote:

I took the patch given earlier by Luis de* and made the changes suggested above along with code for updating preview.

Attaching the patch. This currently does not work on mmx since we totally

ignore the delta layer when calling mmx routine. But doesn;t break mmx.

Working on mmx code changes...

-karthik