GitLab

... and GIMP_STOCK_BUFFER redefined to "edit-paste", half-reverting
commit cb7d93d5 (except we use standard Freedesktop icon naming instead
of the GTK stock name).
The gimp-buffer icon was indeed definitely looking like a typical copy
icon, which was missing, and this has priority over a buffer icon.

This should improve readability in gimp.c, and moves all data factory
handling code to a common place.

We should use "vector icons" instead. Thanks to nomis for raising the
question and ankh to confirm the proper English.

... since commits 36f87b5b and 2b6a10b7.
With the renaming, action-search was showing itself in its list.

Namely dialogs_attach_dialog(), _detach_dialog(), _get_dialog()

So we don't create the same dialogs multiple times, but raise the
already existing ones. Port some places that did this manually
to the new functions, and use it for many other dialogs.

skal authored Sep 07, 2016 and Michael Natterer committed Sep 11, 2016

WebP saving: various improvements in file-webp-save.c

- reorganizes the main loop
- fixes (temporarily) the bad timestamp by using fixed 100ms delay between frames
- fixes the progress bar
- takes care of resource clean-up for some previously unchecked error paths
- uses 'minimize_size' and 'allow_mixed' options more appropriately
- only remuxes the final bytestream when ICC profile is present. For
  most common case, we insert the loop-count information during muxer
  creation instead of by remuxing at the end.

GimpColorTransform abstracts that lcms problem away.

...after entering single window mode

In GimpToolPalette, implement GtkWidget::hierarchy_changed() and
re-set the tooltips using the new toplevel's actions.

In gimp_widget_set_accel_help(), make sure we don't connect to the
same action twice, and make sure we disconnect everything if either
the widget or the accel_group go away.

Use GTK_TYPE_SCALE with orientation = HORIZONTAL instead.

Timm Bäder authored Sep 09, 2016 and Michael Natterer committed Sep 09, 2016

The default implementation does just the right thing for most widgets.

In size_allocate(), call gimp_ruler_make_pixmap() only if the widget
was resized, not only moved around.

They are deprecated in GTK+ 3.x and cause tons of warnings, now we can
port calls to gtk_misc_set_alignment(label) to the new API and have
the same code in master and gtk3-port.

This complements the preceding commit "plug-ins: fix the build on OS X".

It's deprecated in GTK+ 3.x, so we get rid of some deprecation
warnings. Also, it has always only been a utility function that hides
what is really going on in the widget hierarchy.

We need to have -xobjective-c specified only for compile commands
that will perform compilation. Otherwise, for compile commands
that only link the compiler interprets .o files as C source ...

Try harder to show help when webkit is missing or the help browser
won't start:

- offer to switch to the web browser when the help browser is
  present, but won't start for some reason
- in prefs, don't bother showing the help browser GUI when webkit
  is missing, otherwise show a warning label if the help browser
  is not installed even though webkit is there. Switch to the web
  browser in both cases (modified patch from lisanet)
- add OS X replacement for gtk_show_uri() in plug-in-web-browser
  (modified patch from lisanet)

(gimp-2.9:3): Gimp-Core-CRITICAL **: gimp_marshal_VOID__OBJECT_POINTER: assertion 'n_param_values == 3' failed

Conflict between signal marshal function
and signal emission.

AddressSanitizer output (paths manually shortened)

Direct leak of 96 byte(s) in 1 object(s) allocated from:
    #0 0x7f543247d1d0 in realloc (/lib64/libasan.so.3+0xc71d0)
    #1 0x7f542d3ad02f in g_realloc glib/glib/gmem.c:159
    #2 0x57918d in gimp_color_history_set_property gimp/app/widgets/gimpcolorhistory.c:169
    #3 0x7f542d681ec0 in object_set_property glib/gobject/gobject.c:1423
    #4 0x7f542d681ec0 in g_object_new_internal glib/gobject/gobject.c:1817
    #5 0x7f542d6839ed in g_object_new_valist glib/gobject/gobject.c:2042
    #6 0x7f542d683c73 in g_object_new glib/gobject/gobject.c:1626

Commit c6f1761a deletes a stock item id. But this is also part of our API
and cannot be deleted. Yet since the associated label ("_Edit") is
identical to the one for GTK_STOCK_EDIT, we can simply redefine it
to "gtk-edit".
Ideally we would also add a deprecation warning when this macro is used,
but I can't find a good way to add deprecation on macros.

instead of just "require 'plugin-defs.pl';".

Add support for writing linear PNGs. But only if we also save a color
profile, otherwise we would get loaders confused. Please test, I might
have a knot in my brain...

Also clean up some formatting and fix compile warnings, now let's wait
for bugs about shit libpng versions which break the build.

Klaus Staedtler authored Sep 04, 2016 and Jehan committed Sep 05, 2016

These icons were massively renamed by Jehan, from a feature branch, in
order to fit either into Freedesktop's "Icon Naming Specification", or
as standard GTK+ icons.

Klaus Staedtler authored Sep 04, 2016 and Jehan committed Sep 05, 2016

As squashed by Jehan from the work of Klaus Staedtler in a feature
branch.

skal authored Aug 24, 2016 and Michael Natterer committed Sep 04, 2016

...and transparency

Use most recent API for decoding WebP animation. WebPAnimDecoder
handles transparency and blending automatically.

- handle all sorts of image types, gegl_buffer_get() will convert to u8
- specify GIMP_EXPORT_CAN_HANDLE_LAYERS_AS_ANIMATION
- modify the export duplicate, not the original image
- fix handling of the drawable type and only save "R'G'B'A u8"
  or "R'G'B' u8", fixes bug 770664.
- completely redo run(SAVE_PROC) to deal with export correctly
- turn global variables into the standard SAVE_PROC, PLUG_IN_BINARY etc
  defines
- simplify the save dialog

MihailZenkov authored Sep 04, 2016 and Michael Natterer committed Sep 04, 2016

Use the drawable's (indexed bitmap) format instead of "Y' u8" when
exporting to PBM, this got broken when porting to GEGL.

if the plug-in specifies GIMP_EXPORT_CAN_HANDLE_LAYERS_AS_AMINATION
(instead of defaulting to flattening or merging visible layers).