sars (Kåre Särs)
User

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Sunday

  • Clear sailing ahead.

User Details

User Since
Jun 3 2015, 8:56 AM (194 w, 1 d)
Availability
Available

Recent Activity

Wed, Feb 20

sars accepted D18966: Postpone device settings change if scanning is in progress.

Kåre, could you comment my reply?: https://phabricator.kde.org/D18966#411134

Wed, Feb 20, 12:59 PM

Tue, Feb 12

sars added a comment to D18966: Postpone device settings change if scanning is in progress.

The idea is sane :)

Tue, Feb 12, 6:38 PM
sars accepted D17510: Postponed settings change if scanning is in progress.

Can you also return false in setOptVal() in case the scanning is ongoing? (and the corresponding note in the doxygen comments)

Tue, Feb 12, 7:34 AM

Mon, Feb 11

sars added a comment to D17510: Postponed settings change if scanning is in progress.

The idea is good. Could we have setOptVals() return -1 and setOptVal() false if the scanning is ongoing and the delayed setting of the value implemented in Skanlite?

Mon, Feb 11, 3:36 PM
sars accepted D17955: Skanlite may create missing output directories automtically (with user's permission).

OK this can go in.

Mon, Feb 11, 10:11 AM

Sun, Feb 10

sars added a comment to D17955: Skanlite may create missing output directories automtically (with user's permission).

Just these minor adjustments and we are there.

Sun, Feb 10, 9:45 AM

Wed, Feb 6

sars added inline comments to D18788: Refactor KateViewInternal::mouseDoubleClickEvent(QMouseEvent *e).
Wed, Feb 6, 9:12 PM · Frameworks, Kate

Mon, Feb 4

sars committed R40:809cb487530e: S&R: Use canonical path while searching in files, like Kate's openUrl() (authored by sars).
S&R: Use canonical path while searching in files, like Kate's openUrl()
Mon, Feb 4, 11:03 AM

Tue, Jan 29

sars added a comment to D17955: Skanlite may create missing output directories automtically (with user's permission).

Sorry, busy month... new project at work and got a new computer to configure....

Tue, Jan 29, 5:33 PM

Jan 22 2019

sars added a comment to D17955: Skanlite may create missing output directories automtically (with user's permission).

I think the selection if we save through the PNG 16bit/color or QImage version should be done in Skanlite.cpp not in savePng().

Jan 22 2019, 7:25 AM
sars accepted D18446: Add static version of toQImageSilent().

Please commit!

Jan 22 2019, 6:32 AM

Jan 16 2019

sars added a comment to T10317: Search&Replace plugin improvements.
Jan 16 2019, 9:45 AM · Kate

Jan 15 2019

sars added a comment to T10317: Search&Replace plugin improvements.
  1. Yes the "more option" button is a bit far away. At some point it actually was at the right of all the options. That is a real possibility
Jan 15 2019, 8:31 PM · Kate
sars added a comment to D17955: Skanlite may create missing output directories automtically (with user's permission).

Sorry for the late response. A lot of real life activities....

Jan 15 2019, 8:00 PM

Jan 13 2019

sars added a comment to D18083: Search plugin: Add helpful buttons (mainly) for the Regular Expressions switch.

My personal taste would be to only have a small tool-tip saying: See the "Add.." context menu entry for regular expression help.

Jan 13 2019, 4:44 PM · Kate
sars added a comment to T10317: Search&Replace plugin improvements.
  1. These buttons are different than the other buttons in that they modify the UI of the toolview but not the search. So mixing them is maybe not the best? One could think about moving them to the right side of the other buttons and have some type of separator between...
Jan 13 2019, 11:09 AM · Kate
sars added a comment to D17443: Kate: Keyboard shortcuts F6/Shift+F6 for Next/Previous Match.

I would be OK with the addition of F6 and Shift+F6 to the Search plugin, but I think it would be good to have a plan for all the shortcuts and update all at once....

Jan 13 2019, 10:51 AM · Kate
sars accepted D17308: Make CTags/Projects context menu more accessible.

I'm not entirely sure removing the feature of having the lookup word in the menu is an improvement, but I just noticed that the feature is broken and only works when a word is selected.

Jan 13 2019, 10:38 AM · Kate

Jan 9 2019

sars accepted D17955: Skanlite may create missing output directories automtically (with user's permission).

I'm OK with this repaint() and if you want I'm also OK with having the processEvents() until we move the actual saving to a separate thread.

Jan 9 2019, 7:00 AM

Jan 8 2019

sars added a comment to D18083: Search plugin: Add helpful buttons (mainly) for the Regular Expressions switch.

I really like the idea! :)

Jan 8 2019, 10:44 AM · Kate
sars added a comment to D17955: Skanlite may create missing output directories automtically (with user's permission).

But this all is to just re-paint one time before we freeze for the time it takes to save of the file? Would it be better in the long run to think about moving the saving to a thread?

Jan 8 2019, 10:11 AM

Jan 7 2019

sars added a comment to D17955: Skanlite may create missing output directories automtically (with user's permission).

I'm OK with parent->repaint()

Jan 7 2019, 1:18 PM
sars added a comment to D18001: Display last scanned image in tab of libksane's KSaneWidget.

I have not had the time to review this properly sorry :(

Jan 7 2019, 10:30 AM
sars added a comment to D17955: Skanlite may create missing output directories automtically (with user's permission).

Besides the processEvents() I'm OK with this! :)

Jan 7 2019, 8:25 AM

Jan 4 2019

sars added a comment to D17956: DocumentPrivate: Fix broken doc links in qCWarning.

Adding as a warning message in the view could be another review and needs comments from others first

Jan 4 2019, 7:14 AM · Frameworks, Kate
sars accepted D17956: DocumentPrivate: Fix broken doc links in qCWarning.
Jan 4 2019, 7:10 AM · Frameworks, Kate
sars added a comment to D17955: Skanlite may create missing output directories automtically (with user's permission).

Hmm... this is only done for local paths. If we only support this feature for local paths it would be much simpler to just use QDir::mkpath(), but it would be nice if it worked for remote folders too...

Jan 4 2019, 7:05 AM

Jan 3 2019

sars added a comment to D17241: WIP:Disable highlighting after 512 characters on a line..

@dhaumann OK the limit is too low for Kile that is clear. Visual Studio Code is limiting the highlighting on a line to 10000 characters.

Jan 3 2019, 7:44 AM · Frameworks, Kate

Dec 31 2018

sars added a comment to D17241: WIP:Disable highlighting after 512 characters on a line..

The new limit is now limiting the number of characters highlighted on a line. It is only the rest of the line that is not highlighted. This is IMO nicer as you don't see the non-highlighted part at the beginning of the line. The highlighting makes it a bit slower, so I decreased the limit to 512.

Dec 31 2018, 6:39 PM · Frameworks, Kate
sars updated the test plan for D17241: WIP:Disable highlighting after 512 characters on a line..
Dec 31 2018, 6:26 PM · Frameworks, Kate
sars updated the diff for D17241: WIP:Disable highlighting after 512 characters on a line..

Disable highlighting after [limit] characters on a line in stead of the whole line.

Dec 31 2018, 6:22 PM · Frameworks, Kate

Dec 28 2018

sars committed R40:739a730f528c: S&R: Replace matches in chunks also in files. (authored by sars).
S&R: Replace matches in chunks also in files.
Dec 28 2018, 7:06 PM

Dec 23 2018

sars added a comment to D17729: KateStatusBar: Reformatted by astyle command to follow coding style.

I think this is fine, but let Christoph and Dominik decide if we want style fixing commits.

Dec 23 2018, 8:24 AM · Frameworks, Kate

Dec 20 2018

sars added inline comments to D17693: DocumentPrivate: Treat some chars also as "auto bracket" only when we have a selection.
Dec 20 2018, 8:07 AM · Frameworks, Kate

Dec 19 2018

sars added a comment to D17599: Confirm closing muliple tabs..

@shubham I'm wondering about the original issue that you try to fix with this review request. Why do we need a dialog that warns that there are multiple documents open in Kate when we close it?

Dec 19 2018, 12:11 PM · Kate
sars added a comment to D17599: Confirm closing muliple tabs..

Sorry Christmastime is a bit busy...

Dec 19 2018, 10:59 AM · Kate

Dec 16 2018

sars committed R40:33b3e153575f: BuildView: Search item with needed data when navigate in error list (authored by loh.tar).
BuildView: Search item with needed data when navigate in error list
Dec 16 2018, 5:01 PM
sars closed D17602: BuildView: Search item with needed data when navigate in error list.
Dec 16 2018, 5:01 PM · Kate
sars accepted D17602: BuildView: Search item with needed data when navigate in error list.
Dec 16 2018, 4:58 PM · Kate

Dec 15 2018

sars added a comment to D17602: BuildView: Search item with needed data when navigate in error list.

Changing the next/previous slots to go to the items with line numbers would be a good change.

Dec 15 2018, 8:37 PM · Kate
sars added a comment to D17602: BuildView: Search item with needed data when navigate in error list.

I'm not sure why you have to move the setFocus() call to the beginning of the function... your comment says something about selection...

Dec 15 2018, 8:23 PM · Kate
sars added a comment to D16336: BuildView: Avoid to show ToolView when build is invoked.

I think one magic build name is not the best way to enable auto-hiding.

Dec 15 2018, 7:06 PM · Kate

Dec 10 2018

sars committed R40:66a84c9e0f94: Fix include (authored by sars).
Fix include
Dec 10 2018, 12:04 PM
sars added a comment to D16336: BuildView: Avoid to show ToolView when build is invoked.

Sorry, I was a bit busy when this review came and then I forgot about it :(

Dec 10 2018, 7:00 AM · Kate

Dec 9 2018

sars committed R40:cec90e834e6d: SearchView: Harmonize text in searchPlaceCombo (authored by loh.tar).
SearchView: Harmonize text in searchPlaceCombo
Dec 9 2018, 9:25 AM
sars closed D17418: SearchView: Harmonize text in searchPlaceCombo.
Dec 9 2018, 9:25 AM · Kate

Dec 8 2018

sars added a comment to D17432: WIP: Add Analyzer Tool "Cppcheck (for C)".

Wouldn't it be better to make it possible to configure the current CppCheck in stead of adding a new one?

Dec 8 2018, 9:16 PM · Kate
sars added a comment to D17241: WIP:Disable highlighting after 512 characters on a line..

@zetazeta I'm not sure the highlighting length limit is worth an option as the editing of the document is not effected. I it is just a bit inconvenient that the highlighting disappears...

Dec 8 2018, 8:59 PM · Frameworks, Kate
sars added a comment to D17241: WIP:Disable highlighting after 512 characters on a line..

@dhaumann: You are right, the highlighting of the document still takes place and it is only KateRenderer that stops using the highlighting info for all lines longer than 1024 characters. All shorter lines are highlighted.

Dec 8 2018, 8:53 PM · Frameworks, Kate
sars updated the diff for D17241: WIP:Disable highlighting after 512 characters on a line..
  • Use a bit different config key to force a value reset for the limit.
  • Fix reading the config value.
  • Update the line-highlight disabled info message and position.
Dec 8 2018, 8:24 PM · Frameworks, Kate

Dec 7 2018

sars accepted D17186: Fix Windows CI build issues caused by startPreviewScan slot.

There is not much else we can do than just have a dummy like this, with the current state of the twain wrapper.

Dec 7 2018, 7:54 AM
sars added a comment to D17186: Fix Windows CI build issues caused by startPreviewScan slot.

Sorry! This problem has totally passed me by before this... I'll have a look.

Dec 7 2018, 7:50 AM

Dec 6 2018

sars committed R40:bfdd0006d19f: S&R: Only add maximum 70 characters context before & after match. (authored by sars).
S&R: Only add maximum 70 characters context before & after match.
Dec 6 2018, 3:53 PM
sars added a comment to D17241: WIP:Disable highlighting after 512 characters on a line..

The highlighting limit is now returned in a function in KateRenderer as it is used also in katedocument.cpp for the warning/information message.

Dec 6 2018, 1:55 PM · Frameworks, Kate
sars updated the diff for D17241: WIP:Disable highlighting after 512 characters on a line..

Add a message to inform about why the lines are not highlighted.
Add a note about disabled highlighting to the wrapped lines warning.
Increase the default line length limit to 100 000 characters per line.

Dec 6 2018, 1:44 PM · Frameworks, Kate
sars committed R39:cb63ec1ee30a: Only cal updateView() in visibleRange() when endPos() is invalid. (authored by sars).
Only cal updateView() in visibleRange() when endPos() is invalid.
Dec 6 2018, 1:17 PM
sars closed D17243: Only cal updateView() in visibleRange() when endPos() is invalid..
Dec 6 2018, 1:17 PM · Kate, Frameworks
sars added a comment to D16838: S&R: Disable highlighting while replacing.

Now I committed to master. Should I back-ported it to 18.12?

Dec 6 2018, 1:11 PM · Kate
sars committed R40:638cb985cd9e: S&R: Make one transaction for the whole replace to speed up things (authored by sars).
S&R: Make one transaction for the whole replace to speed up things
Dec 6 2018, 1:09 PM
sars closed D16838: S&R: Disable highlighting while replacing.
Dec 6 2018, 1:09 PM · Kate
sars added a comment to D16838: S&R: Disable highlighting while replacing.

I guess there are no objections to commit this :)

Dec 6 2018, 7:04 AM · Kate
sars updated the test plan for D16838: S&R: Disable highlighting while replacing.
Dec 6 2018, 7:04 AM · Kate
sars added a comment to D17243: Only cal updateView() in visibleRange() when endPos() is invalid..

OK, I have used this patch actively for a week now, and I have not noticed any regression... ;)

Dec 6 2018, 6:45 AM · Kate, Frameworks
sars updated the diff for D17243: Only cal updateView() in visibleRange() when endPos() is invalid..

Remove unneeded temporary

Dec 6 2018, 6:39 AM · Kate, Frameworks

Dec 4 2018

sars retitled D17241: WIP:Disable highlighting after 512 characters on a line. from Disable highlighting for lines longer than 1024 characters. to WIP:Disable highlighting for lines longer than 1024 characters..
Dec 4 2018, 11:30 AM · Frameworks, Kate
sars added a comment to D17241: WIP:Disable highlighting after 512 characters on a line..

I agree, we need to do something about the line length limit (that only wraps the lines at the limit when opening the file).

Dec 4 2018, 11:30 AM · Frameworks, Kate

Dec 3 2018

sars accepted D17339: BuildView: Fix broken load of project plugin targets.

It is maybe not optimal to "hide" a bug fix among a bunch of code cosmetics, but the fix is good :)

Dec 3 2018, 7:47 PM · Kate
sars committed R40:53dd31b494bb: SearchView: Fix missing signal/slot connection to project plugin (authored by loh.tar).
SearchView: Fix missing signal/slot connection to project plugin
Dec 3 2018, 7:12 AM
sars closed D17296: SearchView: Fix missing signal/slot connection to project plugin.
Dec 3 2018, 7:12 AM · Kate
sars committed R40:2667d1b4bb23: BuildView: Improve user information when build fails (authored by loh.tar).
BuildView: Improve user information when build fails
Dec 3 2018, 7:06 AM
sars closed D17297: BuildView: Improve user information when build fails.
Dec 3 2018, 7:06 AM · Kate

Dec 2 2018

sars added a comment to D17308: Make CTags/Projects context menu more accessible.

What if you have Project, CTags and the new Zeal plugins active? You need to differentiate them somehow.

Dec 2 2018, 7:36 PM · Kate
sars added a comment to D17297: BuildView: Improve user information when build fails.

Let's take that one :)

Dec 2 2018, 7:05 PM · Kate
sars accepted D17297: BuildView: Improve user information when build fails.

I think the text could be "Cannot run command: %1\nWork path %2 does not exist."

Dec 2 2018, 3:18 PM · Kate
sars accepted D17296: SearchView: Fix missing signal/slot connection to project plugin.

I think this is a fix to the bug 347311

Dec 2 2018, 2:42 PM · Kate

Dec 1 2018

sars updated the diff for D17241: WIP:Disable highlighting after 512 characters on a line..

(Remove the unrelated change in src/view/kateview.cpp)

Dec 1 2018, 5:41 PM · Frameworks, Kate
sars updated the diff for D17241: WIP:Disable highlighting after 512 characters on a line..

Also disable highlighting in selections for lines that are longe than 1024

Dec 1 2018, 5:23 PM · Frameworks, Kate

Nov 29 2018

sars added a comment to D17243: Only cal updateView() in visibleRange() when endPos() is invalid..

Note: this is especially bad when the "on the fly spellchecking" is enabled as visibleRange() is used to "optimize" and only highlight what is currently visible. It is called multiple times.

Nov 29 2018, 8:44 PM · Kate, Frameworks
sars updated the diff for D16838: S&R: Disable highlighting while replacing.

Use EditingTransaction in stead of just disabling highlighting

Nov 29 2018, 8:38 PM · Kate
sars added a comment to D17241: WIP:Disable highlighting after 512 characters on a line..

If we get it working properly we could make the limit much higher. I tried with 30000 characters and it still worked very good. (except when selecting the line)

Nov 29 2018, 8:29 PM · Frameworks, Kate
sars requested review of D17243: Only cal updateView() in visibleRange() when endPos() is invalid..
Nov 29 2018, 8:26 PM · Kate, Frameworks
sars updated the summary of D17241: WIP:Disable highlighting after 512 characters on a line..
Nov 29 2018, 8:06 PM · Frameworks, Kate
sars requested review of D17241: WIP:Disable highlighting after 512 characters on a line..
Nov 29 2018, 7:56 PM · Frameworks, Kate
sars added a comment to D16838: S&R: Disable highlighting while replacing.

I tried it and it improves two things first the multiple replace is much faster and second you get the whole replace operation in one undo. I'll do an update when I have cleaned up the experiments a bit :)

Nov 29 2018, 3:53 AM · Kate

Nov 12 2018

sars requested review of D16838: S&R: Disable highlighting while replacing.
Nov 12 2018, 3:43 PM · Kate

Nov 11 2018

sars committed R40:e783f8b87b02: S&R: Add match marks only at the end of searching/replacing. (authored by sars).
S&R: Add match marks only at the end of searching/replacing.
Nov 11 2018, 8:32 AM
sars committed R40:342baeb69825: S&R: Add match marks only at the end of searching/replacing. (authored by sars).
S&R: Add match marks only at the end of searching/replacing.
Nov 11 2018, 8:31 AM

Nov 10 2018

sars accepted D16813: D-Bus global hotkey for scanner preview.

The new connections are better because you get an error just for these problems. Explicitly failing is better than silently doing so on run-time.

Nov 10 2018, 10:51 PM
sars committed R40:db6b3ea9fa30: S&R: Only highlight checked matches (authored by sars).
S&R: Only highlight checked matches
Nov 10 2018, 10:12 PM
sars committed R40:9405751e3766: S&R: Only highlight checked matches (authored by sars).
S&R: Only highlight checked matches
Nov 10 2018, 10:06 PM
sars accepted D16795: Export preview slot.

Adding a function is not breaking backwards compatibility. You can still use Skanlite compiled against an older version with this new version.

Nov 10 2018, 7:46 PM

Nov 9 2018

sars accepted D16779: kate-ctags plugin: support KDevelop (WIP).

I'm a bit confused... I thought KDevelop had much more advanced plugins/features for this? What does KDevelop need this old plugin for? Non-C/C++ languages or what?

Nov 9 2018, 10:15 PM · KDevelop, KTextEditor, Kate

Nov 4 2018

sars added a comment to D16668: Remove document preview plugin: current technology harms workflow.

I would like to give a -1 for removing the plugin I think that the feature is really something that has potential to be great!

Nov 4 2018, 8:03 PM · Kate
sars committed R40:8fa283046254: S&R: Add replace hints to the replace combo-box. (authored by sars).
S&R: Add replace hints to the replace combo-box.
Nov 4 2018, 4:59 PM

Nov 3 2018

sars committed R40:6c7f4a5982f5: S&R: Update the result summary also with 0 matches. (authored by sars).
S&R: Update the result summary also with 0 matches.
Nov 3 2018, 9:15 PM
sars committed R40:a65a63b54743: S&R: Rework the replacing code and add \L & \U support (authored by sars).
S&R: Rework the replacing code and add \L & \U support
Nov 3 2018, 9:15 PM
sars committed R40:bc400164adf4: S&R: Add match-endLine & endColumn to enable easier handling of ranges (authored by sars).
S&R: Add match-endLine & endColumn to enable easier handling of ranges
Nov 3 2018, 9:15 PM
sars committed R40:240f663f800d: Remove duplicate highlighting of matches in itemSelected() (authored by sars).
Remove duplicate highlighting of matches in itemSelected()
Nov 3 2018, 9:15 PM
sars added a comment to D16650: Terminal Tool View: enable AutoSync by default.

+1

Nov 3 2018, 9:09 PM · Kate
sars added a comment to D16652: Terminal plugin: add default shortcut F4 for Toggle Focus.

+1

Nov 3 2018, 9:06 PM · Kate