New issue
Advanced search Search tips

Issue 845696 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2018
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: ----
Type: ----



Sign in to add a comment

Add D-Bus API to append / pop array of integers

Project Member Reported by abhishekbh@google.com, May 22 2018

Issue description

D-Bus lacks API to AppendArrayOfInts, PopArrayOfInts.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 4 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b460dff90b1795d4e9ba1bfcd06cc9895d253788

commit b460dff90b1795d4e9ba1bfcd06cc9895d253788
Author: Abhishek Bhardwaj <abhishekbh@chromium.org>
Date: Mon Jun 04 19:27:37 2018

dbus: Add API for array of ints

This change adds API to append and pop an array of signed / unsigned
integers.

BUG= chromium:845696 
TEST=Add and run unit tests.

Change-Id: Ib1cabdc8b442b549e27701a804c16114f4ae9582
Reviewed-on: https://chromium-review.googlesource.com/1069714
Commit-Queue: Abhishek Bhardwaj <abhishekbh@chromium.org>
Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#564184}
[modify] https://crrev.com/b460dff90b1795d4e9ba1bfcd06cc9895d253788/dbus/message.cc
[modify] https://crrev.com/b460dff90b1795d4e9ba1bfcd06cc9895d253788/dbus/message.h
[modify] https://crrev.com/b460dff90b1795d4e9ba1bfcd06cc9895d253788/dbus/message_unittest.cc

Status: Fixed (was: Assigned)

Sign in to add a comment