incubator-callback-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Drew Walters <>
Subject Re: [Android] Accelerometer plugin rewrite
Date Tue, 15 May 2012 15:46:02 GMT
Do you need help with the BlackBerry smartphone implementation or is
that under control?

On Tue, May 15, 2012 at 10:16 AM, Filip Maj <> wrote:
> Yep, I've got issues filed for each platform (most assigned to me). They
> will get done this week before we tag the RC.
> If JIRA was up I could link you to them :/
> On 5/15/12 7:44 AM, "Drew Walters" <> wrote:
>>Fil, wouldn't the other native implementations need updated as well
>>before merging in the changes that are required for cordova-js?
>>On Mon, May 14, 2012 at 5:26 PM, Filip Maj <> wrote:
>>> Hey all,
>>> Part of the 1.8 work was to change the accelerometer plugin to line it
>>> more with the way the Geolocation plugin was rewritten. Essentially:
>>> - store callback ids in the plugin for incoming requests
>>> - privatize start() and stop() methods
>>> - start and stop the accel listener based on how many callbacks are
>>> waiting for accel data
>>> The Accel plugin now should implement only 3 actions: getAcceleration,
>>> addWatch, clearWatch. The plugin should handle calling start() and
>>> privately on its own. This also eliminates the awkward "getTimeout" and
>>> "setTimeout" actions, and now we are ensured that the accelerometer
>>> stops as soon as possible (namely, when no callbacks in JS are waiting
>>> acceleration values), instead of waiting for the timeout period to
>>> I added a few extra things to the android plugin rewrite as well, mainly
>>> checking for accuracy level before returning values to the web view.
>>> If any Android maintainers want to take a look at the rewrite, please
>>> a look at the 463 branch on the Apache git repo [1]. Would love feedback
>>> if you have the time to look it over.
>>> I will be merging it later this week unless someone has any issues with
>>> Thanks,
>>> Fil
>>> [1]
>>> ortlog;h=refs/heads/463

View raw message