[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-devel
Subject: Re: Review Request 114983: Misc changes to the file queue
From: "Simeon Bird" <bladud () gmail ! com>
Date: 2014-01-13 23:35:45
Message-ID: 20140113233545.30264.17730 () probe ! kde ! org
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/114983/
-----------------------------------------------------------
(Updated Jan. 13, 2014, 11:35 p.m.)
Status
------
This change has been marked as submitted.
Review request for Baloo and Vishesh Handa.
Repository: baloo
Description
-------
Misc. File queue changes - the top edit should perhaps be applied to IndexScheduler::slotCommitted
as well.
commit ea35771246890c46f63d30f765a0d19844bd3d63
Author: Simeon Bird <bladud@gmail.com>
Date: Sat Jan 11 14:57:39 2014 -0500
Instead of unconditionally resuming the file queue when the basic queue
is done, call slotScheduleIndexing.
Otherwise the file queue would start once the basic queue was done, even
on battery.
commit b6aeea48286e53206726af48dc65da026c3dd7a8
Author: Simeon Bird <bladud@gmail.com>
Date: Sat Jan 11 14:34:17 2014 -0500
Refactor a function in indexscheduler.cpp to reduce duplication
commit 4aa06719a98f740fb55c51232ac306409a860077
Author: Simeon Bird <bladud@gmail.com>
Date: Sat Jan 11 14:21:18 2014 -0500
Do not call fillQueue when the file queue is created, but when the basic
queue is done.
We only know what work needs to be done in the file queue once the basic
queue is complete, so call fillQueue at that time, not when the queue is
created.
Before we could be filling the file queue pointlessly with
tasks to be cleaned, or, if the file queue was empty when
the basic queue started, it might not start when it should have done.
commit 406a6a4ae4f5facbb33b7de433347d84f77182c5
Author: Simeon Bird <bladud@gmail.com>
Date: Sat Jan 11 12:44:52 2014 -0500
FileIndexingQueue doesn't need to listen to ConfigChanged to refill the
indexing queue, because that is handled in the IndexScheduler already
Diffs
-----
src/file/fileindexingqueue.h c112c97
src/file/fileindexingqueue.cpp 0f22caf
src/file/indexscheduler.h 015d23a
src/file/indexscheduler.cpp 79cd5bf
Diff: https://git.reviewboard.kde.org/r/114983/diff/
Testing
-------
Compiled, ran with it.
Thanks,
Simeon Bird
[Attachment #5 (text/html)]
<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 \
solid;"> <tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/114983/">https://git.reviewboard.kde.org/r/114983/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#e0e0e0" width="100%" cellpadding="8" style="border: 1px gray \
solid;"> <tr>
<td>
<h1 style="margin-right: 0.2em; padding: 0; font-size: 10pt;">This change has been \
marked as submitted.</h1> </td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" \
style="background-image: \
url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); \
background-position: left top; background-repeat: repeat-x; border: 1px black \
solid;"> <tr>
<td>
<div>Review request for Baloo and Vishesh Handa.</div>
<div>By Simeon Bird.</div>
<p style="color: grey;"><i>Updated Jan. 13, 2014, 11:35 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
baloo
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" \
style="border: 1px solid #b8b5a0"> <tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">Misc. File queue changes - the top edit should perhaps be applied to \
IndexScheduler::slotCommitted as well.
commit ea35771246890c46f63d30f765a0d19844bd3d63
Author: Simeon Bird <bladud@gmail.com>
Date: Sat Jan 11 14:57:39 2014 -0500
Instead of unconditionally resuming the file queue when the basic queue
is done, call slotScheduleIndexing.
Otherwise the file queue would start once the basic queue was done, even
on battery.
commit b6aeea48286e53206726af48dc65da026c3dd7a8
Author: Simeon Bird <bladud@gmail.com>
Date: Sat Jan 11 14:34:17 2014 -0500
Refactor a function in indexscheduler.cpp to reduce duplication
commit 4aa06719a98f740fb55c51232ac306409a860077
Author: Simeon Bird <bladud@gmail.com>
Date: Sat Jan 11 14:21:18 2014 -0500
Do not call fillQueue when the file queue is created, but when the basic
queue is done.
We only know what work needs to be done in the file queue once the basic
queue is complete, so call fillQueue at that time, not when the queue is
created.
Before we could be filling the file queue pointlessly with
tasks to be cleaned, or, if the file queue was empty when
the basic queue started, it might not start when it should have done.
commit 406a6a4ae4f5facbb33b7de433347d84f77182c5
Author: Simeon Bird <bladud@gmail.com>
Date: Sat Jan 11 12:44:52 2014 -0500
FileIndexingQueue doesn't need to listen to ConfigChanged to refill the
indexing queue, because that is handled in the IndexScheduler already
</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: \
1px solid #b8b5a0"> <tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">Compiled, ran with it.</pre> </td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/file/fileindexingqueue.h <span style="color: grey">(c112c97)</span></li>
<li>src/file/fileindexingqueue.cpp <span style="color: grey">(0f22caf)</span></li>
<li>src/file/indexscheduler.h <span style="color: grey">(015d23a)</span></li>
<li>src/file/indexscheduler.cpp <span style="color: grey">(79cd5bf)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/114983/diff/" style="margin-left: \
3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic