[PATCH 00/16] alsa-gobject: timer: enhancement for error reporting

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



alsa-project/alsa-gobject pull request #49 was opened from takaswie:

The patchset is a part of work about #47.

Current implementation uses library-wide error domain to report error. this
is partly against rule of GError usage. Furthermore, the error delivers
information just about errno and hard to know the cause of error.

This patchset enhances error reporting. The library-wide error domain is
obsoleted. A new error domain is added just for ALSATimer.UserInstance.
The error domain delivers information about the cause of error.

```
Takashi Sakamoto (16):
  timer: user_instance: fix function comment
  timer: skip check of return value from g_malloc()
  timer: check whether method argument for GError is available
  timer: add checks for method arguments
  timer: instance-params: add checks for method arguments
  timer: user_instance: just return when character device is not opened
  timer: add GLib enumeration to report type of error for ALSATimer.UserInstance
  timer: user_instance: add GQuark to report error for ALSATimer.UserInstance
  timer: user_instance: report error due to ioctl failure
  timer: user_instance: report error due to open system call
  timer: user_instance: report error for timer instance not found
  timer: user_instance: report error for unattached timer instance
  timer: user_instance: report error for timer already attached
  timer: query: use GFileError to report error
  timer: query: code refactoring to unify open function
  timer: obsolete library-wide GQuark for error reporting

 src/timer/alsatimer-enum-types.h |  18 ++++
 src/timer/alsatimer.map          |   5 +
 src/timer/instance-params.c      |  20 ++--
 src/timer/instance-status.c      |   3 +-
 src/timer/privates.h             |   6 --
 src/timer/query.c                | 127 ++++++++++++------------
 src/timer/user-instance.c        | 165 +++++++++++++++++++++++--------
 src/timer/user-instance.h        |   4 +
 tests/alsatimer-enums            |   8 ++
 9 files changed, 231 insertions(+), 125 deletions(-)
```

Request URL   : https://github.com/alsa-project/alsa-gobject/pull/49
Patch URL     : https://github.com/alsa-project/alsa-gobject/pull/49.patch
Repository URL: https://github.com/alsa-project/alsa-gobject



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux