Re: KDE Frameworks 5: Rhythm

2012-01-24 Thread Christoph Cullmann
cases, there is no reason for KUrl::protocol to > exist. > > Less difference between Qt and KDE API for no lasting reason is a > good > thing. Hi, just out of curiosity: Why is Intel blocking this to get into Qt? Greetings Christoph -- -- Christo

Re: KDE Frameworks 5: Rhythm

2012-01-24 Thread Christoph Cullmann
- Original Message - > On Tuesday, 24 de January de 2012 19.26.01, Christoph Cullmann wrote: > > Hi, > > > > just out of curiosity: Why is Intel blocking this to get into Qt? > > The Qt Contribution License Agreement has not been approved > internally yet.

Re: Review Request: Port shutdown dialog to QML

2012-02-06 Thread Christoph Feck
> On Feb. 6, 2012, 9:38 p.m., Alexander Neundorf wrote: > > Good from my POV (cmake stuff). UI-wise looks also fine. Was there anything else we needed to do? If not, merge to master. Thanks, you rock! - Christoph --- T

Re: Bug in kcfg, KColorCombo not handled correct

2012-02-09 Thread Christoph Feck
On Thursday 09 February 2012 01:44:07 Ben Cooksley wrote: > On Fri, Nov 4, 2011 at 9:50 AM, Christoph Feck wrote: > > On Thursday 03 November 2011 13:14:31 Dag Andersen wrote: > >> Torsdag 03 november 2011 10:32:43 skrev Christoph Feck: > >> > On Thursday 03 Novem

Re: Review Request: Fix KConfigDialogManager fails to handle subclasses of QComboBox with custom property

2012-02-09 Thread Christoph Feck
paths correctly, the patch fails for cases 2) and 5). It does not find the class name in the map, falls back to user property (what Qt provides now since 4.8), and thus not handle our custom code. - Christoph Feck On Feb. 9, 2012, 11:28 p.m., Albert Astals Cid wrote

Re: Review Request: Fix KConfigDialogManager fails to handle subclasses of QComboBox with custom property

2012-02-12 Thread Christoph Feck
> On Feb. 10, 2012, 12:01 a.m., Christoph Feck wrote: > > Thanks Albert for looking at it. Not sure if I understand everything > > correctly, but what happens, when I have a subclass of Q/KComboBox, that > > does not have its own user property? > > > > I am c

Re: From kdelibs4 to KDE frameworks... how to make KDE more cross platform...

2012-02-13 Thread Christoph Feck
ted in KDE when I told him that there is work in progress to replace the Windows Explorer shell. But I already feared it would be a job needing many more years of development, if it materializes at all. Christoph Feck (kdepepo)

Re: Color Managing KDE

2012-02-22 Thread Christoph Feck
lor-manager" for what is already available. Christoph Feck (kdepepo) KDE Quality Team

Re: Review Request: KImageCache optimization

2012-02-23 Thread Christoph Feck
n? The compression is probably wanted. I have not checked if Qt uses quality 0 or 100 for the fastest mode, but the default PNG compression is utterly slow (as you noticed ;) - Christoph Feck On Feb. 23, 2012, 7:23 p.m., Mark Gaiser

Re: Review Request: KImageCache optimization

2012-02-23 Thread Christoph Feck
> On Feb. 23, 2012, 8:56 p.m., Christoph Feck wrote: > > kdeui/util/kimagecache.cpp, line 106 > > <http://git.reviewboard.kde.org/r/104052/diff/2/?file=50867#file50867line106> > > > > This fails to correctly handle palettized images. > > > >

Re: Review Request: Fix KConfigDialogManager fails to handle subclasses of QComboBox with custom property

2012-02-23 Thread Christoph Feck
/kconfigdialogmanager.cpp <http://git.reviewboard.kde.org/r/103909/#comment8783> if( x ) => if (x) - Christoph Feck On Feb. 21, 2012, 11:55 p.m., Albert Astals Cid wrote: > > --- > This is an automatically generated e-mai

Re: Review Request: kdepasswd KCM: add option to remove the user image

2012-02-24 Thread Christoph Feck
uld be "Remove Image" to be HIG compliant. - Christoph Feck On Feb. 23, 2012, 8:53 p.m., Ralf Jung wrote: > > --- > This is an automatically generated e-mail

