aboutsummaryrefslogtreecommitdiff
path: root/ChangeLog
diff options
context:
space:
mode:
Diffstat (limited to 'ChangeLog')
-rw-r--r--ChangeLog366
1 files changed, 366 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog
index bcfaff8..f70f1a7 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,371 @@
# Generated by Makefile. Do not edit.
+2012-07-11 Charles Kerr <charles.kerr@canonical.com>
+
+ bump version to 12.10.0
+
+2012-07-11 Charles Kerr <charles.kerr@canonical.com>
+
+ merge lp:~charlesk/indicator-power/coverage
+
+2012-07-03 Charles Kerr <charles.kerr@canonical.com>
+
+ disable test-dbus-listener for now
+
+2012-06-07 Charles Kerr <charles.kerr@canonical.com>
+
+ remove some dead code.
+
+2012-06-07 Charles Kerr <charles.kerr@canonical.com>
+
+ improve the variant sanity tests in indicator_power_device_new_from_variant()
+
+2012-06-06 Charles Kerr <charles.kerr@canonical.com>
+
+ reimplement indicator_power_device_get_icon_names() since our CA is incompatible with reusing code from GSD.
+
+2012-06-06 Charles Kerr <charles.kerr@canonical.com>
+
+ remove superfluous #include
+
+2012-06-06 Charles Kerr <charles.kerr@canonical.com>
+
+ simplify the Device properties' name strings
+
+2012-06-06 Charles Kerr <charles.kerr@canonical.com>
+
+ For GTK-Doc, reference functions with function_name() instead of #function_name
+
+2012-06-06 Charles Kerr <charles.kerr@canonical.com>
+
+ In indicator_power_device_new_from_variant(), check whether the input variant has the correct type before using it.
+
+2012-06-06 Charles Kerr <charles.kerr@canonical.com>
+
+ add G_OBJECT_WARN_INVALID_PROPERTY_ID for the 'default' switch case in Device's get/set property methods
+
+2012-06-06 Charles Kerr <charles.kerr@canonical.com>
+
+ IndicatorObjectDevice's finalize() function needs to chain up to the parent class.
+
+2012-06-06 Charles Kerr <charles.kerr@canonical.com>
+
+ in IndicatorPowerDevice's class init function, use g_object_class_install_properties() instead of installing each property separately.
+
+2012-06-06 Charles Kerr <charles.kerr@canonical.com>
+
+ remove prototype for indicator_power_dbus_listener_new() since it's not needed/used
+
+2012-06-06 Charles Kerr <charles.kerr@canonical.com>
+
+ simplify the devices-enumerated signal's name
+
+2012-06-06 Charles Kerr <charles.kerr@canonical.com>
+
+ if self->cancellable is non-NULL in dispose(), pass it to g_cancellable_cancel() before clearing the listener's reference
+
+2012-06-06 Charles Kerr <charles.kerr@canonical.com>
+
+ remove trailing whitespace
+
+2012-06-06 Charles Kerr <charles.kerr@canonical.com>
+
+ add g_bus_unwatch_name() to watcher's dispose() method
+
+2012-06-06 Charles Kerr <charles.kerr@canonical.com>
+
+ add a GTK-Doc signal comment block for indictor-power-dbus-listener's enumerated signal
+
+2012-06-06 Charles Kerr <charles.kerr@canonical.com>
+
+ move the header files from noinst_HEADERS to libpower_la_SOURCES
+
+2012-06-01 Charles Kerr <charles.kerr@canonical.com>
+
+ In indicator-power.c's put_primary_device(), remove dead code.
+
+2012-06-01 Charles Kerr <charles.kerr@canonical.com>
+
+ copyediting: fix tab damage in dbus-listener.[ch]
+
+2012-06-01 Charles Kerr <charles.kerr@canonical.com>
+
+ copyediting: fix tab damage in device.[ch]
+
+2012-06-01 Charles Kerr <charles.kerr@canonical.com>
+
+ use C++-style comments in test-device.cc
+
+2012-06-01 Charles Kerr <charles.kerr@canonical.com>
+
+ In indicator_power_device_get_time_details(), add a test for non-devices being passed in.
+
+2012-06-01 Charles Kerr <charles.kerr@canonical.com>
+
+ Fix edge case in indicator_power_device_get_icon_names() that returned a bad value if the caller passed in a NULL pointer as a device. Added regression test.
+
+2012-06-01 Charles Kerr <charles.kerr@canonical.com>
+
+ In indicator_power_device_get_time_details(), use g_strdup(foo) instead of g_strdup_printf("%s",foo)
+
+2012-06-01 Charles Kerr <charles.kerr@canonical.com>
+
+ Add coverage test for AC Adapters in indicator_power_device_get_time_details()
+
+2012-06-01 Charles Kerr <charles.kerr@canonical.com>
+
+ capitalize the second word in 'AC Adapter'
+
+2012-06-01 Charles Kerr <charles.kerr@canonical.com>
+
+ in indicator_power_device_get_time_details(), don't list 'not present' for AC Adapters that have no % and no time estimate
+
+2012-06-01 Charles Kerr <charles.kerr@canonical.com>
+
+ Add unit tests for indicator_power_device_get_time_details()
+
+2012-06-01 Charles Kerr <charles.kerr@canonical.com>
+
+ In indicator_power_device_get_time_details(), remove an unlikely branch that could result in time/detail strings not being set.
+
+2012-06-01 Charles Kerr <charles.kerr@canonical.com>
+
+ Add gcda/gcno to CLEANFILES s.t. 'make clean' will cover them
+
+2012-05-31 Charles Kerr <charles.kerr@canonical.com>
+
+ in indicator_power_device_get_time_details(), limit the scope of variables short_timestring and detailed_timestring
+
+2012-05-31 Charles Kerr <charles.kerr@canonical.com>
+
+ Move private indicator-power function build_device_time_details() to device.c to public function indicator_power_device_get_time_details() so that we can unit test the user-visible strings.
+
+2012-05-31 Charles Kerr <charles.kerr@canonical.com>
+
+ remove IndicatorPowerDevice's now-unused device icon string.
+
+2012-05-31 Charles Kerr <charles.kerr@canonical.com>
+
+ Add indicator_power_device_get_icon_names().
+
+ The main goal of this change is to make it possible to test the
+ device's icon. A secondary goal is to clarify in the code how
+ indicator-power's icons differ from the ones recommended by GSD.
+
+2012-05-31 Charles Kerr <charles.kerr@canonical.com>
+
+ remove some g_message() calls that shouldn't've been committed.
+
+2012-05-29 Charles Kerr <charles.kerr@canonical.com>
+
+ add a test to make sure that the listener responds to a PropertiesChanged signal
+
+2012-05-29 Charles Kerr <charles.kerr@canonical.com>
+
+ t^Cak to indicator_set_power_differences()
+
+2012-05-29 Charles Kerr <charles.kerr@canonical.com>
+
+ use g_clear_pointer() where useful
+
+2012-05-29 Charles Kerr <charles.kerr@canonical.com>
+
+ add tests for gsd GetDevices returning (a) no devices and (b) an error
+
+2012-05-29 Charles Kerr <charles.kerr@canonical.com>
+
+ exclude G_DEFINE_TYPE from coverage testing in dbus-listener.c
+
+2012-05-28 Charles Kerr <charles.kerr@canonical.com>
+
+ add more tests to improve coverage: different device types, no primary device
+
+2012-05-28 Charles Kerr <charles.kerr@canonical.com>
+
+ add a mock GSD.Power to handle the GetPower requests
+
+2012-05-27 Charles Kerr <charles.kerr@canonical.com>
+
+ use signals to decouple i-power and dbus-listener
+
+2012-05-27 Charles Kerr <charles.kerr@canonical.com>
+
+ Add skeleton test for IndicatorPowerDbusListener
+
+2012-05-26 Charles Kerr <charles.kerr@canonical.com>
+
+ add coverage for charging with >1 minute but <1 hour left
+
+2012-05-26 Charles Kerr <charles.kerr@canonical.com>
+
+ add coverage for DBusListener get_property()
+
+2012-05-26 Charles Kerr <charles.kerr@canonical.com>
+
+ try adding LCOV_EXCL_{START,STOP} for unreachable conditions (glib looking for subclasses of IndicatorPowerDevice; unreachables in G_DEFINE_TYPE)
+
+2012-05-26 Charles Kerr <charles.kerr@canonical.com>
+
+ call g_object_run_dispose() in a standalone test to get coverage on the NULL / non-NULL branches of dispose()'s g_clear_pointer() calls
+
+2012-05-26 Charles Kerr <charles.kerr@canonical.com>
+
+ nope, LCOV_EXCL_LINE doesn't work on macros
+
+2012-05-26 Charles Kerr <charles.kerr@canonical.com>
+
+ try adding LCOV_EXCL_LINE for unreachable conditions (glib looking for subclasses of IndicatorPowerDevice; unreachables in G_DEFINE_TYPE)
+
+2012-05-26 Charles Kerr <charles.kerr@canonical.com>
+
+ reuse the same cancellable across multiple non-concurrent dbus calls
+
+2012-05-26 Charles Kerr <charles.kerr@canonical.com>
+
+ avoid a cyclical refcount dependency between IndicatorPower and its DBusListener
+
+2012-05-26 Charles Kerr <charles.kerr@canonical.com>
+
+ extract-method to simplify testing the accessible text
+
+2012-05-26 Charles Kerr <charles.kerr@canonical.com>
+
+ separate the dbus org.gnome.SettingsDaemon.Power logic into a separate class
+
+2012-05-26 Charles Kerr <charles.kerr@canonical.com>
+
+ add coverage for various charging/discharging/charged states
+
+2012-05-26 Charles Kerr <charles.kerr@canonical.com>
+
+ first draft of adding tests for a discharging battery
+
+2012-05-25 Charles Kerr <charles.kerr@canonical.com>
+
+ make indicator_power_set_devices() safe for passing in the same devices more than once
+
+2012-05-25 Charles Kerr <charles.kerr@canonical.com>
+
+ fix negated logic in menu_add_device() introduced in r160
+
+2012-05-25 Charles Kerr <charles.kerr@canonical.com>
+
+ indicator_power_set_devices() should fail gracefully when no devices are available
+
+2012-05-25 Charles Kerr <charles.kerr@canonical.com>
+
+ make Jenkins happy
+
+2012-05-25 Charles Kerr <charles.kerr@canonical.com>
+
+ modify IndicatorPower to use IndicatorPowerDevices internally
+
+2012-05-25 Charles Kerr <charles.kerr@canonical.com>
+
+ put IndicatorPower's fields back inside a priv struct
+
+2012-05-24 Charles Kerr <charles.kerr@canonical.com>
+
+ ah, c++ is too smart for the last commit's implicit conversion.
+
+2012-05-24 Charles Kerr <charles.kerr@canonical.com>
+
+ test passing bad non-NULL pointers to Device's accessors
+
+2012-05-24 Charles Kerr <charles.kerr@canonical.com>
+
+ add unit tests confirming that the device accessor functions won't crash when you pass NULL for the device
+
+2012-05-24 Charles Kerr <charles.kerr@canonical.com>
+
+ remove the 'default' clause from get_property() and set_property().
+
+ Coverage testing isn't reaching them... glib is weeding out these invalid property keys before the device.c functions are ever reached. Nevertheless, leaving out a 'default' clause in a switch statement feels very unnatural to me. *twitch* *twitch*
+
+2012-05-24 Charles Kerr <charles.kerr@canonical.com>
+
+ remove some unnecessary type compatibilty tests... g_object_get_property() does these tests for us
+
+2012-05-24 Charles Kerr <charles.kerr@canonical.com>
+
+ remove the tests that attempted to exercise the code removed in r151
+
+2012-05-24 Charles Kerr <charles.kerr@canonical.com>
+
+ don't include the unit tests' code in coverage metrics
+
+2012-05-24 Charles Kerr <charles.kerr@canonical.com>
+
+ remove some unnecessary type compatibilty tests... g_object_set_property() does these tests for us
+
+2012-05-24 Charles Kerr <charles.kerr@canonical.com>
+
+ add unit tests for getting/setting device properties
+
+2012-05-24 Charles Kerr <charles.kerr@canonical.com>
+
+ first draft of getting GSettings working in the unit tests before the schema is installed.
+
+2012-05-24 Charles Kerr <charles.kerr@canonical.com>
+
+ remove the g_clear_pointer() calls s.t. things will build and run on alesage's Jenkins setup running Precise
+
+2012-05-24 Charles Kerr <charles.kerr@canonical.com>
+
+ remove the g_clear_pointer() calls s.t. things will build and run on alesage's Jenkins setup running Precise
+
+2012-05-24 Charles Kerr <charles.kerr@canonical.com>
+
+ add test-indicator.cc
+
+2012-05-24 Charles Kerr <charles.kerr@canonical.com>
+
+ add a 'hello world' test for instantiating IndicatorPower
+
+2012-05-24 Charles Kerr <charles.kerr@canonical.com>
+
+ add private container struct IndicatorPowerDevice
+
+2012-05-23 Charles Kerr <charles.kerr@canonical.com>
+
+ replace 'Power Settings...' with 'Power Settingsā€¦'
+
+2012-05-23 Charles Kerr <charles.kerr@canonical.com>
+
+ move libpower's automake rules into src/
+
+2012-04-26 Charles Kerr <charles.kerr@canonical.com>
+
+ fix a handful of string and GVariant leaks in indicator-power.
+
+2012-04-15 Charles Kerr <charles.kerr@canonical.com>
+
+ tweak: in put_primary_device(), peek at the variant's strings instead of dup'ing them
+
+2012-04-15 Charles Kerr <charles.kerr@canonical.com>
+
+ sync with lp:indicator-power
+
+2012-04-15 Charles Kerr <charles.kerr@canonical.com>
+
+ fix leaked strings in menu_add_device when (kind == UP_DEVICE_KIND_LINE_POWER)
+
+2012-03-17 Charles Kerr <charles.kerr@canonical.com>
+
+ Fix memory leaks in get_primary_device().
+
+ 1. All the calls to g_variant_get_child_value() were leaked. Fixed by changing the use to g_variant_get_child() and keeping index values of the interesting children instead of pointers to them.
+
+ 2. There were several paths where the local string "object_path" and "device_icon" were leaked. (For example, any non-battery entry given to us by upower). Fixed by making these const strings and peeking them from the variant with "&s" instead of "s".
+
+2012-03-17 Charles Kerr <charles.kerr@canonical.com>
+
+ Fix variant leak in count_batteries() -- the returned value of g_variant_get_child_value() needs to be freed with g_variant_unref() when we're done with it.
+
+2012-03-17 Charles Kerr <charles.kerr@canonical.com>
+
+ fix variant leaks in menu_add_devices()
+
2012-04-11 Charles Kerr <charles.kerr@canonical.com>
2.0