lp:~dandrader/grail/slice_ref_loop

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:grail at revision 214

Chase Douglas (community): Approve on 2012-03-26

Diff: 362 lines (+65/-64)

3 files modified

src/v3/gesture.cpp (+5/-5)
src/v3/slice.cpp (+50/-48)
src/v3/slice.h (+10/-11)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #965260: Needless memory growth from undeleted Gesture instances

Medium

Fix Released

Link a bug report

Related blueprints

212. By Daniel d'Andrada on 2012-03-26

Break reference loop between Slice and Gesture classes

Break it by removing the Gesture pointer variable from the Slice class.
Slice doesn't need to keep a pointer to the Gesture that originated it.

211. By Daniel d'Andrada on 2012-03-23

      AtomicRecognizer::FindGestureToAccept - handle negative time deltas

      It's possible that a touch start time is bigger than the event time.

210. By Daniel d'Andrada on 2012-03-23

Merge fixes to Recognizer::kCompositionTime

 - Turn it into a real constant (add const keyword)

 - Make it follow latest version of Unity's C++ Style Guide:
http://unity.ubuntu.com/wp-content/uploads/2012/03/cppguide.html#Constant_Names

209. By Chase Douglas on 2012-03-23

Merge fix for linking tests against appropriate libtool files

208. By Jussi Pakkanen on 2012-03-23

Merged out-of-source build fix.

207. By Jussi Pakkanen on 2012-03-23

Merged uninitialization fix.

206. By Jussi Pakkanen on 2012-03-23

Merged virtual destructor fix.

205. By Daniel d'Andrada on 2012-03-22

Remove threaded recording playback from parallel-atomic-gestures test as well

This has been done already for the other tests by revision 200.4.4

204. By Chase Douglas on 2012-03-22

Release version 3.0.3

203. By Chase Douglas on 2012-03-21

Merge fix for allowing multiple atomic gestures at the same time