Issue 293

23rd June 2013 by KDE Commit-Digest Team

Contributors

Marta Rybczynska

This Week...

Nepomuk adds new Indexer for EPub files. KAlgebra supports fullscreen mode. Artikulate gains new language models. Krita implements a global alpha lock "Preserve Layer Alpha". In Amarok, volume fadeout is now also available for pause. libkface adds a new preprocessing algrithm. Optimizations in Nepomuk, Dolphin and rekonq. GSoC work in Nepomuk and Digikam.

Statistics

Commits 1461 by 155 developers
Open Bugs 21074
Open Wishes 15922
Bugs Opened 266 in the last 7 days
Bugs Closed 302 in the last 7 days

Commit Summary

Module Commits
/trunk/l10n-kde4
133
 
/branches/stable
54
 
/trunk/l10n-support
40
 
/plasma/desktop
24
 
/doc/index.docbook
16
 
/trunk/www
15
 
/branches/work
12
 
/server/lib
12
 
/projectmanagers/cmake
9
 
/desktop/applets
9
 
Files Developer Commits
456
 
Laurent Montel
158
 
171
 
Heena Mahour
57
 
123
 
Shantanu Tushar Jha
44
 
132
 
Boudewijn Rempt
44
 
99
 
Martin Klapetek
33
 
96
 
Denis Steckelmacher
32
 
96
 
Aaron J. Seigo
32
 
90
 
David Faure
31
 
90
 
Aleix Pol Gonzalez
31
 
90
 
Vishesh Handa
30
 

Internationalization (i18n) Status

Language Percentage Complete
Ukrainian (uk)
100%
 
Brazilian Portuguese (pt_BR)
99%
 
Swedish (sv)
99%
 
Polish (pl)
97%
 
French (fr)
97%
 
Dutch (nl)
97%
 
Spanish (es)
97%
 
German (de)
96%
 
Estonian (et)
95%
 
Italian (it)
92%
 

Bug Killers

Person Bugs Closed
Alex Fiestas
75
 
Christoph Feck
33
 
Jekyll Wu
18
 
Frank Reininghaus
12
 
Myriam Schweingruber
11
 
Boudewijn Rempt
11
 
Gilles Caulier
10
 
Simon Andric
10
 
Laurent Montel
9
 
Thomas Lübking
7
 

Commit Countries

Commit Demographics

Sex

Age

Contents

  Bug Fixes Features Optimization Security Other

Accessibility

[]     

Development Tools

[]     []

Educational

[] []    

Graphics

[]   []   []

KDE Base

[] [] []   []

KDE-PIM

[] [] []   

Office

[] [] []   

Konqueror

     

Multimedia

[] [] []   

Networking Tools

[] [] []   

User Interface

     

Utilities

[] []    

Games

     

Other

     

There are 76 selections this week

Bug Fixes

Accessibility

Peter Grasch committed changes in [simon] /speechmodelcompilation:

Direct (non-threaded) call to poison phonemes to avoid race condition

Development Tools

Kevin Funk committed changes in [kdevelop/4.5] /common:

Correct Basic C++ Template

Removes unnecessary new lines in Header & Create proper implementations for functions (with return type.)

Diffs: 1, 2, 3 Revision 26cfd87...
Ralf Habacker committed changes in [umbrello] /:

Fixed "mouse double click collision."

This patch fixes the 'delete point' collision:
- remove of 'delete points' double click action
- a new context menu entry named 'delete point'
- a new 'delete point' mouse action triggered by ctrl-mouse-left-click

Diffs: 1, 2, 3, 4, 5, 6, 7, 8 Revision ba16461...
Ralf Habacker committed changes in [umbrello] umbrello/widgets/combinedfragmentwidget.cpp:

Fixed "Combined fragment has incomplete color settings".

Andreas Pakulat committed changes in [kdevelop] projectmanagers/custom-buildsystem/custombuildsystemplugin.cpp:

Try to ensure that the configuration is valid before returning it

This is an attempt to avoid running into (IMO incorrect) assert's in
KConfigGroup's read* functions when having kdelibs built in debug mode.
Unfortunately KConfig* has no other API to tell it that it should create
a KConfigGroup thats been fetched, other than causing a filesystem access
to sync.

So this trades crashing KDevelop with one doing potentially more fs syncs.

I couldn't verify wether this fixes the issue since I don't have a
debug-kdelibs anywhere.

Educational

Aleix Pol Gonzalez committed changes in [kalgebra] src/kalgebra.cpp:

Fully disable menus when changing the tab

This way we get to disable the shortcuts as well, so we have better
scoping. If you press Ctrl+S in 2D, you'll get to save the thing you
are viewing.
Additionally Ctrl+- or Ctrl++ won't change the zoom in 2D.

Andreas Cord-Landwehr committed changes in [rocs] /Ui:

Save code files of projects correctly.

Script files that were added to project after initial project creation
were not being saved. Upon reopening the project, these script files
would be missing and potentially irrecoverable. Script files were also
not being closed in the actual editor.

-Rocs now prompts user to name the untitled script that is created for
all new projects if the user decides to save the new project.
-Script files that are added to the project after initial project
creation are saved to it.
-When the user closes a project, script files open in the editor are now
closed.
-Scripts can now only be saved if the project has been saved.
-Scripts are given unique names to prevent files from being overwritten.


REVIEW: 110999

Graphics

Fabio D'Urso committed changes in [okular/KDE/4.10] core/tilesmanager.cpp:

Fix tiles issue when zooming out a rotated document

The issue can be reproduced by zooming a rotated document to 1600% then
back to 800%

REVIEW: 110905

Aurélien Gâteau committed changes in [gwenview] /:

Use a QPersistentModelIndex to store the index under cursor

This fixes the crash described in:

https://git.reviewboard.kde.org/r/111158/

Only ThumbnailBarViewDelegate seems to suffer from this, but it is not a good
idea to store a QModelIndex anyway (see QModelIndex doc)

REVIEW: 111161

Committing w/o review because this is a minor change and I want it to go in
before beta2 gets out

Diffs: 1, 2, 3 Revision a7dceb9...
Aurélien Gâteau committed changes in [gwenview] /:

When going from view to browse, ensure thumbnail view is scrolled to show the current item

REVIEW: 110968

FIXED-IN: 4.11.0

KDE Base

Vishesh Handa committed changes in [nepomuk-core] services/filewatch/kinotify.cpp:

Kinotify: Handle overflow events

There are cases when we are not able to handle all of the events fast
enough, and an overflow event is generated. In that case, we just need
to continue processing events.

Prior to this patch, the path would get constructed as '/' and that
would lead to an assert being hit.

This behaviour can be reproduced by running the nepomukfilewatch in
valgrind and creating 10000 files. The event occurs somewhere between
the 9200 and 9500th file on my system.

Frank Reininghaus committed changes in [kde-baseapps/KDE/4.10] /src:

Ensure that the "Sort by Type" setting is respected

Before this commit, switching from, e.g., "Sort by Name" to "Sort by
Type" sometimes had no effect until the view was refreshed. The problem
was that the re-sorting was triggered before the type information was
actually added to the model.



FIXED-IN: 4.10.5
REVIEW: 111004

Thomas Lübking committed changes in [kde-workspace] kwin/geometry.cpp:

exclude padding from snap delta of screen snap

using the actual delta this casewise causes false
preference for window snapping (less to move)
this restores the pre snap-to-content behavior
in that regard and delta isn't used for anything else.


FIXED-IN: 4.11
REVIEW: 111139

David Gil Oliva committed changes in [kdelibs/frameworks] /http:

Port time_t to QDateTime in HTTP (KIOSLAVE)

--Fix some warnings about parentheses in conditionals.
--Delete some includes that apparently weren't used.

TODO:
--Port QRegExp to QRegularExpression
--Port QLatin1String to QStringLiteral where needed

Vishesh Handa committed changes in [nepomuk-core] services/filewatch/metadatamover.cpp:

FileWatch: Do not delete metadata for files in a folder

When we receive an event to delete the metadata of a folder, we do not
need to query for all the files in that folder and delete the metadata
for those files.

Inotify gives us a separate event for each file that has been deleted
before it gives the folder deletion event. So the query being used was
rather useless.

This seems to have been implemented in 1b7ea56fbf0 in 2010 to mimic what
we were doing for file moves. Strange.

Heena Mahour committed changes in [kde-workspace/heenamahour/window-list] /:

Enable color correction only after successfuly contacting KolorManager

Prevents the possiblity of using shaders modified for color correction
without valid data from KolorManager. If that happened, everthing
blacked out.

Now the color correction shaders are enabled only after successfuly
contacting KolorManager.

The issue was highlighted after ab7e228d.

Frank Reininghaus committed changes in [kde-baseapps] /src/statusbar:

Do not use fixed widths for the widgets in the status bar

This fixes the problem that the Dolphin window has a rather large
minimum size. A side effect of this was that splitting the view could
resize the window (because twice the minimum status bar width was
needed then).


REVIEW: 110966
FIXED-IN: 4.10.5

Vishesh Handa committed changes in [kde-baseapps] dolphin/src/kitemviews/kfileitemmodelrolesupdater.cpp:

Dolphin: Do not try to connect to Nepomuk if it is not running

Each time one uses any of the Nepomuk classes, an attempt is made to
connect to the database. This slows down the application since
connecting to Nepomuk is not so cheap.

Heena Mahour committed changes in [kde-workspace/heenamahour/window-list] libs/taskmanager/taskmanager.cpp:

ignore full screen windows that are on different physical screens

imho this is doing window management, and since it really crosses over
into knowing which window has focus ought to be done by the window
manager. kwin devs disagree, so here we go implementing window
managery logic in libtaskmanager.

KDE-PIM

Aurélien Gâteau committed changes in [kdepim] mailcommon/folder/foldertreewidget.cpp:

Fix "jump to folder" sometimes not unselecting previously selected folder

Steps to reproduce:
- Start KMail
- Press J
- Pick a folder
- folder is selected, but Inbox is still selected

REVIEW: 111100

Jan Kundrát committed changes in [trojita] src/MSA/SMTP.cpp:

SMTP: do not signal success after hitting a failure earlier on