Re: Review Request: kcontrol mouse coursor: better performance

2012-02-27 Thread Christoph Feck
QPixmap, so I do not think we should try to be smart here. There general rule is: Use QImage, if you need to access individual pixels, use QPixmap everywhere else. This way we can benefit from any optimizations that can only be applied to opaque datatypes (which QImage isn't). - Christoph

Re: Review Request: Update KWidgetItemDelegate widgets when selection changes

2012-03-01 Thread Christoph Feck
ttp://git.reviewboard.kde.org/r/104134/#comment8944> const ModelIndex &index - Christoph Feck On March 1, 2012, 6:26 p.m., Daniele Elmo Domenichelli wrote: > > --- > This is an automatically generated e-mail. To r

Re: Review Request: kdepasswd KCM: add option to remove the user image

2012-03-07 Thread Christoph Feck
ttp://git.reviewboard.kde.org/r/104054/#comment9014> These two lines should have the same identation. New string => master Thanks :) - Christoph Feck On March 7, 2012, 9:44 p.m., Ralf Jung wrote: > > --- > This is an automatically ge

Re: Review Request: Fix Drkonqi to work with bugzilla 4 (partial port to xml-rpc)

2012-03-09 Thread Christoph Feck
esting. Crash reporting does not work right now, so there is nothing that could break. If possible, this should also be committed to 4.7 branch later, and distributions informed for an eventual online update. - Christoph Feck On March 9, 2012, 12:45 a.m., George Kiagiadakis

Re: Review Request: Fix Drkonqi to work with bugzilla 4 (partial port to xml-rpc)

2012-03-13 Thread Christoph Feck
y, either > move kxmlrpcclient to kdelibs or make kde-runtime depend on kdepimlibs. There were compatibility fixes in bugs.kde.org, so do we still need this? I guess using xmlrpc is the more correct way to do it, so we don't break it again later, but maybe this can be postponed to 4.9+

Re: DrKonqi improvement idea

2012-03-13 Thread Christoph Feck
ing this in a way that is > only a little related to bugzilla and get free from the thinking > imposed by bugzilla. Bugzilla has a long tradition of being a tool that annoys users, developers, as well as bug triagers. Anything that improves the situation is welcomed :) Christoph Feck (kdepepo) KDE Quality Team

Re: Review Request: Remember current desktop when changing activity

2012-03-15 Thread Christoph Feck
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104261/#review11447 --- Could you please add group "Plasma" to reviewers? -

Re: Review Request: Pass decorationSize to QStyleOptionView in KWidgetItemDelegatePrivate

2012-03-17 Thread Christoph Feck
s makes me believe there should be a helper function, even if inline, that does all the option setup? - Christoph Feck On March 18, 2012, 1:44 a.m., Daniele Elmo Domenichelli wrote: > > --- > This is an automatically generated e-ma

Re: Review Request: Pass decorationSize to QStyleOptionView in KWidgetItemDelegatePrivate

2012-03-29 Thread Christoph Feck
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104322/#review11973 --- Ship it! Ship It! - Christoph Feck On March 28, 2012, 8:32

Re: Review Request: Better stars rating in dolphin

2012-04-04 Thread Christoph Feck
(see surrounding code). x=y; => x = y; x+3 => x + 3 etc. - Christoph Feck On April 4, 2012, 5:10 a.m., Anant Kamath wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboar

Re: Where do you need help?

2010-11-01 Thread Christoph Feck
On Saturday 30 October 2010 19:56:36 Camila Ayres wrote: > *What project is in need of some love and dedication?* Check https://bugs.kde.org/weekly-bug-summary.cgi (In other words, helping hands are needed everywhere...) Christoph Feck (kdepepo)

Re: Changing my mind: reverting my menubar, toolbars and statusbar changes

2010-11-06 Thread Christoph Feck
ot; when it is > toggled. This is clearly wrong. Exactly. Left-over from the days where KToggleAction did not force a check box. David Faure is the maintainer of XMLGUI stuff and actually was in favor of those API changes, so wait for his decision. Christoph Feck (kdepepo)

Re: Changing my mind: reverting my menubar, toolbars and statusbar changes

