Feed Advanced Search

Apr 15 2020

jtamate abandoned D10989: Check for nullptr in indexForNode.
Apr 15 2020, 12:57 PM · Frameworks

Mar 14 2020

jtamate added a comment to T12815: Create Calligra Framework by separating out applications and libraries.

As I have some free time due to the virus and I'm trying to get up to date.

Mar 14 2020, 5:04 PM · Calligra: 3.0

Sep 10 2019

jtamate added a comment to D21277: Add "Cig Waste No. 2" theme.

Sorry, bad Copy/Paste. In the same article:

Sep 10 2019, 5:26 PM

Sep 9 2019

jtamate added a comment to D21277: Add "Cig Waste No. 2" theme.

I'm not totally sure this theme is legal in Spain:
https://www.boe.es/buscar/act.php?id=BOE-A-2005-21261

Sep 9 2019, 5:40 PM

Jul 21 2019

jtamate closed D10742: get rid of the raw KFileItem pointers in KCoreDirListerCache.

@jtamate Does that "let's continue in that other review" mean we should cancel this one? Still shows on the list of reviews to consider

Jul 21 2019, 6:11 PM · Frameworks

Apr 19 2019

jtamate added a comment to T10812: KDE Applications.

Apps that are in the bundle have inconsistent versioning; most use the bundle's own versioning scheme, but others use their own

I personally disagree this is a problem. It let's applications hop on and off the release and keep their versioning number intact.

I'm using Dolphin 18.12.3, which comes from KDE Applications 18.12.3. But the bundle also comes with Okular. Am I using Okular 18.12.3? Or Okular 1.6.3? The promo material says 18.12.3. The version number in my distro's packaging says 18.12.3. Discover, which uses the distro packaging metadata, says 18.12.3. But the About Okular window says 1.6.3. This is confusing and problematic from a user, packager, and promo perspective because it's not clear which version number they should use.

Apr 19 2019, 8:21 AM

Apr 1 2019

jtamate accepted D14724: autotests: don't fail if an unrelated window shows up..

As I don't have enough contiguous free time even to test it again, feel free to commit.
You can add a comment to remember that it doesn't fix all the cases but is an improvement over previous situation.

Apr 1 2019, 7:13 AM · Frameworks

Mar 5 2019

jtamate planned changes to D15408: Don't assert deleting the temporary file.

The more I try to understand my own explanations, the less I agree with my past me.

Mar 5 2019, 6:29 PM · Frameworks

Mar 4 2019

jtamate added a comment to D19500: [KDirModel] Fix job urls change signal connection.

+1.
Just out of curiosity, Did you notice the problem because something was missing in execution? Unfortunately, neither the compiler nor I didn't noticed it. :-(

Mar 4 2019, 8:11 AM · Plasma

Dec 24 2018

jtamate committed R241:211c3a680ea7: Change the path for every item of the subdirectories in a directory rename (authored by jtamate).
Change the path for every item of the subdirectories in a directory rename
Dec 24 2018, 8:35 AM
jtamate closed D17619: Change the path for every item of the subdirectories in a directory rename.
Dec 24 2018, 8:35 AM · Frameworks

Dec 23 2018

jtamate updated the diff for D17619: Change the path for every item of the subdirectories in a directory rename.

With this version, the test with the patch applied is everlasting in:

Dec 23 2018, 8:34 AM · Frameworks

Dec 22 2018

jtamate added inline comments to D17619: Change the path for every item of the subdirectories in a directory rename.
Dec 22 2018, 1:14 PM · Frameworks
jtamate updated the diff for D17619: Change the path for every item of the subdirectories in a directory rename.

Changes requested done and reduced code duplication.

Dec 22 2018, 1:13 PM · Frameworks
jtamate updated the diff for D17619: Change the path for every item of the subdirectories in a directory rename.

Added the unit test. It crashes for me every time without the patch and none with the patch.

Dec 22 2018, 8:54 AM · Frameworks

Dec 17 2018

jtamate added a comment to D17619: Change the path for every item of the subdirectories in a directory rename.

I'll work on the unit test this weekend. I don't currently have enough free time on weekdays.

Dec 17 2018, 9:09 PM · Frameworks
jtamate retitled D17619: Change the path for every item of the subdirectories in a directory rename from fix for bug 401552 to Change the path for every item of the subdirectories in a directory rename.
Dec 17 2018, 9:07 PM · Frameworks
jtamate updated the summary of D17619: Change the path for every item of the subdirectories in a directory rename.
Dec 17 2018, 8:16 PM · Frameworks
jtamate updated the diff for D17619: Change the path for every item of the subdirectories in a directory rename.

There is no need for a unit test, it is already in kdirmodeltest (but perhaps could be expanded in another patch).

Dec 17 2018, 6:52 PM · Frameworks

Dec 16 2018

jtamate updated the diff for D17619: Change the path for every item of the subdirectories in a directory rename.

I think this time I got the problem right.
One of the classics: I was modifying the list while it was being used.

Dec 16 2018, 12:36 PM · Frameworks
jtamate updated the summary of D17619: Change the path for every item of the subdirectories in a directory rename.
Dec 16 2018, 8:52 AM · Frameworks
jtamate added a comment to D10742: get rid of the raw KFileItem pointers in KCoreDirListerCache.

Let's continue on D17619

Dec 16 2018, 8:49 AM · Frameworks
jtamate requested review of D17619: Change the path for every item of the subdirectories in a directory rename.
Dec 16 2018, 8:48 AM · Frameworks

Dec 15 2018

jtamate added a comment to D10742: get rid of the raw KFileItem pointers in KCoreDirListerCache.

I'm able to reproduce the bug with the following steps:

  • Create a folder structure X/X1/X2/X3/X4 and add a new empty text file into each folder.
  • Within Dolphin, open a tab for each folder.
  • In the tab showing X contents, rename X1 to X1_ and the crash is there.
Dec 15 2018, 11:21 AM · Frameworks
jtamate added a comment to D10742: get rid of the raw KFileItem pointers in KCoreDirListerCache.

I've been able to reproduce the bug:

Dec 15 2018, 10:30 AM · Frameworks

Oct 26 2018

jtamate added a comment to D16344: Do not try to fallback to "less secure" protocols.

