Differential D8708 Diff 43516 src/view/kateannotationitemdelegate.h

Changeset View

Standalone View

View Options

src/view/kateannotationitemdelegate.h

  • This file was added.
1​/* This file is part of the KDE libraries
2​ Copyright (C) 2017-18 Friedrich W. H. Kossebau <kossebau@kde.org>
3​
4​ This library is free software; you can redistribute it and/or
5​ modify it under the terms of the GNU Library General Public
dhaumannUnsubmitted
Done

Is this copyright really correct?

dhaumann: Is this copyright really correct?
kossebauAuthorUnsubmitted
Not Done

Indeed for the headerthere is no code copied over, well, besides the API being inspired by QAbstractItemDelegate. Reduced to mine.

kossebau: Indeed for the headerthere is no code copied over, well, besides the API being inspired by…
6​ License as published by the Free Software Foundation; either
7​ version 2 of the License, or (at your option) any later version.
8​
9​ This library is distributed in the hope that it will be useful,
10​ but WITHOUT ANY WARRANTY; without even the implied warranty of
dhaumannUnsubmitted
Done

This is v2 only, I think this should be avoided at all costs... We try to get to v2+... I think you can change this, since this is your work. Even if it's based on others, the work of others is in the other files.

dhaumann: This is v2 only, I think this should be avoided at all costs... We try to get to v2+... I think…
kossebauAuthorUnsubmitted
Not Done

Changed this for the header and also for the source. While the initial code was copied over from kateviewhelpers.cpp (thus the complete license header), comparing it now to the original, I would think it has been that much rewritten to match the delegate API, that it safely can be called a copyrightable product of its own, with no substantial parts of the original left (where not required by the general Qt API).

kossebau: Changed this for the header and also for the source. While the initial code was copied over…
11​ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
12​ Library General Public License for more details.
13​
14​ You should have received a copy of the GNU Library General Public License
15​ along with this library; see the file COPYING.LIB. If not, write to
16​ the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
17​ Boston, MA 02110-1301, USA.
18​*/
19​
20​#ifndef __KATE_ANNOTATIONITEMDELEGATE_H__
dhaumannUnsubmitted
Done

All keywords starting with __ are reserved, I suggest to remove __ at the beginning and at the end.

dhaumann: All keywords starting with __ are reserved, I suggest to remove __ at the beginning and at the…
kossebauAuthorUnsubmitted
Not Done

Had those to be in line with other headers in that dir. Will do a separate patch to change that for the existing headers then.

kossebau: Had those to be in line with other headers in that dir. Will do a separate patch to change that…
21​#define __KATE_ANNOTATIONITEMDELEGATE_H__
22​
23​#include <ktexteditor/abstractannotationitemdelegate.h>
24​
25​namespace KTextEditor {
26​class ViewPrivate;
27​}
28​class KateViewInternal;
29​
30​class KateAnnotationItemDelegate : public KTextEditor::AbstractAnnotationItemDelegate
31​{
32​ Q_OBJECT
33​
34​public:
35​ explicit KateAnnotationItemDelegate(KateViewInternal *internalView, QObject *parent);
36​ ~KateAnnotationItemDelegate() override;
37​
38​public:
39​ void paint(QPainter *painter, const KTextEditor::StyleOptionAnnotationItem &option,
40​ KTextEditor::AnnotationModel *model, int line) const override;
41​ bool helpEvent(QHelpEvent *event, KTextEditor::View *view,
42​ const KTextEditor::StyleOptionAnnotationItem &option,
43​ KTextEditor::AnnotationModel *model, int line) override;
44​ void hideTooltip(KTextEditor::View *view) override;
45​ QSize sizeHint(const KTextEditor::StyleOptionAnnotationItem &option,
46​ KTextEditor::AnnotationModel *model, int line) const override;
47​
48​private:
49​ KateViewInternal *m_internalView;
50​ KTextEditor::ViewPrivate *m_view;
51​
52​ mutable qreal m_maxCharWidth = 0.0;
53​ mutable QFontMetricsF m_cachedDataContentFontMetrics;
54​};
55​
56​#endif