2010-11-08 Thread Christoph Feck
ck box, either checked or unchecked. If the difference isn't clear enough using Oxygen style, I am sure Hugo can improve it. Christoph Feck (kdepepo)

Re: Changing my mind: reverting my menubar, toolbars and statusbar changes

2010-11-09 Thread Christoph Feck
your mind *and* actually > made this whole public experiment. It's certainly an advancement to know > what's not an advancement. :-) > > Have a nice day, > Hans Agreed. I wish a few more developers would be that honest with their "mistakes" instead of being

Re: Changing my mind: reverting my menubar, toolbars and statusbar changes

2010-11-09 Thread Christoph Feck
/hig-book/stable/menus-types.html.en [2] http://developer.apple.com/library/mac/documentation/UserExperience/Conceptual/AppleHIGuidelines/XHIGMenus/XHIGMenus.html Christoph Feck (kdepepo)

Re: Fwd: User bus conclusion

2010-11-09 Thread Christoph Feck
On Wednesday 10 November 2010 00:29:32 Thiago Macieira wrote: > So I have to ask again: do we allow or support multiple logins of the same > user, on the same machine (same $HOME)? With the current mess^Wstate of Multi Screen support we need to. At least I need. Christoph Feck (kdepepo)

Re: kdelibs tests (using valgrind)

2010-11-17 Thread Christoph Feck
rd member is not initialized? The code looks good :) Christoph Feck (kdepepo)

Re: kdelibs tests (using valgrind)

2010-11-17 Thread Christoph Feck
On Wednesday 17 November 2010 21:29:54 Christoph Feck wrote: > On Wednesday 17 November 2010 20:59:13 Jaime wrote: > > Conditional jump or move depends on uninitialised value(s) > > > > at 0x5362167: KLed::setLook(KLed::Look) (kled.cpp:225) > > by 0x40

Re: Review Request: Merge changes from ffmpethumbnail projekt into the kde version

2010-11-22 Thread Christoph Feck
n the QImage to render the filmstrip holes at different sizes? - Christoph On 2010-11-22 20:16:46, Andreas Scherf wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://svn.reviewboa

kdereview? libkmap, libkface

2010-11-28 Thread Christoph Feck
Hi, kdereview contains two new modules, but I did not see the announcement here. libkmap probably needs a changed/clarified name, I suggest kmap -> kgeomap. I first thought it was some replacement for QMap ;) Not sure about kface. Christoph Feck (kdepepo)

Re: Review Request: Merge changes from ffmpethumbnail projekt into the kde version

2010-12-09 Thread Christoph Feck
rip.h file. The upstream project has to use it, because it doesn't have a way to paint on a QImage. But if you want to work on it later, please commit this for now, too small holes don't look nice on HD video files :) - Christoph On 2010-11-22 20:16:46, Andre

Re: Large mouse cursors - Wish/Bug No. 90444

2010-12-21 Thread Christoph Feck
ize [2] 3) find out how the setting is/was passed to kapplymousetheme 4) add an option to the UI to allow changing the size Fredrik Höglund wrote the Xcursor theme support, so he is probably the right person to ask if you got stuck. Patches go to reviewboard, please :P Christoph Feck (kdep

Re: patches too big for kdelibs 4.8?

2012-04-25 Thread Christoph Feck
that I am very happy to see that you are improving Konqueror. Be assured that there are still users who did not ditch it yet ;) Thanks! Christoph Feck (kdepepo) KDE Quality Team

Re: Review request: AppMenu support for KDE

2012-04-27 Thread Christoph Feck
It can be tested using an adapted version of the plasmoid menubar: > https://code.launchpad.net/~megabigbug/plasma-widget-menubar/appmen > u-kde Christoph Feck (kdepepo) KDE Quality Team

Re: Review Request: Fix capacity text in places panel (and possibly unwanted spin-up behavior)

2012-04-30 Thread Christoph Feck
e spin-up behavior, because that seems to be caused by Solid's icon determination code, according to comments in mentioned bug reports. - Christoph --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kd

Re: QtScript considered dangerous

2012-05-25 Thread Christoph Cullmann
is anyway "deprecated" in favour of the V8 based new variant? But does the new engine provide the same nice features to easily wrap your objects and expose a custom API? That was the major reason we switched over from KJS to QtScript. Greetings Christoph -- -