What protocol does KTcpSocket::SecureProtocols implement (I can't guess it)? If it is the same as QSsl:SecureProtocols
it does:
On the client side, this will send a TLS 1.0 Client Hello, enabling TLSv1_0 and SSLv3 connections. On the server side, this will enable both SSLv3 and TLSv1_0 connections.

Oct 26 2018, 3:20 PM · Frameworks

Oct 22 2018

jtamate closed D16349: [kdirlistertest] Wait a little longer for the lister to finish.
Oct 22 2018, 5:20 PM · Frameworks
jtamate committed R241:87a2f084be83: [kdirlistertest] Wait a little longer for the lister to finish (authored by jtamate).
[kdirlistertest] Wait a little longer for the lister to finish
Oct 22 2018, 5:20 PM

Oct 21 2018

jtamate requested review of D16349: [kdirlistertest] Wait a little longer for the lister to finish.
Oct 21 2018, 9:55 AM · Frameworks

Oct 14 2018

jtamate committed R241:63093179579c: Avoid waiting for user actions when kwin Focus stealing prevention is high or… (authored by jtamate).
Avoid waiting for user actions when kwin Focus stealing prevention is high or…
Oct 14 2018, 9:57 AM
jtamate closed D16072: Avoid waiting for user actions when kwin Focus stealing prevention is high or extreme.
Oct 14 2018, 9:57 AM · Frameworks
jtamate added a comment to D16072: Avoid waiting for user actions when kwin Focus stealing prevention is high or extreme.

Created https://bugreports.qt.io/browse/QTBUG-71137

Oct 14 2018, 9:53 AM · Frameworks

Oct 10 2018

jtamate abandoned D11282: less expensive findByUrl in KCoreDirListerCache.
Oct 10 2018, 6:42 AM · Frameworks

Oct 9 2018

jtamate committed R241:20b89972b643: get rid of the raw KFileItem pointers in KCoreDirListerCache (authored by jtamate).
get rid of the raw KFileItem pointers in KCoreDirListerCache
Oct 9 2018, 4:47 PM
jtamate closed D10742: get rid of the raw KFileItem pointers in KCoreDirListerCache.
Oct 9 2018, 4:46 PM · Frameworks
jtamate requested review of D16072: Avoid waiting for user actions when kwin Focus stealing prevention is high or extreme.
Oct 9 2018, 1:21 PM · Frameworks

Oct 8 2018

jtamate added a comment to D16020: KFilePlacesModel: fix previous commit to avoid duplicating devices.

Just in case, I'm subscribed now to the RSS https://build.kde.org/job/Frameworks/job/kio/rssFailed

Oct 8 2018, 2:40 PM · Frameworks
jtamate planned changes to D15328: kfilewidget: convert connect syntax.

QTest::qWaitForWindowActive fails because I use kwin Focus stealing prevention High, therefore the windows doesn't become active until I click on them in the task bar or switch to them.
And qWaitForWindowExposed doesn't help because the window/widget needs to have the focus.

Oct 8 2018, 8:34 AM · Frameworks
jtamate updated the diff for D15328: kfilewidget: convert connect syntax.

Fix a crash ,caused by a still connected signal, after running again the unittests.
kfilewidgettest still doesn't pass because QTest::qWaitForWindowActive fails for me.

Oct 8 2018, 7:46 AM · Frameworks
jtamate accepted D16020: KFilePlacesModel: fix previous commit to avoid duplicating devices.

I am really, really sorry. I have no excuses to not have run the autotests (that I usually run after my first fiasco). Hopefully, after my second fiasco, I'll run them before creating any review request.

Oct 8 2018, 7:17 AM · Frameworks

Oct 7 2018

jtamate committed R37:409e7ac4256d: Trivial micro optimizations (initialization lists) (authored by jtamate).
Trivial micro optimizations (initialization lists)
Oct 7 2018, 9:07 AM
jtamate closed D15922: Trivial micro optimizations (initialization lists).
Oct 7 2018, 9:07 AM · Krita

Oct 6 2018

jtamate closed D15371: Don't assign twice the same key to the action New Tab.

closed again by commits:
https://commits.kde.org/dolphin/5d180c5bb280f68cb98996044055e6891e29bab7
https://commits.kde.org/dolphin/b610ce5913ef3172f3cf15c0a1d93c255ea0da4f

Oct 6 2018, 8:35 AM · Dolphin
jtamate committed R318:b610ce5913ef: Merge remote-tracking branch 'origin/Applications/18.08' (authored by jtamate).
Merge remote-tracking branch 'origin/Applications/18.08'
Oct 6 2018, 8:33 AM
jtamate committed R318:5d180c5bb280: Don't assign twice the same key to the action New Tab (authored by jtamate).
Don't assign twice the same key to the action New Tab
Oct 6 2018, 8:27 AM

Oct 5 2018

jtamate updated the diff for D15371: Don't assign twice the same key to the action New Tab.

Keep both key sequences in case someone used ctrl+shift+n.

Oct 5 2018, 3:17 PM · Dolphin
jtamate updated the diff for D15371: Don't assign twice the same key to the action New Tab.

As it turns out, QKeySequence::AddTab is Ctrl+Shift+N under plasma.
Filled Qt bug: https://bugreports.qt.io/browse/QTBUG-70963
Change to Qt::CTRL + Qt::Key_T.

Oct 5 2018, 2:40 PM · Dolphin

Oct 4 2018

jtamate committed R37:0f87010adf82: Descend the array in the right way (authored by jtamate).
Descend the array in the right way
Oct 4 2018, 4:55 PM
jtamate closed D15905: Descend the array in the right way.
Oct 4 2018, 4:55 PM · Krita

Oct 3 2018

jtamate updated the diff for D15922: Trivial micro optimizations (initialization lists).

A comment disappeared in the conversion.

Oct 3 2018, 2:47 PM · Krita
jtamate requested review of D15922: Trivial micro optimizations (initialization lists).
Oct 3 2018, 2:43 PM · Krita
jtamate updated the diff for D15905: Descend the array in the right way.

Changed 2 more 5 by 4.

Oct 3 2018, 1:30 PM · Krita

Oct 2 2018

jtamate requested review of D15905: Descend the array in the right way.
Oct 2 2018, 6:57 PM · Krita
jtamate committed R37:97b9e0abfebc: Trivial micro optimizations (initialization lists) (authored by jtamate).
Trivial micro optimizations (initialization lists)
Oct 2 2018, 5:07 PM
jtamate closed D15894: Trivial micro optimizations (initialization lists).
Oct 2 2018, 5:07 PM · Krita
jtamate updated the diff for D15894: Trivial micro optimizations (initialization lists).

Changed comma style in initialization lists.
Using member initialization in the class declaration.
Added a constructor not detected before.

Oct 2 2018, 11:29 AM · Krita
jtamate requested review of D15894: Trivial micro optimizations (initialization lists).
Oct 2 2018, 10:23 AM · Krita
jtamate added inline comments to D15890: kimg_rgb: optimize away QRegExp and QString::fromLocal8Bit..
Oct 2 2018, 6:49 AM · Frameworks

Sep 30 2018

jtamate committed R37:1b50235ce3c2: Add a + button and the clear button to Add New Tag (authored by jtamate).
Add a + button and the clear button to Add New Tag
Sep 30 2018, 7:37 AM
jtamate closed D15811: Add a + button and the clear button to Add New Tag.
Sep 30 2018, 7:37 AM · Krita

Sep 29 2018

jtamate added a comment to D15811: Add a + button and the clear button to Add New Tag.

No email was sent with yesterday update (or I didn't receive any), this comment is just to notify yesterday change.

Sep 29 2018, 7:37 AM · Krita

Sep 28 2018

jtamate updated the diff for D15811: Add a + button and the clear button to Add New Tag.

Removed the tooltip, I think it is counterproductive now.
Changed the button text into + icon.

Sep 28 2018, 9:58 AM · Krita
jtamate requested review of D15811: Add a + button and the clear button to Add New Tag.
Sep 28 2018, 8:29 AM · Krita

Sep 26 2018

jtamate updated the diff for D10742: get rid of the raw KFileItem pointers in KCoreDirListerCache.

I've tried to do an automatic benchmark of the rename case, without success.
Changed code as requested.

Sep 26 2018, 6:46 AM · Frameworks

Sep 25 2018

jtamate committed R293:a931be1d7e3a: Use non deprecated fastInsert in baloo (authored by jtamate).
Use non deprecated fastInsert in baloo
Sep 25 2018, 5:22 PM
jtamate closed D15740: Use non deprecated fastInsert in baloo.
Sep 25 2018, 5:22 PM · Baloo, Frameworks
jtamate committed R293:95af521127c1: Use String to store UDS_USER and UDS_GROUP of String type. (authored by jtamate).
Use String to store UDS_USER and UDS_GROUP of String type.
Sep 25 2018, 5:21 PM
jtamate closed D15635: Use String to store UDS_USER and UDS_GROUP of String type..
Sep 25 2018, 5:21 PM · Baloo, Frameworks
jtamate requested review of D15740: Use non deprecated fastInsert in baloo.
Sep 25 2018, 8:06 AM · Baloo, Frameworks

Sep 21 2018

jtamate updated the diff for D15635: Use String to store UDS_USER and UDS_GROUP of String type..

Copy the methods getUserName and getGroupName from kfile, those are cached, should be faster.
Keep the fastInsert for another revision.

Sep 21 2018, 10:18 AM · Baloo, Frameworks
jtamate accepted D15638: force-finish canberra notifications on close().

Forget what I just wrote. I've seen that I was missing 15 lines in between then in the phabricator diff view. :-(
Looks good to me.

Sep 21 2018, 9:47 AM · Frameworks
jtamate added a comment to D15638: force-finish canberra notifications on close().

What would happen if finishNotification is called twice, In line 161 and then in line 192?
My guess is that finished signal will be called twice with the same notification, and therefore KNotificationManager::notifyPluginFinished will be called twice. I don't know what will happen the second time.
Perhaps a return after line 161 is needed?

Sep 21 2018, 9:43 AM · Frameworks
jtamate updated the diff for D15635: Use String to store UDS_USER and UDS_GROUP of String type..

White at it, use fastInsert.
And this also fixes the search in the home directory :-).
The title in English :-).

