[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-devel
Subject: [PATCH] kdeui krazy spelling fixes
From: Aaron Reichman <aaron.c.reichman () gmail ! com>
Date: 2009-12-04 21:05:20
Message-ID: 5556b6b80912041305o5ce39369od508e08bbb846f5b () mail ! gmail ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
Hello,
I recently read a blog entry on planetkde (
http://www.layt.net/john/blog/odysseus/going_just_a_little_krazy) about
krazy and some of the easier jobs that can be done to help out. I went
through and changed the things krazy had marked as spelling errors in
kdelibs/kdeui. In all but 3 files, the spelling mistakes were in comments
and had no effect on the actual code. The other 3 files I haven't changed
because I wasn't sure if by changing them I would break something. Those
files were windowmanagement/
kwindowsystem_x11.cpp at lines 981 & 982. The word infos is being flagged as
incorrect. In widgets/kruler.h on line 108 the world millimetres is being
flagged. The same word is being flagged in widgets/kruler.cpp on lines 437 &
460. Attached is the patch file that resulted. This is one of the first
times I've created a patch file so if it's done incorrectly, let me know and
I'll redo it the right way.
Hope this is helpful in some way,
Aaron
[Attachment #5 (text/html)]
Hello,<br><br>I recently read a blog entry on planetkde (<a \
href="http://www.layt.net/john/blog/odysseus/going_just_a_little_krazy" \
target="_blank">http://www.layt.net/john/blog/odysseus/going_just_a_little_krazy</a>) \
about krazy and some of the easier jobs that can be done to help out. I went through \
and changed the things krazy had marked as spelling errors in kdelibs/kdeui. In all \
but 3 files, the spelling mistakes were in comments and had no effect on the actual \
code. The other 3 files I haven't changed because I wasn't sure if by \
changing them I would break something. Those files were windowmanagement/<div \
id=":dm" class="ii gt">kwindowsystem_x11.cpp at lines 981 & 982. The word infos \
is being flagged as incorrect. In widgets/kruler.h on line 108 the world millimetres \
is being flagged. The same word is being flagged in widgets/kruler.cpp on lines 437 \
& 460. Attached is the patch file that resulted. This is one of the first
times I've created a patch file so if it's done incorrectly, let me
know and I'll redo it the right way.<br>
<br>Hope this is helpful in some way,<br><font color="#888888">Aaron</font></div>
--000325574fb232af950479ed773f--
["kdeuikrazyspellingfixes.patch" (text/x-diff)]
Index: shortcuts/kglobalaccel.cpp
===================================================================
--- shortcuts/kglobalaccel.cpp (revision 1057602)
+++ shortcuts/kglobalaccel.cpp (working copy)
@@ -545,7 +545,7 @@
//static
void KGlobalAccel::stealShortcutSystemwide(const QKeySequence &seq)
{
- //get the shortcut, remove seq, and set the new shorctut
+ //get the shortcut, remove seq, and set the new shortcut
const QStringList actionId = self()->d->iface.action(seq[0]);
if (actionId.size() < 4) // not a global shortcut
return;
Index: shortcuts/kacceleratormanager.cpp
===================================================================
--- shortcuts/kacceleratormanager.cpp (revision 1057602)
+++ shortcuts/kacceleratormanager.cpp (working copy)
@@ -715,7 +715,7 @@
TODO:
- * The result is always correct, but not neccesarily optimal. Perhaps
+ * The result is always correct, but not necessarily optimal. Perhaps
it would be a good idea to add another algorithm with higher complexity
that gets used when this one fails, i.e. leaves widgets without
accelerators.
@@ -728,7 +728,7 @@
{
KAccelStringList accel_strings = result;
- // initally remove all accelerators
+ // initially remove all accelerators
for (KAccelStringList::Iterator it = result.begin(); it != result.end(); ++it) {
(*it).setAccel(-1);
}
Index: sonnet/dialog.h
===================================================================
--- sonnet/dialog.h (revision 1057602)
+++ sonnet/dialog.h (working copy)
@@ -88,7 +88,7 @@
/**
* Controls whether the spell checking is continued after the replacement of \
a
- * mispelled word has been performed. By default it is continued.
+ * misspelled word has been performed. By default it is continued.
*
* @since 4.4
*/
Index: dialogs/kshortcutseditor.h
===================================================================
--- dialogs/kshortcutseditor.h (revision 1057602)
+++ dialogs/kshortcutseditor.h (working copy)
@@ -207,7 +207,7 @@
public Q_SLOTS:
/**
- * Resize colums to width required
+ * Resize columns to width required
*/
void resizeColumns();
Index: actions/kaction.h
===================================================================
--- actions/kaction.h (revision 1057602)
+++ actions/kaction.h (working copy)
@@ -447,7 +447,7 @@
/**
* Sets the globalShortcutEnabled property to false and sets the global shortcut \
to an
* empty shortcut.
- * This will also wipe out knowlegde about the existence of this action's global \
shorctut + * This will also wipe out knowlegde about the existence of this \
action's global shortcut
* so it will not be considered anymore for shortcut conflict resolution. It \
will also not be
* visible anymore in the shortcuts KControl module.
* This method should not be used unless these effects are explicitly desired.
Index: widgets/ktextbrowser.h
===================================================================
--- widgets/ktextbrowser.h (revision 1057602)
+++ widgets/ktextbrowser.h (working copy)
@@ -83,7 +83,7 @@
* of links being clicked. Do not call this.
*
* If you need to set an initial source url in the text browser, call
- * the QTextBrowser method explicitely, like this:
+ * the QTextBrowser method explicitly, like this:
* <code>myTextBrowser->QTextBrowser::setSource(url)</code>
*/
void setSource( const QUrl& name );
Index: widgets/krichtextwidget.h
===================================================================
--- widgets/krichtextwidget.h (revision 1057602)
+++ widgets/krichtextwidget.h (working copy)
@@ -126,7 +126,7 @@
SupportTextBackgroundColor = 0x80,
/**
- * A combination of all the falgs above.
+ * A combination of all the flags above.
* Includes all actions that change the format of the text.
*/
FullTextFormattingSupport = 0xff,
Index: widgets/ktabbar.cpp
===================================================================
--- widgets/ktabbar.cpp (revision 1057602)
+++ widgets/ktabbar.cpp (working copy)
@@ -72,7 +72,7 @@
d->mActivateDragSwitchTabTimer = new QTimer( this );
d->mActivateDragSwitchTabTimer->setSingleShot( true );
connect( d->mActivateDragSwitchTabTimer, SIGNAL( timeout() ), SLOT( \
activateDragSwitchTab() ) );
- connect( this, SIGNAL(tabCloseRequested(int)), this, SIGNAL(closeRequest(int))); \
// just for backward compatibilty, KDE5 remove + connect( this, \
SIGNAL(tabCloseRequested(int)), this, SIGNAL(closeRequest(int))); // just for \
backward compatibility, KDE5 remove
//connect( this, SIGNAL( layoutChanged() ), SLOT( onLayoutChange() ) );
}
@@ -112,7 +112,7 @@
}
return;
} else if (QTabBar::isMovable() && event->button() == Qt::MidButton) {
- // compatibilty feature for old middle mouse tab moving
+ // compatibility feature for old middle mouse tab moving
event->accept();
QMouseEvent fakedMouseEvent(event->type(), event->pos(), Qt::LeftButton, \
Qt::LeftButton, event->modifiers()); QCoreApplication::sendEvent(this, \
&fakedMouseEvent); @@ -168,7 +168,7 @@
}
}
} else if ( event->button() == Qt::NoButton && event->buttons() == Qt::MidButton \
&& isMovable() ) {
- // compatibilty feature for old middle mouse tab moving
+ // compatibility feature for old middle mouse tab moving
d->mMiddleMouseTabMoveInProgress = true;
event->accept();
QMouseEvent fakedMouseEvent(event->type(), event->pos(), event->button(), \
Qt::LeftButton, event->modifiers()); @@ -209,7 +209,7 @@
case Qt::MidButton:
if (d->mMiddleMouseTabMoveInProgress && QTabBar::isMovable()) {
- // compatibilty feature for old middle mouse tab moving
+ // compatibility feature for old middle mouse tab moving
d->mMiddleMouseTabMoveInProgress = false;
event->accept();
QMouseEvent fakedMouseEvent(event->type(), event->pos(), Qt::LeftButton, \
Qt::LeftButton, event->modifiers());
Index: util/kmodifierkeyinfo.h
===================================================================
--- util/kmodifierkeyinfo.h (revision 1057602)
+++ util/kmodifierkeyinfo.h (working copy)
@@ -39,7 +39,7 @@
* <li><i>latched</i>: the key is temporarily locked but will be unlocked upon
* the next keypress.</li></ul>
*
- * An application can either query the states synchroneously (@see isKeyLatched,
+ * An application can either query the states synchronously (@see isKeyLatched,
* @see isKeyLocked) or connect to KModifierKeyInfo's signals to be notified about
* changes (@see keyLatched, @see keyLocked).
*/
@@ -74,7 +74,7 @@
const QList<Qt::Key> knownKeys() const;
/**
- * Synchroneously check if a key is pressed.
+ * Synchronously check if a key is pressed.
*
* @param key The key to check
* @return true if the key is pressed, false if the key is not pressed or \
unknown. @@ -83,7 +83,7 @@
bool isKeyPressed(Qt::Key key) const;
/**
- * Synchroneously check if a key is latched.
+ * Synchronously check if a key is latched.
*
* @param key The key to check
* @return true if the key is latched, false if the key is not latched or \
unknown. @@ -102,7 +102,7 @@
bool setKeyLatched(Qt::Key key, bool latched);
/**
- * Synchroneously check if a key is locked.
+ * Synchronously check if a key is locked.
*
* @param key The key to check
* @return true if the key is locked, false if the key is not locked or unknown.
@@ -121,7 +121,7 @@
bool setKeyLocked(Qt::Key key, bool locked);
/**
- * Synchroneously check if a mouse button is pressed.
+ * Synchronously check if a mouse button is pressed.
*
* @param button The mouse button to check
* @return true if the mouse button is pressed, false if the mouse button
Index: util/kpixmapsequence.h
===================================================================
--- util/kpixmapsequence.h (revision 1057602)
+++ util/kpixmapsequence.h (working copy)
@@ -65,7 +65,7 @@
* a multiple of the frame width; the same is true for the height. If an invalid
* size is specified the file is considered to be one column of square frames.
*
- * \return The sequence loaded from \a path or an invalid sequence if an error \
occured + * \return The sequence loaded from \a path or an invalid sequence if an \
error occurred
* (file could not be opened or \a frameSize does not match the file's size)
*/
explicit KPixmapSequence(const QPixmap &pixmap, const QSize &frameSize = \
QSize());
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic