
Looking further at this widget, many things are not right. Here are the
changes:
- Use binary prefixes (i.e. kibibyte, mebibyte and gibibyte) instead of
decimal ones. We are making binary shifts so we were actually showing
the wrong units.
- Round the value to the closest integer when showing it, not towards 0.
Otherwise I had cases where it was showing 7GiB for an actual value of
7.69GiB (default as computed by GIMP from my actual physical memory).
Note that I am actually unsure even rounding makes sense. Shouldn't we
rather show double values with a few digits after the decimal points?
For such values, I think it would make sense.
- Do not edit the internally saved accurate value when the entry is
edited to the same less accurate value as our saved value would be
shown too. In particular when changing the display unit to a bigger
one, we don't want to lose accuracy. This is especially true for low
values. Say you don't have a lot of memory and you set the Tile cache
size to 1.5GiB (1536MiB), you certainly don't want it to become either
1 or 2GiB when switching display unit to GiB. Now even if the number
will still display with less accuracy, the internal value will stay
accurate.
(cherry picked from commit d886bb1b90
)
321 lines
9.1 KiB
C
321 lines
9.1 KiB
C
/* LIBGIMP - The GIMP Library
|
|
* Copyright (C) 1995-1997 Peter Mattis and Spencer Kimball
|
|
*
|
|
* gimpmemsizeentry.c
|
|
* Copyright (C) 2000-2003 Sven Neumann <sven@gimp.org>
|
|
*
|
|
* This library is free software: you can redistribute it and/or
|
|
* modify it under the terms of the GNU Lesser General Public
|
|
* License as published by the Free Software Foundation; either
|
|
* version 3 of the License, or (at your option) any later version.
|
|
*
|
|
* This library is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
|
* Library General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU Lesser General Public
|
|
* License along with this library. If not, see
|
|
* <https://www.gnu.org/licenses/>.
|
|
*/
|
|
|
|
#include "config.h"
|
|
|
|
#include <gegl.h>
|
|
#include <gtk/gtk.h>
|
|
|
|
#include "gimpwidgetstypes.h"
|
|
|
|
#include "gimpmemsizeentry.h"
|
|
#include "gimpspinbutton.h"
|
|
#include "gimpwidgets.h"
|
|
|
|
#include "libgimp/libgimp-intl.h"
|
|
|
|
|
|
/**
|
|
* SECTION: gimpmemsizeentry
|
|
* @title: GimpMemSizeEntry
|
|
* @short_description: A composite widget to enter a memory size.
|
|
*
|
|
* Similar to a #GimpSizeEntry but instead of lengths, this widget is
|
|
* used to let the user enter memory sizes. A combo box allows one to
|
|
* switch between Kilobytes, Megabytes and Gigabytes. Used in the GIMP
|
|
* preferences dialog.
|
|
**/
|
|
|
|
|
|
enum
|
|
{
|
|
VALUE_CHANGED,
|
|
LAST_SIGNAL
|
|
};
|
|
|
|
|
|
static void gimp_memsize_entry_finalize (GObject *object);
|
|
|
|
static void gimp_memsize_entry_adj_callback (GtkAdjustment *adj,
|
|
GimpMemsizeEntry *entry);
|
|
static void gimp_memsize_entry_unit_callback (GtkWidget *widget,
|
|
GimpMemsizeEntry *entry);
|
|
|
|
static guint64 gimp_memsize_entry_get_rounded_value (GimpMemsizeEntry *entry,
|
|
guint64 value);
|
|
|
|
G_DEFINE_TYPE (GimpMemsizeEntry, gimp_memsize_entry, GTK_TYPE_BOX)
|
|
|
|
#define parent_class gimp_memsize_entry_parent_class
|
|
|
|
static guint gimp_memsize_entry_signals[LAST_SIGNAL] = { 0 };
|
|
|
|
|
|
static void
|
|
gimp_memsize_entry_class_init (GimpMemsizeEntryClass *klass)
|
|
{
|
|
GObjectClass *object_class = G_OBJECT_CLASS (klass);
|
|
|
|
object_class->finalize = gimp_memsize_entry_finalize;
|
|
|
|
klass->value_changed = NULL;
|
|
|
|
gimp_memsize_entry_signals[VALUE_CHANGED] =
|
|
g_signal_new ("value-changed",
|
|
G_TYPE_FROM_CLASS (klass),
|
|
G_SIGNAL_RUN_FIRST,
|
|
G_STRUCT_OFFSET (GimpMemsizeEntryClass, value_changed),
|
|
NULL, NULL,
|
|
g_cclosure_marshal_VOID__VOID,
|
|
G_TYPE_NONE, 0);
|
|
}
|
|
|
|
static void
|
|
gimp_memsize_entry_init (GimpMemsizeEntry *entry)
|
|
{
|
|
gtk_orientable_set_orientation (GTK_ORIENTABLE (entry),
|
|
GTK_ORIENTATION_HORIZONTAL);
|
|
|
|
gtk_box_set_spacing (GTK_BOX (entry), 4);
|
|
|
|
entry->value = 0;
|
|
entry->lower = 0;
|
|
entry->upper = 0;
|
|
entry->shift = 0;
|
|
entry->adjustment = NULL;
|
|
entry->menu = NULL;
|
|
}
|
|
|
|
static void
|
|
gimp_memsize_entry_finalize (GObject *object)
|
|
{
|
|
GimpMemsizeEntry *entry = (GimpMemsizeEntry *) object;
|
|
|
|
g_clear_object (&entry->adjustment);
|
|
|
|
G_OBJECT_CLASS (parent_class)->finalize (object);
|
|
}
|
|
|
|
static void
|
|
gimp_memsize_entry_adj_callback (GtkAdjustment *adj,
|
|
GimpMemsizeEntry *entry)
|
|
{
|
|
guint64 size = gtk_adjustment_get_value (adj);
|
|
|
|
if (gimp_memsize_entry_get_rounded_value (entry, entry->value) != size)
|
|
/* Do not allow losing accuracy if the converted/displayed value
|
|
* stays the same.
|
|
*/
|
|
entry->value = size << entry->shift;
|
|
|
|
g_signal_emit (entry, gimp_memsize_entry_signals[VALUE_CHANGED], 0);
|
|
}
|
|
|
|
static void
|
|
gimp_memsize_entry_unit_callback (GtkWidget *widget,
|
|
GimpMemsizeEntry *entry)
|
|
{
|
|
guint shift;
|
|
|
|
gimp_int_combo_box_get_active (GIMP_INT_COMBO_BOX (widget), (gint *) &shift);
|
|
|
|
#if _MSC_VER < 1300
|
|
# define CAST (gint64)
|
|
#else
|
|
# define CAST
|
|
#endif
|
|
|
|
if (shift != entry->shift)
|
|
{
|
|
entry->shift = shift;
|
|
|
|
gtk_adjustment_configure (entry->adjustment,
|
|
gimp_memsize_entry_get_rounded_value (entry, entry->value),
|
|
CAST entry->lower >> shift,
|
|
CAST entry->upper >> shift,
|
|
gtk_adjustment_get_step_increment (entry->adjustment),
|
|
gtk_adjustment_get_page_increment (entry->adjustment),
|
|
gtk_adjustment_get_page_size (entry->adjustment));
|
|
}
|
|
|
|
#undef CAST
|
|
}
|
|
|
|
/**
|
|
* gimp_memsize_entry_get_rounded_value:
|
|
* @entry: #GimpMemsizeEntry whose set unit is used.
|
|
* @value: value to convert to @entry unit, and rounded.
|
|
*
|
|
* Returns: the proper integer value to be displayed for current unit.
|
|
* This value has been appropriately rounded to the nearest
|
|
* integer, away from zero.
|
|
*/
|
|
static guint64
|
|
gimp_memsize_entry_get_rounded_value (GimpMemsizeEntry *entry,
|
|
guint64 value)
|
|
{
|
|
guint64 converted;
|
|
|
|
#if _MSC_VER < 1300
|
|
# define CAST (gint64)
|
|
#else
|
|
# define CAST
|
|
#endif
|
|
|
|
converted = (CAST value >> entry->shift) +
|
|
((CAST entry->value >> (entry->shift - 1)) & 1);
|
|
|
|
#undef CAST
|
|
|
|
return converted;
|
|
}
|
|
|
|
|
|
/**
|
|
* gimp_memsize_entry_new:
|
|
* @value: the initial value (in Bytes)
|
|
* @lower: the lower limit for the value (in Bytes)
|
|
* @upper: the upper limit for the value (in Bytes)
|
|
*
|
|
* Creates a new #GimpMemsizeEntry which is a #GtkHBox with a #GtkSpinButton
|
|
* and a #GtkOptionMenu all setup to allow the user to enter memory sizes.
|
|
*
|
|
* Returns: Pointer to the new #GimpMemsizeEntry.
|
|
**/
|
|
GtkWidget *
|
|
gimp_memsize_entry_new (guint64 value,
|
|
guint64 lower,
|
|
guint64 upper)
|
|
{
|
|
GimpMemsizeEntry *entry;
|
|
GtkAdjustment *adj;
|
|
guint shift;
|
|
|
|
#if _MSC_VER < 1300
|
|
# define CAST (gint64)
|
|
#else
|
|
# define CAST
|
|
#endif
|
|
|
|
g_return_val_if_fail (value >= lower && value <= upper, NULL);
|
|
|
|
entry = g_object_new (GIMP_TYPE_MEMSIZE_ENTRY, NULL);
|
|
|
|
for (shift = 30; shift > 10; shift -= 10)
|
|
{
|
|
if (value > (G_GUINT64_CONSTANT (1) << shift) &&
|
|
value % (G_GUINT64_CONSTANT (1) << shift) == 0)
|
|
break;
|
|
}
|
|
|
|
entry->value = value;
|
|
entry->lower = lower;
|
|
entry->upper = upper;
|
|
entry->shift = shift;
|
|
|
|
adj = (GtkAdjustment *) gtk_adjustment_new (gimp_memsize_entry_get_rounded_value (entry,
|
|
entry->value),
|
|
CAST (lower >> shift),
|
|
CAST (upper >> shift),
|
|
1, 8, 0);
|
|
|
|
entry->spinbutton = gimp_spin_button_new (adj, 1.0, 0);
|
|
gtk_spin_button_set_numeric (GTK_SPIN_BUTTON (entry->spinbutton), TRUE);
|
|
|
|
#undef CAST
|
|
|
|
entry->adjustment = GTK_ADJUSTMENT (adj);
|
|
g_object_ref_sink (entry->adjustment);
|
|
|
|
gtk_entry_set_width_chars (GTK_ENTRY (entry->spinbutton), 7);
|
|
gtk_box_pack_start (GTK_BOX (entry), entry->spinbutton, FALSE, FALSE, 0);
|
|
gtk_widget_show (entry->spinbutton);
|
|
|
|
g_signal_connect (entry->adjustment, "value-changed",
|
|
G_CALLBACK (gimp_memsize_entry_adj_callback),
|
|
entry);
|
|
|
|
entry->menu = gimp_int_combo_box_new (_("Kibibyte"), 10,
|
|
_("Mebibyte"), 20,
|
|
_("Gibibyte"), 30,
|
|
NULL);
|
|
|
|
gimp_int_combo_box_set_active (GIMP_INT_COMBO_BOX (entry->menu), shift);
|
|
|
|
g_signal_connect (entry->menu, "changed",
|
|
G_CALLBACK (gimp_memsize_entry_unit_callback),
|
|
entry);
|
|
|
|
gtk_box_pack_start (GTK_BOX (entry), entry->menu, FALSE, FALSE, 0);
|
|
gtk_widget_show (entry->menu);
|
|
|
|
return GTK_WIDGET (entry);
|
|
}
|
|
|
|
/**
|
|
* gimp_memsize_entry_set_value:
|
|
* @entry: a #GimpMemsizeEntry
|
|
* @value: the new value (in Bytes)
|
|
*
|
|
* Sets the @entry's value. Please note that the #GimpMemsizeEntry rounds
|
|
* the value to full Kilobytes.
|
|
**/
|
|
void
|
|
gimp_memsize_entry_set_value (GimpMemsizeEntry *entry,
|
|
guint64 value)
|
|
{
|
|
guint shift;
|
|
|
|
g_return_if_fail (GIMP_IS_MEMSIZE_ENTRY (entry));
|
|
g_return_if_fail (value >= entry->lower && value <= entry->upper);
|
|
|
|
for (shift = 30; shift > 10; shift -= 10)
|
|
{
|
|
if (value > (G_GUINT64_CONSTANT (1) << shift) &&
|
|
value % (G_GUINT64_CONSTANT (1) << shift) == 0)
|
|
break;
|
|
}
|
|
|
|
if (shift != entry->shift)
|
|
gimp_int_combo_box_set_active (GIMP_INT_COMBO_BOX (entry->menu), shift);
|
|
|
|
gtk_adjustment_set_value (entry->adjustment,
|
|
(gdouble) gimp_memsize_entry_get_rounded_value (entry, value));
|
|
|
|
#undef CASE
|
|
}
|
|
|
|
/**
|
|
* gimp_memsize_entry_get_value:
|
|
* @entry: a #GimpMemsizeEntry
|
|
*
|
|
* Retrieves the current value from a #GimpMemsizeEntry.
|
|
*
|
|
* Returns: the current value of @entry (in Bytes).
|
|
**/
|
|
guint64
|
|
gimp_memsize_entry_get_value (GimpMemsizeEntry *entry)
|
|
{
|
|
g_return_val_if_fail (GIMP_IS_MEMSIZE_ENTRY (entry), 0);
|
|
|
|
return entry->value;
|
|
}
|