Sep 21 2018, 8:12 AM · Baloo, Frameworks
jtamate abandoned D15623: Finish the notification before removing it from the hash.

I didn't realize the notifications are not owned by m_notifications :-(.
Just looking at the backtrace of the crash, it was the only suspicious thing.

Sep 21 2018, 7:51 AM · Frameworks
jtamate requested review of D15635: Use String to store UDS_USER and UDS_GROUP of String type..
Sep 21 2018, 7:43 AM · Baloo, Frameworks

Sep 20 2018

jtamate requested review of D15623: Finish the notification before removing it from the hash.
Sep 20 2018, 2:41 PM · Frameworks

Sep 19 2018

jtamate accepted D15591: Add Open Document thumbnailer.

Ok by side. It installs now, and don't ask for passwords.

Sep 19 2018, 10:05 AM · Frameworks, Dolphin

Sep 18 2018

jtamate requested changes to D15591: Add Open Document thumbnailer.

I'm trying to check if it works with password-protected files, but I can't install it.
The patch is missing opendocumentthumbnail.desktop.

Sep 18 2018, 6:35 PM · Frameworks, Dolphin
jtamate committed R241:2b52b47211a6: Don't try to restore invalid user places (authored by jtamate).
Don't try to restore invalid user places
Sep 18 2018, 5:26 PM
jtamate closed D14072: Don't try to restore invalid user places.
Sep 18 2018, 5:26 PM · Frameworks
jtamate committed R241:6ab78df8c86e: Make it possible to change directory up even with trailing slashes in the url (authored by jtamate).
Make it possible to change directory up even with trailing slashes in the url
Sep 18 2018, 5:24 PM
jtamate closed D13676: Make it possible to change directory up even with trailing slashes in the url .
Sep 18 2018, 5:24 PM · Frameworks
jtamate added a comment to D15591: Add Open Document thumbnailer.