OK, this is some ugly code -- but it appears that it makes QwwSmtpClient more or
less work. That class is in a bad need for replacement :(.

The problem was that the onDisconnected event handler within the QwwSmtpClient
simply checked whether there are any pending commands, and if there are none, it
proceeded further towards emitting success even though the connection has never
succeeded. This is a wrong assumption in the client's code -- signalling success
just because the connection was severed before any command had a chance to be
queued is not what I would expect when using such component.

This code does *not* prevent duplicate signals about success, but I have yet to
see a situation where it breaks something.


REVIEW: 111073

Laurent Montel committed changes in [kdepim] plugins/messageviewer/bodypartformatter/vcardmemento.cpp:

Be sure to show a vcard even if we have several item with same emails

Allen Winter committed changes in [kdepimlibs] /parsers/plan2:

Plug 2 groups of memory leaks in the parser plan2.

Thanks for the patch Eugene!

FIXED-IN: 4.11

Laurent Montel committed changes in [kdepim] /src:

Fix Bug 111324 - shortcut for show/hide of an application bar

FIXED-IN: 4.11


(bug was created in 2005...)

Aurélien Gâteau committed changes in [kdepim] /folder:

Fix FolderSelectionDialog not always selecting matching folders

Easiest way to reproduce the bug is to type slowly enough to cause
QTreeView::keyboardSearch() to reset itself.


FIXED-IN: 4.11.0

https://git.reviewboard.kde.org/r/111098/

Laurent Montel committed changes in [kdepim] incidenceeditor-ng/incidencedialog.cpp:

Fix Bug 225720 - Journal/Calendar (etc) dialogs do not remember their last size

FIXED-IN: 4.11

Lindsay Mathieson committed changes in [kdepim-runtime] /nepomukfeeder:

Fix bug in running job count

Fixes problem where the count of running jobs becomes less than zero
(-1), causing an assert in itemqueue.cpp at line 113 to trigger. This
was happening because ItemQueue::clear() was setting the running job
count to zero when there was unfinished jobs, when the jobs actually did
finish the running job count was decremented to below zero.
The batch fixes this by keeping a QList<KJob *> of running jobs, adding
them when created and removing when the job finishes (via the result
signal). When ItemQueue::clear() is called any outstanding jobs are
silently killed and the list cleared.


REVIEW: 111089

Office

Boudewijn Rempt committed changes in [calligra] /ui:

Fix a bug when enabling mirroring and trying the scratchpad

When enabling horizontal and vertical mirroring, painting on the scratchpad
was broken. This was because the scratchpad creates a KisResourceSnapshot
with a 0 image pointer. The snapshot then created a default KisDefaultBounds
object with a 0 image pointer.

That means the KisToolFreehandHelper was told to use an infinite rect
for calculating the mirroring, and iterating over infinity takes a bit of
time.

Fixed by passing the scratchpad's defaultBounds object to the freehand
tool helper, which passes it to the resources snapshot.

Diffs: 1, 2, 3, 4, 5 Revision 4763ed0...
Boudewijn Rempt committed changes in [calligra] /ui/opengl:

Fix bug 320679: correctly convert input pixels to textures

We hardcoded GL_BGRA, but we actually have GL_RGBA as a possibility now,
for the floating point colorspaces.



Conflicts:
krita/ui/opengl/kis_opengl_image_textures.cpp
krita/ui/opengl/kis_texture_tile.cpp

Multimedia

Mark Kretschmann committed changes in [amarok] /dbus/mpris1:

Add dbus methods for updating podcasts and adding new podcasts.

Bug 205509 - JJ: add dbus functions to update podcasts and download podcast tracks
I added two new methods for dbus: UpdateAllPodcast and AddPodcast

REVIEW: 110855

FIXED-IN: 2.8

Mark Kretschmann committed changes in [amarok] src/browsers/filebrowser/FileBrowser.cpp:

Fix potential null pointer dereference.

Error was introduced with b2149a7. Detected by clang-analyzer.

Networking Tools

Emmanuel Lepage Vallee committed changes in [sflphone-kde] /:

[ #25283 ] Fix double click with calls on hold

Diffs: 1, 2, 3, 4, 5, 6, 7 Revision be301d5...
Emmanuel Lepage Vallee committed changes in [sflphone-kde] /:

[ #21732 ] Make call categories sortable again

Diffs: 1, 2, 3, 4, 5, 6, 7, 8 Revision c0e9739...
Andrea Diamantini committed changes in [rekonq] /webtab:

Let fixed behavior when site is NOT loaded
work also when you don't have connection from the beginning.

Utilities

Konstantinos Smanis committed changes in [kcm-grub2] src/convertDlg.cpp:

Manage some ImageMagick crashes.

1. Do not crash when saving in a directory where we don't have write
permissions.
2. Do not crash when using path names with non-ASCII characters.

Features

Educational

Aleix Pol Gonzalez committed changes in [kalgebra] /:

Add fullscreen mode to KAlgebra

Oindrila Gupta committed changes in [artikulate] /languages:

Hindi Language specification has been added.
REVIEW: 111045

Andreas Cord-Landwehr committed changes in [rocs] /Ui:

Query close for Quit action.

Simplify query for closing unsaved project and connect Quit
standard action to query.

KDE Base

Frank Reininghaus committed changes in [kde-baseapps/KDE/4.10] /src/views:

Allow renaming multiple files without number if extensions are different

Normally, we only allow renaming multiple files if the new file name
contains a contiguous sequence of '#' placeholders, which are then
replaced by numbers.

However, if all extensions are different, we can also rename the files
without such a placeholder because the original extension is preserved
when renaming.

This had been possible some time ago already. That this "accidental
feature" was lost was a side effect of the fix for bug 318942.


FIXED-IN: 4.10.5
REVIEW: 111079

Aaron J. Seigo committed changes in [kdeplasma-addons] /:

add translator runner by David Baum


REVIEW:109773

Diffs: 1, 2, 3, 4, 5, 6, 7 Revision aa7f5ef...
Vishesh Handa committed changes in [nepomuk-core] /fileindexer/indexer:

Indexer: Make the plugins only extract a part of the full text

Introduce a maxPlainTextSize() which informs the plugin how much text
they should extract.

This is useful in two ways -

1. Sometimes one doesn't want any of the plain text, so one can set it
to 0. This is used by the FileMetadataWidget to directly display the
indexed data. Since we do not show the plain text, we do not need to
extract it.

2. Virtuoso cannot handle queries above a certain number of bytes.
2500243 seems to be the magic number. If you go above this limit,
just a '0' is stored. Therefore it doesn't make sense to extract all
of the plain text, when virtuoso can clearly not handle all of it.

Virtuoso does not support streaming in text

Emmanuel Pescosta committed changes in [kde-baseapps] /src:

Update the "Move to Trash" action's and the "Delete" action's
enabled state in the context menu for read only files/folders (also
archives).


FIXED-IN: 4.11
REVIEW: 111160

KDE-PIM

Sergio Luis Martins committed changes in [kdepim-runtime] resources/icaldir/icaldirresource.cpp:

Add the sub mime types too (event/todo/journal).

The calendar ETM doesn't listen to the generic text/calendar
mimetypes for optimization reasons ( korgac doesn't want journals,
plasma doesn't want todos/journals, summary view doesnt want to-dos
etc).

all other resources already use the sub mime types.

Michael Bohlender committed changes in [kdepim] /mail:

kmail-mobile: move Identities to the Settings page

Diffs: 1, 2, 3 Revision 74e066c...
Sven Brauch committed changes in [ktp-common-internals] /debugger:

Use KatePart to display debug output in the debugger

To get old features back:
- to save the log, just press Ctrl+S
- to set marks, press Ctrl+B; you can jump between them with Alt+PgUp/Dn
- to search, press Ctrl+F
You might need to focus the editor widget.
When a new message arrives, the view jumps to the current cursor position.

Diffs: 1, 2, 3, 4 Revision a06ad55...
Sergio Luis Martins committed changes in [kdepimlibs] /calendar:

Add the ability to load only specific mime types.

Will save memory in plasma-clock, kontact-mobile, summary-view
etc, which don't need all 3 mime types ( events, journals, to-dos).

Diffs: 1, 2, 3 Revision a5744ff...
Dan Vratil committed changes in [ktp-desktop-applets] /src/declarative/contents/ui:

Make it possible to send email from contact applet even when contact is offline

Slightly changes behavior of the applet: the avatar is always clickable, but the
individual actions are enabled only when online (except for the "send mail" action,
which is enabled all the time)

REVIEW: 111095

FIXED-IN: 0.7.0

Office

Inge Wallin committed changes in [calligra/flake-shapeborders-ingwa] libs/odf/KoBorder.cpp:

Improve the old loading code for borders.

The old code didn't check for individual edges (-top, -bottom, -left,
-right) when there was a common border definition.

C. Boemann committed changes in [calligra] /plugins/tools/tool_crop:

Make the crop tool able to grow

Introduce an option to have different kinds of decorations on the croptool. For
now the following guides are implemented, but they are defined in a parametric way so
it is a junior job to create more guides:
- passport photo
- thirds
- fifths

Diffs: 1, 2, 3 Revision 4dd828b...
Boudewijn Rempt committed changes in [calligra] /:

Load the resultion information from the embedded PNG images

Now Krita can roundtrip the resolution information in ora files,
but other apps will still reset that to 72 DPI. We need to extend
OpenRaster for this.

Boudewijn Rempt committed changes in [calligra] /plugins/extensions/dockers/imagedocker:

Change the image docker combobox to show common locations and drives

Boudewijn Rempt committed changes in [calligra] /ui:

FEATURE: Implement a global alpha lock "Preserve Layer Alpha"



This works by overriding the channel flags set in the KisResourcesSnapshot.

Bug 687451:
 
735437 days
Diffs: 1, 2, 3, 4, 5, 6, 7 Revision 7bc0fd2...

Multimedia

Mark Kretschmann committed changes in [amarok] src/context/applets/analyzer/AnalyzerApplet.cpp:

Add tooltip to analyzer applet explaining how to configure it.

As the analyzer applet does not have a header with a configure button
like the other applets do (it would take up too much space I think),
it's not obvious how it can be configured. So I've added this tooltip
saying that you can right-click it.

Is this a good compromise?

Shantanu Tushar Jha committed changes in [plasma-mediacenter] mediaelements/mediaplayer/MediaPlayer.qml:

Restore key bindings for media player

Shantanu Tushar Jha committed changes in [plasma-mediacenter/pagestack-shantanu] mediaelements/mediaplayer/MediaPlayer.qml:

Show a busy indicator when streaming

Mark Kretschmann committed changes in [amarok] /:

Volume fadeout is now also available for pause.

We had fadeout on stop before, but as Amarok uses pause instead of stop
in its default configuration, the feature was lost for many use cases.
The new (optional) fadeout on pause fixes this.

Diffs: 1, 2, 3, 4, 5, 6, 7 Revision 6d2adde...

Networking Tools

Dan Vratil committed changes in [ktp-text-ui] /:

Show a message when no logs are available for given contact or day

Shows a label saying "There are no logs for this day" or "There are no
logs for this contact" when user selects a date without logs or starts
logviewer with ID of a contact for which we have no logs.

REVIEW: 111088

FIXED-IN: 0.7.0

Daniel Kreuter committed changes in [choqok] /:

Follow / Unfollow and Images

Switched from get method to post method for follow, unfollow and block.

Changed URL for Image uploading.

Utilities

Jonathan Michael Thomas committed changes in [muon/USCResource] /backends/ApplicationBackend:

Reload the sources list after adding a debline for the newly-purchased app.
(Still todo: only reload for the single repository)

Diffs: 1, 2, 3, 4 Revision 7361ce1...

Optimization

Graphics

Marcel Wiesweg committed changes in [libkface] /:

Add preprocessing based on the technique developed by Tan and Triggs:

Tan, X., and Triggs, B. "Enhanced local texture feature sets for face recognition under difficult lighting conditions.". IEEE
Transactions on Image Processing 19 (2010), 1635–650.

I noted an unsignificant change in recognition performance on the ORL database,
the real value will need to be evaluated with real-world pictures.

Diffs: 1, 2, 3, 4 Revision 501cbd9...

KDE Base

Vishesh Handa committed changes in [nepomuk-core] /:

New Indexer for EPub files

Diffs: 1, 2, 3, 4, 5, 6 Revision cfcdb30...
Emmanuel Pescosta committed changes in [kde-baseapps] dolphin/src/kitemviews/kfileitemmodel.cpp:

Avoid a unnecessary resorting when items are changed, only resort the items when the sorting role value is changed.


FIXED-IN: 4.11
REVIEW: 111146

Vishesh Handa committed changes in [nepomuk-core] services/filewatch/activefilequeue.cpp:

ActiveFileQueue: Use a QHash instead of a QList for the emitted entires

By default everything gets inserted in the emitted entries list, and
then when it appears the second time, it's forwarded to the actual
queue.

Since we need to do a find operation on the emitted entries, it's much
better to use a hash, instead of a list.

Tested by creating 10000 files

Frank Reininghaus committed changes in [kde-baseapps] /src/kitemviews:

Try to do at least a "fast" icon loading for all items

If all icons for the visible items could be loaded in 200 ms, we
continue loading icons without mime type determination for all items
until the 200 ms are over. This reduces the risk that the user ever
sees "unknown" icons.

REVIEW: 111011

Frank Reininghaus committed changes in [kde-baseapps] /src/kitemviews:

Simplify handling of preview jobs

This patch changes two things about the way we handle the preview jobs:

(a) Rather than passing a KFileItemList to startPreviewJob(),
remembering the leftovers in the member variable
m_pendingPreviewItems and then starting a new preview job for
these, we append items that need a preview to this member, and let
startPreviewJob() take its input from there. This simplifies the
code greatly.

(b) To prevent that we start preview jobs with just one item and also
that the GUI is frozen too long by startPreviewJob(), we take the
following approach:

* If the mime type of the first pending item is known, the function
has probably been called by startUpdating(), which has determined
mime types for the visible items already. startUpdating() has
also blocked the GUI, so we just take all items at the beginning
of the list with known mime type, and do not do any expensive
mime type determination in startPreviewJob().

* If the mime type of the first pending item is unknown, the
function has probably been called by slotPreviewJobFinished(). In
that case, we can afford to block the GUI for a short while, so
we determine mime types for 200 ms.

REVIEW: 111008

KDE-PIM

Sergio Luis Martins committed changes in [kdepimlibs] kcalcore/icalformat_p.cpp:

Reset dirty fields after reading the incidence.

The incidence is new, nothing is dirty.
This saves 500 bytes of heap per incidence.
12MB for my test calendar.

Jan Kundrát committed changes in [trojita] src/Gui/PartWidgetFactory.cpp:

GUI: enable loading of cached items not immediately shown when offline

Previously, the second part of a multipart/alternative (i.e. something which is
not shown on startup) was replaced by a label showing "Offline" even though the
actual data were available in the cache. That was not correct.

Sergio Luis Martins committed changes in [kdepim] /:

Don't load journals for groupware purposes.

Saves memory.

Sergio Luis Martins committed changes in [kdepimlibs] kcalcore/incidencebase.cpp:

Delay initialization of mOrganizer until first use.

Reduces Incidence's memory footprint since you wont call
organizer() for most Incidences.

Office

Boudewijn Rempt committed changes in [calligra] krita/ui/kis_view2.cpp:

Do not save the original image when saving a backup



Apparently, the original design was to save the current state both as
a backup and under the original filename. That makes sort of sense,
but it is confusing because F4 doesn't tell the user it is "Saving the
plus a backup", but that it is "saving a backup".

Multimedia

Edward Hades Toroshchin committed changes in [amarok] /:

more unused private members removed

Please double check the "never been used" items, especially if the
authors are still available.

ServiceBrowser::m_usingContextView has been left over since
dd303617dd2cf0c0113f710660df78fba38421d6

SqlPodcastEpisode::m_batchUpdate has never been used since its
introduction in 680b5102d0f9f011ec993d5277b3a9b417848443

Transcoding::AssistantDialog::m_defaultEncoder has never been used
(introduced in 1a0287f7925d92a05a530ab83fffbc9afae67e86)

AppletToolbarAppletItem::m_labelPadding left over after
568bfcb808ad43bfad39b20441a291a6392f4092

LikeBackDialog::m_requestNumber_ unused in the library imported in
f4683691c5ddc692de2f3ccf54f4d8316c807c4c

DynamicView::m_expandToggledWhenPressed _was_ used, but the code has
always been commented out since 40b768ec4a08d23dceea8f72df1c601574614081
and removed completely in 2fe7dd743c66b2fb9f9a6bbb7b2d50730a843056

Daap::Reader::m_loadedTracks has never been used since patch of
382d427f87f6702a5431b325c56be3b79de45b6c

OpmlDirectoryMetaFactory::m_service never used (introduced in
19eb9efe689628de819842487fde489bddaca13b)

Diffs: 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 (+ 5 more) Revision 57d2c7f...

Networking Tools

Andrea Diamantini committed changes in [rekonq] src/rekonqpage/newtabpage.cpp:

Speed up construction of the history page

The old version was taking ages to even appear on my setup ("ages" being defined
as many minutes at least). Callgrind was crashing, perhaps due to webkit's JIT,
so I simply sampled the backtrace "randomly" by hand to see where the most CPU
time was being spent. These two methods came up way too often, so I did the
following:

1) Do not use KIconLoader within the inner loop, it's slow. It was slow when I
debugged too long startup of KPhotoAlbum, it produced visible file IO, and it is
trivial to speed this up. However, it was still taking time.

2) Do not request the ".historyitem" over and over again. A cached "pre-found"
copy (we're still cloning it) seems to help.

Together, these make it possible to load the history in <4 minutes here. It's
still a completely insane amount of time, but at least it appears, ultimately.

REVIEW:111053

Other

Development Tools

Michael Pyne committed changes in [kdesrc-build] /:

svn: Remove SSL signature for https://svn.kde.org/

In the past some KDE SVN modules used the "svn external" feature to pull
in other SVN repositories. These svn-external links were on the *server*
side so the link URL had to use a single scheme.

This scheme was https, which caused quite some problems for anonsvn
users just trying to update from SVN being an unattended kdesrc-build
run, as the SSL cert for svn.kde.org was not signed by a well-known CA.
As a result kdesrc-build updated svn automatically with the expected KDE
SVN SSL signature to avoid blocking on the interactive warning.

This is no longer required. The modules that used svn externals have
pretty much moved onto git now. The new svn server does not even have an
https interface, so this is dead code now, and removed accordingly.

Diffs: 1, 2, 3 Revision d48fd6e...

Graphics

Gowtham Ashok committed changes in [digikam/development/gsoc2013-imgqsort] /imgqsort:

Added Blur Detection Methods

Diffs: 1, 2, 3, 4, 5 Revision 8b41687...

KDE Base

Wojciech Kapuscinski committed changes in [kdelibs/frameworks] /:

Create KWallet framework and move kwallet.h and others

V2: fix issues from RR

REVIEW: 111105

Diffs: 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 (+ 1 more) Revision 7983fd2...
Denis Steckelmacher committed changes in [nepomuk-core/steckdenis-gsoc2013] /:

Simplify the pattern language

Translators do not need to know that the parser wants to match a comparison
operator, a literal string or a type hint. This kind of matching is now done
directly into the passes.

Diffs: 1, 2, 3, 4, 5, 6, 7, 8 Revision 7c5e6ef...