gtg-contributors team mailing list archive

[Thibault Fevry <635152@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

[Trunk.] [Don't care about debug]

 First, if you're running trunk, you may have seen some syntax warning when launching GTG, this fixes two of them.
 Second, we had an incorrect test in rtm_backend : http://bazaar.launchpad.net/~gtg-user/gtg/fix_asserts/revision/889
 As you can see there, the assert wasn't checking that there was only one element, it was checking that they were at least one element. (See the comment.)

 Rev 889 and 888 are the important ones.

** Affects: gtg
     Importance: Low
     Assignee: Thibault Fevry (thibaultfevry)
         Status: In Progress


** Tags: patch toreview

** Changed in: gtg
     Assignee: (unassigned) => Thibault Fevry (thibaultfevry)

** Changed in: gtg
   Importance: Undecided => Low

** Changed in: gtg
       Status: New => In Progress

-- 
Incorrect assert test.
https://bugs.launchpad.net/bugs/635152
You received this bug notification because you are a member of Gtg
contributors, which is subscribed to Getting Things GNOME!.

Status in Getting Things GNOME!: In Progress

Bug description:
[Trunk.] [Don't care about debug]

 First, if you're running trunk, you may have seen some syntax warning when launching GTG, this fixes two of them.
 Second, we had an incorrect test in rtm_backend : http://bazaar.launchpad.net/~gtg-user/gtg/fix_asserts/revision/889
 As you can see there, the assert wasn't checking that there was only one element, it was checking that they were at least one element. (See the comment.)

 Rev 889 and 888 are the important ones.