I haven't tested it, therefore I ask: Does this thumbnailer ask for password for password protected files, like in https://bugs.kde.org/show_bug.cgi?id=394284 ?

Sep 18 2018, 5:16 PM · Frameworks, Dolphin
jtamate updated the diff for D14072: Don't try to restore invalid user places.

Continue processing the XML with invalid urls.
And a trivial code deduplication, I couldn't resist.

Sep 18 2018, 7:19 AM · Frameworks

Sep 14 2018

jtamate added a comment to D15068: Bindings: Correct handling of sources containing utf-8.

Another solution (not tested here but used in other projects) could be to use
with open(source, "r", encoding="utf-8") as f:
(or if the file could contain the aberration BOM, you could use "utf-8-sig")

Sep 14 2018, 6:28 AM · Build System, Frameworks

Sep 12 2018

jtamate added a member for Bugsquad: jtamate.
Sep 12 2018, 3:01 PM

Sep 11 2018

jtamate committed R39:3348145dbfbf: Intialize m_lastPosition (authored by jtamate).
Intialize m_lastPosition
Sep 11 2018, 2:47 PM
jtamate closed D15420: Intialize m_lastPosition.
Sep 11 2018, 2:47 PM · Frameworks, Kate
jtamate added a comment to D12016: [ktexteditor] much faster positionFromCursor.

I get the following warning from valgrind because of this change:

Fix in D15420

Sep 11 2018, 11:23 AM · Frameworks, Kate
jtamate requested review of D15420: Intialize m_lastPosition.
Sep 11 2018, 11:22 AM · Frameworks, Kate

Sep 10 2018

jtamate added a comment to D14724: autotests: don't fail if an unrelated window shows up..

It works for me if I use ctest -j6 . but fails for me if I use ctest -j12 . or ctest -j4 .

Sep 10 2018, 8:03 PM · Frameworks
jtamate requested review of D15408: Don't assert deleting the temporary file.
Sep 10 2018, 3:28 PM · Frameworks
jtamate committed R318:9991eb0f5eaf: Don't assign twice the same key to the action New Tab (authored by jtamate).
Don't assign twice the same key to the action New Tab
Sep 10 2018, 2:47 PM
jtamate closed D15371: Don't assign twice the same key to the action New Tab.
Sep 10 2018, 2:47 PM · Dolphin

Sep 9 2018

jtamate requested review of D15371: Don't assign twice the same key to the action New Tab.
Sep 9 2018, 11:25 AM · Dolphin
jtamate updated the diff for D15328: kfilewidget: convert connect syntax.

Use nullptr instead of 0.
Remove the copied/pasted part of the cause of two dissconnects.

Sep 9 2018, 9:08 AM · Frameworks

Sep 7 2018

jtamate updated the diff for D15328: kfilewidget: convert connect syntax.

Changed to 'Anonymous' connects and disconnects.

Sep 7 2018, 12:26 PM · Frameworks
jtamate added inline comments to D15328: kfilewidget: convert connect syntax.
Sep 7 2018, 12:00 PM · Frameworks
jtamate added inline comments to D15328: kfilewidget: convert connect syntax.
Sep 7 2018, 11:27 AM · Frameworks
jtamate updated the diff for D15328: kfilewidget: convert connect syntax.

Changed [&] by [this]in the lambdas.
Added a lambda for KActionCollection::addAction. I didn't knew it was already possible.

Sep 7 2018, 11:06 AM · Frameworks