Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | reuse the same cancellable across multiple non-concurrent dbus calls | Charles Kerr | 2012-05-26 | 1 | -18/+18 |
| | |||||
* | avoid a cyclical refcount dependency between IndicatorPower and its DBusListener | Charles Kerr | 2012-05-26 | 1 | -3/+17 |
| | |||||
* | extract-method to simplify testing the accessible text | Charles Kerr | 2012-05-26 | 1 | -47/+45 |
| | |||||
* | separate the dbus org.gnome.SettingsDaemon.Power logic into a separate class | Charles Kerr | 2012-05-26 | 4 | -147/+366 |
| | |||||
* | add coverage for various charging/discharging/charged states | Charles Kerr | 2012-05-26 | 1 | -3/+101 |
| | |||||
* | first draft of adding tests for a discharging battery | Charles Kerr | 2012-05-26 | 1 | -0/+35 |
| | |||||
* | make indicator_power_set_devices() safe for passing in the same devices more ↵ | Charles Kerr | 2012-05-25 | 1 | -12/+14 |
| | | | | than once | ||||
* | fix negated logic in menu_add_device() introduced in r160 | Charles Kerr | 2012-05-25 | 1 | -1/+1 |
| | |||||
* | indicator_power_set_devices() should fail gracefully when no devices are ↵ | Charles Kerr | 2012-05-25 | 1 | -2/+5 |
| | | | | available | ||||
* | make Jenkins happy | Charles Kerr | 2012-05-25 | 1 | -2/+2 |
| | |||||
* | modify IndicatorPower to use IndicatorPowerDevices internally | Charles Kerr | 2012-05-25 | 3 | -188/+193 |
| | |||||
* | put IndicatorPower's fields back inside a priv struct | Charles Kerr | 2012-05-25 | 2 | -74/+99 |
| | |||||
* | ah, c++ is too smart for the last commit's implicit conversion. | Charles Kerr | 2012-05-24 | 1 | -1/+1 |
| | |||||
* | test passing bad non-NULL pointers to Device's accessors | Charles Kerr | 2012-05-24 | 1 | -6/+16 |
| | |||||
* | add unit tests confirming that the device accessor functions won't crash ↵ | Charles Kerr | 2012-05-24 | 1 | -0/+14 |
| | | | | when you pass NULL for the device | ||||
* | remove the 'default' clause from get_property() and set_property(). | Charles Kerr | 2012-05-24 | 1 | -8/+0 |
| | | | | | 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* | ||||
* | remove some unnecessary type compatibilty tests... g_object_get_property() ↵ | Charles Kerr | 2012-05-24 | 1 | -6/+0 |
| | | | | does these tests for us | ||||
* | remove the tests that attempted to exercise the code removed in r151 | Charles Kerr | 2012-05-24 | 1 | -33/+2 |
| | |||||
* | don't include the unit tests' code in coverage metrics | Charles Kerr | 2012-05-24 | 1 | -3/+2 |
| | |||||
* | remove some unnecessary type compatibilty tests... g_object_set_property() ↵ | Charles Kerr | 2012-05-24 | 1 | -6/+0 |
| | | | | does these tests for us | ||||
* | add unit tests for getting/setting device properties | Charles Kerr | 2012-05-24 | 1 | -0/+90 |
| | |||||
* | first draft of getting GSettings working in the unit tests before the schema ↵ | Charles Kerr | 2012-05-24 | 2 | -4/+33 |
| | | | | is installed. | ||||
* | remove the g_clear_pointer() calls s.t. things will build and run on ↵ | Charles Kerr | 2012-05-24 | 1 | -2/+6 |
| | | | | alesage's Jenkins setup running Precise | ||||
* | remove the g_clear_pointer() calls s.t. things will build and run on ↵ | Charles Kerr | 2012-05-24 | 2 | -5/+5 |
| | | | | alesage's Jenkins setup running Precise | ||||
* | add test-indicator.cc | Charles Kerr | 2012-05-24 | 1 | -0/+52 |
| | |||||
* | add a 'hello world' test for instantiating IndicatorPower | Charles Kerr | 2012-05-24 | 3 | -52/+94 |
| | |||||
* | add private container struct IndicatorPowerDevice | Charles Kerr | 2012-05-24 | 9 | -3/+706 |
| | |||||
* | replace 'Power Settings...' with 'Power Settings…' | Charles Kerr | 2012-05-23 | 1 | -1/+1 |
| | |||||
* | move libpower's automake rules into src/ | Charles Kerr | 2012-05-23 | 3 | -27/+30 |
| | |||||
* | fix a handful of string and GVariant leaks in indicator-power. | Charles Kerr | 2012-04-26 | 1 | -39/+35 |
|\ | |||||
| * | tweak: in put_primary_device(), peek at the variant's strings instead of ↵ | Charles Kerr | 2012-04-15 | 1 | -5/+3 |
| | | | | | | | | dup'ing them | ||||
| * | sync with lp:indicator-power | Charles Kerr | 2012-04-15 | 5 | -49/+76 |
| |\ | |/ |/| | |||||
* | | 2.02.0 | Charles Kerr | 2012-04-11 | 1 | -1/+1 |
| | | |||||
* | | merge lp:~allanlesage/indicator-power/TDD to improve our gcov fules in autotools | Charles Kerr | 2012-04-10 | 4 | -45/+65 |
|\ \ | |||||
| * | | Pedantic name change for gcovr xml results. | Allan LeSage | 2012-03-27 | 1 | -2/+2 |
| | | | |||||
| * | | Added gcov coverage tooling. | Allan LeSage | 2012-03-27 | 4 | -45/+65 |
| | | | |||||
* | | | use atk_object_set_name(foo,bar)" instead of ↵ | Charles Kerr | 2012-04-09 | 1 | -1/+1 |
| | | | | | | | | | | | | g_object_set(foo,"accessible-name",bar,NULL)" | ||||
* | | | leak fix 3 of 3: don't leak the GIcons returned by get_device_icon() | Charles Kerr | 2012-04-09 | 1 | -0/+2 |
| | | | |||||
* | | | leak fix 2 of 3: fix leaky fallback GIcon in get_device_icon() | Charles Kerr | 2012-04-09 | 1 | -3/+4 |
| | | | |||||
* | | | leak fix 1 of 3: don't leak object_path and device_icon if kind == ↵ | Charles Kerr | 2012-04-09 | 1 | -6/+4 |
| | | | | | | | | | | | | UP_DEVICE_KIND_LINE_POWER | ||||
* | | | if possible, set the device menuitems' accessible names. | Charles Kerr | 2012-04-09 | 1 | -0/+4 |
| | | | |||||
* | | | 1.931.93 | Charles Kerr | 2012-03-21 | 1 | -1/+1 |
| | | | |||||
| | * | fix leaked strings in menu_add_device when (kind == UP_DEVICE_KIND_LINE_POWER) | Charles Kerr | 2012-04-15 | 1 | -6/+4 |
| | | | |||||
| | * | Fix memory leaks in get_primary_device(). | Charles Kerr | 2012-03-17 | 1 | -32/+28 |
| | | | | | | | | | | | | | | | | | | | | | 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". | ||||
| | * | Fix variant leak in count_batteries() -- the returned value of ↵ | Charles Kerr | 2012-03-17 | 1 | -0/+2 |
| | | | | | | | | | | | | g_variant_get_child_value() needs to be freed with g_variant_unref() when we're done with it. | ||||
| | * | fix variant leaks in menu_add_devices() | Charles Kerr | 2012-03-17 | 1 | -2/+2 |
| |/ |/| | |||||
* | | merge lp:~kelemeng/indicator-power/bug957542 to fix extracting translatable ↵ | Charles Kerr | 2012-03-17 | 1 | -1/+1 |
|\ \ | | | | | | | | | | strings from ipower (Bug #957542) | ||||
| * | | Add g_dngettext to the list of known keywords. LP: #957542 | Gabor Kelemen | 2012-03-17 | 1 | -1/+1 |
|/ / | |||||
* / | 1.911.92 | Charles Kerr | 2012-03-14 | 1 | -1/+1 |
|/ | |||||
* | 1.911.91 | Ted Gould | 2012-02-23 | 1 | -1/+1 |
| |