Skip to content

GitLab

Open
Created by Manish Sinha@manishMaintainer

TimeRange.intersect name is cryptic

Submitted by Manish Sinha @manish

Assigned to zeitgeist-bugs@lists.freedesktop.org

Link to original bug (#62086)

Description

libzeitgeist-2.0 has a new method in libzeitgeist-2.0

public TimeRange? intersect (TimeRange time_range) Check whether two time ranges are intersecting.

OPTION1: Either the doc should be

"Find the intersection of two TimeRange. If two TimeRange don't intersect, then null is returned"

OR OPTION2:

the method should be

public bool does_intersect (TimeRange time_range)

if you want to keep the documentation since the documentation for this method starts with "Check whether....", when you check, it should return a boolean value.

I guess the first one makes sense.

Version: 0.9.x

Edited by Bugzilla Migration User
To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information