Bug 62086

Summary: TimeRange.intersect name is cryptic
Product: Zeitgeist Reporter: Manish Sinha <manishsinha>
Component: libzeitgeistAssignee: zeitgeist-bugs <zeitgeist-bugs>
Status: RESOLVED MOVED QA Contact: zeitgeist-bugs <zeitgeist-bugs>
Severity: normal    
Priority: medium    
Version: 0.9.x   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:

Description Manish Sinha 2013-03-10 05:37:53 UTC
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.
Comment 1 GitLab Migration User 2018-05-31 09:12:02 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/zeitgeist/zeitgeist/issues/6.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.