Re: Review Request: Fix for SSL handeshake failure in TcpSlaveBase due to server not supporting SSL compression

2012-05-30 Thread Christoph Feck
/kdelibs/kio/kio/tcpslavebase.cpp: In member function 'int KIO::TCPSlaveBase::connectToHost(const QString&, quint16, QString*)': /local/git/KDE/libs/kdelibs/kio/kio/tcpslavebase.cpp:460:1: error: control reaches end of non-void function [-Werror=return-type] - Christoph Feck On May

Re: Using userbase for manuals

2012-07-01 Thread Christoph Cullmann
ts own and I doubt we would have the same quality if arbitrary users just write up their knowledge. I think an additional standardized point of contact in the wiki would be great, just like hitting "Help..." hit "Support Base..." or what ever and get there the community info

Re: About Writing Documentation in KDE

2012-07-02 Thread Christoph Cullmann
eems like the "The > > > one that does the work decides" is being ingored. > > > > This is not in any way dissing the work of those that put in much > > effort but - > > > > If the current system is so good, can someo

Re: more information in backtraces

2012-07-05 Thread Christoph Feck
egisters, and you cannot get them in the backtrace. > > In gdb itslef, there are addons that already provide some pretty > printing through KDevelop: > https://projects.kde.org/projects/extragear/kdevelop/kdevelop/repos > itory/revisions/master/show/debuggers/gdb/printers (file kde4.py) > Maybe this can be reused? > > Thanks, > Dominik Christoph Feck (kdepepo) KDE Quality Team

Re: Review Request: Fix a memory leak in KDateTable

2012-07-05 Thread Christoph Feck
e the structure directly in the hash? - Christoph Feck On July 5, 2012, 12:23 p.m., Dan Vratil wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard

Re: Review Request: Fix a memory leak in KDateTable

2012-07-05 Thread Christoph Feck
ttp://git.reviewboard.kde.org/r/105451/#comment12061> Actually, I wonder how that would have worked before. If the date was not in hash, it would have inserted and returned a null pointer. Probably a Qt 3 porting relict, looks better now. - Christoph Feck On July 5, 2012, 6:07 p.m., Dan Vratil

Re: Review Request: Fix a memory leak in KDateTable

2012-07-05 Thread Christoph Feck
> On July 5, 2012, 7:53 p.m., Christoph Feck wrote: > > ... to 4.9 branch, not master. - Christoph --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105451/#re

Re: Review Request: KHBox cleanup in frameworks

2012-07-12 Thread Christoph Feck
ttp://git.reviewboard.kde.org/r/105531/#comment12240> Do not set fixed spacing, let the style figure it out. - Christoph Feck On July 12, 2012, 10:18 p.m., Albert Astals Cid wrote: > > --- > This is an automatically generated e-mail.

Review Request: Do not crash avahi DNSSD service

2012-07-18 Thread Christoph Feck
it works, so I did not test it. Thanks, Christoph Feck

Review Request: Account for markup in KSqueezedLabel selection

2012-07-19 Thread Christoph Feck
Diff: http://git.reviewboard.kde.org/r/105620/diff/ Testing --- I tested with the steps from the bug report. Thanks, Christoph Feck

Re: Review Request: KWallet Password Prompt Dialog In Your Face

2012-07-20 Thread Christoph Feck
KDE, we should make sure they still act integrated. - Christoph Feck On July 20, 2012, 1:41 p.m., Allen Winter wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request: Make "New File -> Text File" create empty file

2012-08-07 Thread Christoph Feck
cters removed from the template file, and again it creates "Test.doc" when trying to create a file "Test". Note that I did not enter "Test.txt", but only "Test". So unless this issue is fixed (nobody wants to see a file called "NEWS.doc", but only &q

Re: Review Request: Make "New File -> Text File" create empty file

2012-08-09 Thread Christoph Feck
> On Aug. 7, 2012, 11 a.m., Christoph Feck wrote: > > I tried again on today's master with the two characters removed from the > > template file, and again it creates "Test.doc" when trying to create a file > > "Test". > > > > Note tha

Re: C++11 requirement?

2012-08-10 Thread Christoph Cullmann
- [supported by GCC since] 4.3 > > I too would be interested in variadic templates, would this be > acceptable in > a tier 2 KF component (i18n)? Then there is no longer Windows support. MSVC (even the upcoming one) doesn't support that. Greetings Christoph -- -

Review Request: Fix multiple issues in colors KCM

2012-08-17 Thread Christoph Feck
/show_bug.cgi?id=305350 Diffs - kcontrol/colors/colorscm.cpp 465a90c Diff: http://git.reviewboard.kde.org/r/106066/diff/ Testing --- Thanks, Christoph Feck

Re: Review Request: Do not crash avahi DNSSD service

2012-08-22 Thread Christoph Feck
continously getting crash reports for KTorrent. I tried contacting kdnssd maintainer, but got no answer. - Christoph Feck On July 19, 2012, 1:55 a.m., Christoph Feck wrote: > > --- > This is an automatically generated e-mail.

Re: Review Request: print-manager on kdereview

2012-08-23 Thread Christoph Feck
coming days. > and please review the code :D which is right now at > https://projects.kde.org/projects/playground/base/print-manager > but a sysadmin request to move has already been filled Sorry, cannot read sysadmin bugs. Where do you want it to move? Christoph Feck (kdepepo) KDE Quality Team

Re: Review Request: print-manager on kdereview

2012-08-25 Thread Christoph Feck
On Saturday 25 August 2012 04:29:19 Daniel Nicoletti wrote: > 2012/8/23 Christoph Feck : > > On Wednesday 22 August 2012 21:39:11 Daniel Nicoletti wrote: > >> two years ago I started print-manager, at that time > >> I was using Debian which is affected by this bug

Re: Review Request: print-manager on kdereview

2012-08-27 Thread Christoph Feck
ing them out to unmaintained. Replacing a Python-based code with (debuggable) C++ code is worth the change alone. You do not want to see the Python backtraces we get... (and yes, Daniel already asked Jonathan :) Christoph Feck (kdepepo)

Re: Review Request: print-manager on kdereview

2012-08-27 Thread Christoph Feck
On Sunday 26 August 2012 06:34:30 Daniel Nicoletti wrote: > 2012/8/25 Christoph Feck : > > On Saturday 25 August 2012 04:29:19 Daniel Nicoletti wrote: > >> 2012/8/23 Christoph Feck : > >> > On Wednesday 22 August 2012 21:39:11 Daniel Nicoletti wrote: >

KCM Authorization (was: Re: Review Request: print-manager on kdereview)

2012-08-27 Thread Christoph Feck
On Monday 27 August 2012 22:15:18 Daniel Nicoletti wrote: > 2012/8/27 Christoph Feck : > > I am talking about whatever design mistake is responsible for bug > > 242648, which isn't about CUPS at all. Sorry for side-tracking > > this thread. I was just thinking that

Re: Review Request: fix bounding box in EPS kimgio plugin

2012-08-31 Thread Christoph Feck
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106290/#review18338 --- Ship it! Ship It! - Christoph Feck On Aug. 31, 2012, 4:31

Re: Review Request: add kimgio WebP image format plugin

2012-09-01 Thread Christoph Feck
bp.cpp <http://git.reviewboard.kde.org/r/106300/#comment14515> Same here, should handle pure RGB case. kimgio/webp.cpp <http://git.reviewboard.kde.org/r/106300/#comment14516> "header.size() == 12 && ..." Otherwise, it would return true for the file

Re: Review Request: add kimgio WebP image format plugin

2012-09-01 Thread Christoph Feck
> On Sept. 1, 2012, 1:20 p.m., Christoph Feck wrote: > > kimgio/webp.cpp, line 49 > > <http://git.reviewboard.kde.org/r/106300/diff/1/?file=82610#file82610line49> > > > > Needs to check image.isValid() before writing data via scanLine() > > pointers.

Re: Review Request: add kimgio WebP image format plugin

2012-09-02 Thread Christoph Feck
the data stored in the file unmodified, and if an application works faster with premultiplied data (e.g. in case it composes using QPainter), it can opt to convert to premultiplied alpha. Christoph Feck (kdepepo)

Re: How Can I change wallpaper from CLI?

2012-09-05 Thread Christoph Feck
On Wednesday 05 September 2012 20:10:53 Alexander Neundorf wrote: > In KDE3 it was possible to set the wallpaper via dcop in kdesktop. > So we can't do this anymore ? It's still being worked on, see bug 217950 and linked review request.

Review Request: Allow setting a custom font list in KFontComboBox

2012-09-11 Thread Christoph Feck
3cb16 kdeui/fonts/kfontcombobox.cpp 0ded6fb Diff: http://git.reviewboard.kde.org/r/106425/diff/ Testing --- Thanks, Christoph Feck

Re: Review Request: Allow setting a custom font list in KFontComboBox

2012-09-11 Thread Christoph Feck
http://bugs.kde.org/show_bug.cgi?id=306625 Diffs - kdeui/fonts/kfontcombobox.h de3cb16 kdeui/fonts/kfontcombobox.cpp 0ded6fb Diff: http://git.reviewboard.kde.org/r/106425/diff/ Testing --- Thanks, Christoph Feck

Re: How and what does (plasma theme) caching operate? / KSharedDataCache issue?

2012-09-16 Thread Christoph Feck
On Sunday 16 September 2012 19:16:27 Thomas Lübking wrote: > Because any access to the plasma theme cache takes considerable > time here, i just took a short look. > > 1. all plasma theme caches are 82324 kB and mostly consist of > paddings (actually the default one seems to consist of nothing > e

Re: Review Request: In Konqueror's session restore dialog, allow user to chose which sessions to restore

2012-09-19 Thread Christoph Feck
something along the line that you can choose which pages to restore. - Christoph Feck On Sept. 19, 2012, 9:15 p.m., Dawit Alemayehu wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboa

Re: Review Request: Check if the startDir has an schema before discarting it

2012-10-04 Thread Christoph Feck
." in Konqueror, the file dialog starts at "Custom Path" (and lists root folder), without remembering the previously used path. - Christoph Feck On Sept. 17, 2012, 10:39 a.m., Alex Fiestas wrote: > > --- > This is an autom

Re: Review Request: In Konqueror's session restore dialog, allow user to chose which sessions to restore

2012-10-12 Thread Christoph Feck
tch end up being cherry-picked > into 4.9 branch ??!?! It is only supposed to be in master for 4.10 release. See bug 260282 comment #3. - Christoph --- This is an automatically generated e-mail. To reply, visit: http://git.reviewbo

Re: Review Request: KMainWindow: fix position support in --geometry option

2012-10-24 Thread Christoph Feck
about KUniqueApplication (Konsole, etc.) not passing the mentioned arguments to the spawned process. - Christoph Feck On Oct. 24, 2012, 5:56 p.m., Dominik Haumann wrote: > > --- > This is an automatically generated e-mail. To rep

Re: Review Request: KMainWindow: fix position support in --geometry option

2012-10-24 Thread Christoph Feck
> On Oct. 24, 2012, 7:38 p.m., Christoph Feck wrote: > > Are you sure it fixes bug 165355 ? That bug tracks the issue about > > KUniqueApplication (Konsole, etc.) not passing the mentioned arguments to > > the spawned process. To clarify: I don't object to have this

Re: Review Request: Cache icon and accessibility in KFilePlacesItem

2012-10-24 Thread Christoph Feck
Solid, but a patch there should be of similar complexity as this one, and I would rather wait for it instead of cluttering kfile with a workaround. If the Solid fix, however, is not expected before 4.10, then "Ship it!" - Christoph Feck On Oct. 24, 2012, 9:13 p.m., Dan Vr

Re: Review Request: Cache icon and accessibility in KFilePlacesItem

2012-10-24 Thread Christoph Feck
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107030/#review20831 --- Ship it! On a second look, I see no cluttering. - Christoph

Re: Review Request: Check if the startDir has an schema before discarting it

2012-10-25 Thread Christoph Feck
> On Oct. 4, 2012, 4:28 p.m., Christoph Feck wrote: > > Causes a regression. When using "Save Image As..." in Konqueror, the file > > dialog starts at "Custom Path" (and lists root folder), without remembering > > the previously used path. > >

Re: What is the git workflow for kdelibs ?

2012-11-01 Thread Christoph Feck
On Thursday 01 November 2012 12:05:51 David Faure wrote: > as soon as we branch out other modules for 4.10, we'll reopen > master, it will be less confusing to everyone. Good grief :P

Re: Cleaning house: KDE Review

2012-11-01 Thread Christoph Feck
On Thursday 01 November 2012 22:19:03 Lamarque V. Souza wrote: > Em Thursday 01 November 2012, Ben Cooksley escreveu: > > Hi everyone, > > Hi, > > > The following projects which are in KDE Review appear to have > > been there for more than 2 weeks: > > - appmenu kded module (kded-appmenu) >

kdelibs git mess

2012-11-07 Thread Christoph Feck
Hi, my git foo is limited, but if I interpret http://ompldr.org/vZzZxaw correctly, then somehow master got merged into KDE/4.9 branch, meaning, for example, that Alex's commits to depend on cmake 2.8.8 are now in KDE/4.9 branch. I suggest to not to commit to kdelibs, until this is resolved.

Re: kde-workspace/4.9 messed up

2012-11-11 Thread Christoph Feck
--hard" in the 4.9 branch, to get rid of an initial commit I made in that branch. So HEAD doesn't refer to the remote head in the current branch, but always to the head of master? Christoph Feck (kdepepo)

Re: Trojitá, an IMAP e-mail client, and KDE

2012-11-14 Thread Christoph Feck
php You could then also request an entry in the bug tracker, so that bugs can be easily reassigned to KDE components, in case they are "not your bugs". PS: Be prepared to handle 1000+ votes on "Add POP3 support" request ;) Christoph Feck (kdepepo) KDE Quality Team

Re: Review Request: Fix KIO listDir entries batching

2012-12-06 Thread Christoph Feck
Mark Gaiser wrote: > I regret to ask if you could do it... > I'm getting some merge conflicts where i don't quite know what the > intention is.. (deleted files, but modified in head.. and some other merge > conflicts). What does the code do, if the QTime wraps ar

Re: Review Request: Fix bug 234407 - systemsettings iconview wordwrapping cause narrow icon in zhcn locale

2013-01-09 Thread Christoph Feck
... is there a > > better way of determining if a language is CJK? "CJK" is actually naming the languages which use CJK, so the list is hardcoded by definition. - Christoph --- This is an automatically generated e-mail. To repl

Re: Review Request 107409: [KMenuEdit] Ability to sort elements

2013-01-17 Thread Christoph Feck
-devel irc channel, or on an KDE developers mailing list. The diff needs to be uploaded differently. - Christoph Feck On Jan. 17, 2013, 6:07 p.m., Julien Borderie wrote: > > --- > This is an automatically generated e-mail. To rep

Re: Review Request 108592: Added some more operations in analitza

2013-01-27 Thread Christoph Feck
nalitza" is used as a library name, visible to developers only. As such, I see no reason to change it. Christoph Feck (kdepepo) KDE Quality Team

Re: Login for bug reporting

2013-02-06 Thread Christoph Cullmann
, the bug won't help at all ;) Greetings Christoph - Ursprüngliche Mail - > Hi folks, > > at FOSDEM I was approached by a person who asked me to relay his > dissatisfaction with the requirement of having a KDE Bugzilla account > to > report crashes via the KDE crash

Re: Login for bug reporting

2013-02-07 Thread Christoph Cullmann
ean, I have nothing against allowing people to login with their Google account or whatever if that is possible, but I would not like to see bug reports by . Greetings Christoph -- -- Christoph Cullmann - AbsInt Angewandte Informatik GmbH Email

Re: Review Request 108434: [High-dpi issues] Fix icon in KPageDialog that shows up in every KCM being hardcoded to 22px

2013-02-09 Thread Christoph Feck
On a common setup this should result in a 22px icon - the same you use > now (for the configured toolbar size) Regarding 22 vs. 24, the geometric mean of 16 and 32 is 22.63, which got rounded down to 22 to get even sized icons. For the newer (larger) icon sizes, a simple "multiple of

Re: Interesting issue for kde-baseapps, Jenkins & kdesrc-build, proposed solution

2013-03-11 Thread Christoph Cullmann
; modules in > question? (And if so, I'd appreciate ideas for better ways to fix). Hi, I would have no problem with moving kate.git around, splitting is IMHO a pain for later developing on it, as all parts are tightly coupled anyway. Greetings Christoph -- ----

Re: KSaveFile, QSaveFile, QFile

2013-03-24 Thread Christoph Cullmann
itable is anyway problematic, guess users will have to live with "less safe" saving in such settings. Will KSaveFile get some of that patches, too? Greetings Christoph -- - Dr.-Ing. Christoph Cullmann - AbsInt Angewandte Informatik GmbH Email: c

Re: Password strengh meter in KNewPasswordDialog

2013-04-04 Thread Christoph Feck
There > are a few exceptions (qbF\FdHCy, U2WVF9kLH) that still score worse > with the new algorithm. One of them has no digit, the other no > special, so I am not surprised as there are very few transitions > between character classes in them. > > So, yes, you are absolutely right. Suggestions about how to improve > that absolutely welcome. > > Eike Christoph Feck (kdepepo) KDE Quality Team

Re: Review Request 109997: Tint selected icon in KFileItemDelegate by highlight color (as Dolphin 2.0)

2013-04-13 Thread Christoph Feck
Dolphin behavior, so I am not sure we should replicate it in kdelibs. - Christoph Feck On April 13, 2013, 3:12 p.m., Eike Hein wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboa

Re: Review Request 109997: Tint selected icon in KFileItemDelegate by highlight color (as Dolphin 2.0)

2013-04-13 Thread Christoph Feck
> On April 13, 2013, 5 p.m., Christoph Feck wrote: > > Regarding the icon tinting, there are bugs filed against this Dolphin > > behavior, so I am not sure we should replicate it in kdelibs. bug 304643, bug 309722 to be more specific :

Re: Review Request 110043: Proposed fix/workaround for legacy encoded filename handling

2013-04-16 Thread Christoph Feck
instead of 0x18000 codes. - Christoph Feck On April 16, 2013, 2:59 p.m., Róbert Szókovács wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.

Re: Review Request 110057: Fix mem leaks in kunitconversion

2013-04-16 Thread Christoph Feck
++ skills, could you try creating the objects on the stack instead, so they are automatically deleted once out of scope? - Christoph Feck On April 16, 2013, 9:31 p.m., Dario Cambié wrote: > > --- > This is an automatically generated e

Re: Review Request 106300: add kimgio WebP image format plugin

2013-05-21 Thread Christoph Feck
out this change? - Christoph --- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106300/#review32734 --- On May 18, 2013, 12:44 p.m., Mart

Re: Review Request 106300: add kimgio WebP image format plugin

2013-06-15 Thread Christoph Feck
> On May 18, 2013, 12:44 p.m., Commit Hook wrote: > > This review has been submitted with commit > > df845b06bc440fff5a43a9cdc149e8c88c29bc67 by Martin Koller to branch master. > > Christoph Feck wrote: > Just noticed this has been committed to kolourpaint instead o

Re: Review Request 111379: Fix for kio_file crash on non-utf filenames

2013-07-03 Thread Christoph Feck
, though. - Christoph Feck On July 3, 2013, 1:54 p.m., Róbert Szókovács wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.

Re: Review Request 111379: Fix for kio_file crash on non-utf filenames

2013-07-03 Thread Christoph Feck
report. Using Dolphin I could rename, delete, and "Open With" Okteta the file. - Christoph Feck On July 3, 2013, 1:54 p.m., Róbert Szókovács wrote: > > --- > This is an automatically generated e-mail. To r

gdk_x_error (was: Re: [kile] [Bug 321759] Kile crashes when file is opened from menu)

2013-07-05 Thread Christoph Feck
org/buglist.cgi?query_format=advanced&long_desc_type=substring&long_desc=gdk_x_error&bug_status=UNCONFIRMED&bug_status=CONFIRMED&bug_status=ASSIGNED&bug_status=REOPENED&bug_status=NEEDSINFO&bug_severity=crash&list_id=697588 Christoph Feck (kdepepo) KDE Quality Team

Review Request 111626: Fix Amarok crash with Audio CD inserted

2013-07-21 Thread Christoph Feck
kfile/kfileplacesitem.cpp 5a12486 Diff: http://git.reviewboard.kde.org/r/111626/diff/ Testing --- I could not test, my system has no disc drives. Thanks, Christoph Feck

<    1   2   3   4   5   >