From cd20fb3432498b8871401bdd65a143197e2a6538 Mon Sep 17 00:00:00 2001 From: Bjørn Lindeijer Date: Wed, 25 Mar 2009 23:42:57 +0100 Subject: A host of code style fixes Mostly putting & and * in the right place and making some getters const. --- src/gui/button.cpp | 2 +- src/gui/chat.h | 2 +- src/gui/checkbox.cpp | 2 +- src/gui/checkbox.h | 2 +- src/gui/help.cpp | 2 +- src/gui/help.h | 2 +- src/gui/itempopup.cpp | 2 +- src/gui/itempopup.h | 2 +- src/gui/label.cpp | 5 +++-- src/gui/label.h | 4 ++-- src/gui/linkhandler.h | 2 +- src/gui/palette.h | 10 +++++----- src/gui/passwordfield.cpp | 2 +- src/gui/passwordfield.h | 2 +- src/gui/popup.cpp | 13 ++++++------- src/gui/popup.h | 26 +++++++++++++------------- src/gui/popupmenu.cpp | 2 +- src/gui/popupmenu.h | 2 +- src/gui/radiobutton.cpp | 2 +- src/gui/radiobutton.h | 2 +- src/gui/sell.cpp | 2 +- src/gui/setup_colors.cpp | 2 +- src/gui/shop.cpp | 6 +++--- src/gui/skin.cpp | 10 +++++----- src/gui/textdialog.cpp | 2 +- src/gui/textdialog.h | 2 +- src/gui/textfield.cpp | 2 +- src/gui/textfield.h | 5 +++-- src/gui/textrenderer.h | 12 ++++++++---- src/gui/truetypefont.h | 2 +- src/gui/widgets/textpreview.cpp | 8 ++++---- src/gui/widgets/textpreview.h | 20 ++++++++++---------- src/gui/window.cpp | 2 +- 33 files changed, 84 insertions(+), 79 deletions(-) (limited to 'src/gui') diff --git a/src/gui/button.cpp b/src/gui/button.cpp index f9e5e9dc..28b7c0d7 100644 --- a/src/gui/button.cpp +++ b/src/gui/button.cpp @@ -66,7 +66,7 @@ Button::Button(): init(); } -Button::Button(const std::string& caption, const std::string &actionEventId, +Button::Button(const std::string &caption, const std::string &actionEventId, gcn::ActionListener *listener): gcn::Button(caption), mIsLogged(false) diff --git a/src/gui/chat.h b/src/gui/chat.h index 3c553c67..2bbf32ae 100644 --- a/src/gui/chat.h +++ b/src/gui/chat.h @@ -121,7 +121,7 @@ class ChatWindow : public Window, /** * Gets the focused tab's name */ - const std::string& getFocused() const; + const std::string &getFocused() const; /** * Clear the tab with the given name diff --git a/src/gui/checkbox.cpp b/src/gui/checkbox.cpp index 5695a23f..2623e2c6 100644 --- a/src/gui/checkbox.cpp +++ b/src/gui/checkbox.cpp @@ -35,7 +35,7 @@ Image *CheckBox::checkBoxChecked; Image *CheckBox::checkBoxDisabled; Image *CheckBox::checkBoxDisabledChecked; -CheckBox::CheckBox(const std::string& caption, bool selected): +CheckBox::CheckBox(const std::string &caption, bool selected): gcn::CheckBox(caption, selected) { if (instances == 0) diff --git a/src/gui/checkbox.h b/src/gui/checkbox.h index dd59493c..303782b0 100644 --- a/src/gui/checkbox.h +++ b/src/gui/checkbox.h @@ -37,7 +37,7 @@ class CheckBox : public gcn::CheckBox /** * Constructor. */ - CheckBox(const std::string& caption, bool selected = false); + CheckBox(const std::string &caption, bool selected = false); /** * Destructor. diff --git a/src/gui/help.cpp b/src/gui/help.cpp index 03dfd08d..4b4ba983 100644 --- a/src/gui/help.cpp +++ b/src/gui/help.cpp @@ -70,7 +70,7 @@ void HelpWindow::action(const gcn::ActionEvent &event) } } -void HelpWindow::handleLink(const std::string& link) +void HelpWindow::handleLink(const std::string &link) { std::string helpFile = link; loadHelp(helpFile); diff --git a/src/gui/help.h b/src/gui/help.h index 98e3aa67..8764e733 100644 --- a/src/gui/help.h +++ b/src/gui/help.h @@ -49,7 +49,7 @@ class HelpWindow : public Window, public LinkHandler, /** * Handles link action. */ - void handleLink(const std::string& link); + void handleLink(const std::string &link); /** * Loads help in the dialog. diff --git a/src/gui/itempopup.cpp b/src/gui/itempopup.cpp index 1b0a2bb2..8023f0c2 100644 --- a/src/gui/itempopup.cpp +++ b/src/gui/itempopup.cpp @@ -173,7 +173,7 @@ void ItemPopup::updateColors() graphics->setColor(guiPalette->getColor(Palette::TEXT)); } -gcn::Color ItemPopup::getColor(const std::string& type) +gcn::Color ItemPopup::getColor(const std::string &type) { gcn::Color color; diff --git a/src/gui/itempopup.h b/src/gui/itempopup.h index 03e79886..0e386ef7 100644 --- a/src/gui/itempopup.h +++ b/src/gui/itempopup.h @@ -77,7 +77,7 @@ class ItemPopup : public Popup ScrollArea *mItemEffectScroll; ScrollArea *mItemWeightScroll; - gcn::Color getColor(const std::string& type); + static gcn::Color getColor(const std::string &type); }; #endif // ITEMPOPUP_H diff --git a/src/gui/label.cpp b/src/gui/label.cpp index 68195659..d2fba12b 100644 --- a/src/gui/label.cpp +++ b/src/gui/label.cpp @@ -18,6 +18,7 @@ * along with this program; if not, write to the Free Software * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA */ + #include "label.h" #include "palette.h" @@ -26,12 +27,12 @@ Label::Label() : { } -Label::Label(const std::string& caption) : +Label::Label(const std::string &caption) : gcn::Label(caption) { } -void Label::draw(gcn::Graphics* graphics) +void Label::draw(gcn::Graphics *graphics) { setForegroundColor(guiPalette->getColor(Palette::TEXT)); gcn::Label::draw(static_cast(graphics)); diff --git a/src/gui/label.h b/src/gui/label.h index 026ec081..dcda8e9d 100644 --- a/src/gui/label.h +++ b/src/gui/label.h @@ -42,12 +42,12 @@ class Label : public gcn::Label * Constructor. This version of the constructor sets the label with an * inintialization string. */ - Label(const std::string& caption); + Label(const std::string &caption); /** * Draws the label. */ - void draw(gcn::Graphics* graphics); + void draw(gcn::Graphics *graphics); }; #endif diff --git a/src/gui/linkhandler.h b/src/gui/linkhandler.h index fc9da6da..30267f87 100644 --- a/src/gui/linkhandler.h +++ b/src/gui/linkhandler.h @@ -35,7 +35,7 @@ class LinkHandler public: virtual ~LinkHandler() { } - virtual void handleLink(const std::string& link) = 0; + virtual void handleLink(const std::string &link) = 0; }; #endif diff --git a/src/gui/palette.h b/src/gui/palette.h index 1a466ed4..fab8cd09 100644 --- a/src/gui/palette.h +++ b/src/gui/palette.h @@ -122,7 +122,7 @@ class Palette : public gcn::ListModel * * @return the requested color or Palette::BLACK */ - const gcn::Color& getColor(char c, bool &valid); + const gcn::Color &getColor(char c, bool &valid); /** * Gets the color associated with the type. Sets the alpha channel @@ -133,7 +133,7 @@ class Palette : public gcn::ListModel * * @return the requested color */ - inline const gcn::Color& getColor(ColorType type, int alpha = 255) + inline const gcn::Color &getColor(ColorType type, int alpha = 255) { gcn::Color* col = &mColVector[type].color; col->a = alpha; @@ -147,7 +147,7 @@ class Palette : public gcn::ListModel * * @return the requested committed color */ - inline const gcn::Color& getCommittedColor(ColorType type) + inline const gcn::Color &getCommittedColor(ColorType type) { return mColVector[type].committedColor; } @@ -159,7 +159,7 @@ class Palette : public gcn::ListModel * * @return the requested test color */ - inline const gcn::Color& getTestColor(ColorType type) + inline const gcn::Color &getTestColor(ColorType type) { return mColVector[type].testColor; } @@ -342,7 +342,7 @@ class Palette : public gcn::ListModel * * @return the transformed string */ - static std::string getConfigName(const std::string& typeName); + static std::string getConfigName(const std::string &typeName); }; extern Palette *guiPalette; diff --git a/src/gui/passwordfield.cpp b/src/gui/passwordfield.cpp index 345ee1c3..fd8ebe22 100644 --- a/src/gui/passwordfield.cpp +++ b/src/gui/passwordfield.cpp @@ -21,7 +21,7 @@ #include "passwordfield.h" -PasswordField::PasswordField(const std::string& text): +PasswordField::PasswordField(const std::string &text): TextField(text) { } diff --git a/src/gui/passwordfield.h b/src/gui/passwordfield.h index 86195bd1..3b0b5dab 100644 --- a/src/gui/passwordfield.h +++ b/src/gui/passwordfield.h @@ -35,7 +35,7 @@ class PasswordField : public TextField /** * Constructor, initializes the password field with the given string. */ - PasswordField(const std::string& text = ""); + PasswordField(const std::string &text = ""); /** * Draws the password field. diff --git a/src/gui/popup.cpp b/src/gui/popup.cpp index b7dd6a58..47c3c43c 100644 --- a/src/gui/popup.cpp +++ b/src/gui/popup.cpp @@ -50,8 +50,8 @@ class PopupConfigListener : public ConfigListener } }; -Popup::Popup(const std::string& name, Window *parent, - const std::string& skin): +Popup::Popup(const std::string &name, Window *parent, + const std::string &skin): mParent(parent), mPopupName(name), mMinWidth(100), @@ -170,22 +170,22 @@ void Popup::setLocationRelativeTo(gcn::Widget *widget) getY() + (wy + (widget->getHeight() - getHeight()) / 2 - y)); } -void Popup::setMinWidth(unsigned int width) +void Popup::setMinWidth(int width) { mMinWidth = width > mSkin->getMinWidth() ? width : mSkin->getMinWidth(); } -void Popup::setMinHeight(unsigned int height) +void Popup::setMinHeight(int height) { mMinHeight = height > mSkin->getMinHeight() ? height : mSkin->getMinHeight(); } -void Popup::setMaxWidth(unsigned int width) +void Popup::setMaxWidth(int width) { mMaxWidth = width; } -void Popup::setMaxHeight(unsigned int height) +void Popup::setMaxHeight(int height) { mMaxHeight = height; } @@ -206,4 +206,3 @@ void Popup::setGuiAlpha() mAlphaChanged = false; } - diff --git a/src/gui/popup.h b/src/gui/popup.h index f2f5f49b..fcecf4d4 100644 --- a/src/gui/popup.h +++ b/src/gui/popup.h @@ -56,7 +56,7 @@ class Popup : public gcn::Container * a Popup will never go below its parent Window. * @param skin The location where the Popup's skin XML can be found. */ - Popup(const std::string& name = "", Window *parent = NULL, + Popup(const std::string &name = "", Window *parent = NULL, const std::string &skin = "graphics/gui/gui.xml"); /** @@ -87,42 +87,42 @@ class Popup : public gcn::Container /** * Sets the minimum width of the popup. */ - void setMinWidth(unsigned int width); + void setMinWidth(int width); /** * Sets the minimum height of the popup. */ - void setMinHeight(unsigned int height); + void setMinHeight(int height); /** * Sets the maximum width of the popup. */ - void setMaxWidth(unsigned int width); + void setMaxWidth(int width); /** * Sets the minimum height of the popup. */ - void setMaxHeight(unsigned int height); + void setMaxHeight(int height); /** * Gets the minimum width of the popup. */ - int getMinWidth() { return mMinWidth; } + int getMinWidth() const { return mMinWidth; } /** * Gets the minimum height of the popup. */ - int getMinHeight() { return mMinHeight; } + int getMinHeight() const { return mMinHeight; } /** * Gets the maximum width of the popup. */ - int getMaxWidth() { return mMaxWidth; } + int getMaxWidth() const { return mMaxWidth; } /** * Gets the minimum height of the popup. */ - int getMaxHeight() { return mMaxHeight; } + int getMaxHeight() const { return mMaxHeight; } /** * Gets the padding of the popup. The padding is the distance between @@ -140,14 +140,14 @@ class Popup : public gcn::Container * @param padding The padding of the popup. * @see getPadding */ - void setPadding(unsigned int padding) { mPadding = padding; } + void setPadding(int padding) { mPadding = padding; } /** * Returns the parent Window. * * @return The parent Window or NULL if there is none. */ - Window* getParentWindow() { return mParent; } + Window *getParentWindow() const { return mParent; } /** * Sets the name of the popup. This is only useful for debug purposes. @@ -157,7 +157,7 @@ class Popup : public gcn::Container /** * Returns the name of the popup. This is only useful for debug purposes. */ - const std::string& getPopupName() { return mPopupName; } + const std::string &getPopupName() const { return mPopupName; } /** * Schedule this popup for deletion. It will be deleted at the start @@ -179,7 +179,7 @@ class Popup : public gcn::Container int mMinHeight; /**< Minimum Popup height */ int mMaxWidth; /**< Maximum Popup width */ int mMaxHeight; /**< Maximum Popup height */ - unsigned int mPadding; /**< Holds the padding of the window. */ + int mPadding; /**< Holds the padding of the window. */ /** * The config listener that listens to changes relevant to all Popups. diff --git a/src/gui/popupmenu.cpp b/src/gui/popupmenu.cpp index 9ff9b23f..c2796b59 100644 --- a/src/gui/popupmenu.cpp +++ b/src/gui/popupmenu.cpp @@ -146,7 +146,7 @@ void PopupMenu::showPopup(int x, int y, FloorItem *floorItem) showPopup(x, y); } -void PopupMenu::handleLink(const std::string& link) +void PopupMenu::handleLink(const std::string &link) { Being *being = beingManager->findBeing(mBeingId); diff --git a/src/gui/popupmenu.h b/src/gui/popupmenu.h index 0a6877d9..d9fb4777 100644 --- a/src/gui/popupmenu.h +++ b/src/gui/popupmenu.h @@ -61,7 +61,7 @@ class PopupMenu : public Window, public LinkHandler /** * Handles link action. */ - void handleLink(const std::string& link); + void handleLink(const std::string &link); private: BrowserBox* mBrowserBox; diff --git a/src/gui/radiobutton.cpp b/src/gui/radiobutton.cpp index 52ceb837..b05e37e6 100644 --- a/src/gui/radiobutton.cpp +++ b/src/gui/radiobutton.cpp @@ -34,7 +34,7 @@ Image *RadioButton::radioChecked; Image *RadioButton::radioDisabled; Image *RadioButton::radioDisabledChecked; -RadioButton::RadioButton(const std::string& caption, const std::string& group, +RadioButton::RadioButton(const std::string &caption, const std::string &group, bool marked): gcn::RadioButton(caption, group, marked) { diff --git a/src/gui/radiobutton.h b/src/gui/radiobutton.h index 3d952b3f..9aec3add 100644 --- a/src/gui/radiobutton.h +++ b/src/gui/radiobutton.h @@ -35,7 +35,7 @@ class RadioButton : public gcn::RadioButton /* * Constructor. */ - RadioButton(const std::string& caption,const std::string& group, + RadioButton(const std::string &caption,const std::string &group, bool marked = false); /** diff --git a/src/gui/sell.cpp b/src/gui/sell.cpp index 22f56195..7039f901 100644 --- a/src/gui/sell.cpp +++ b/src/gui/sell.cpp @@ -205,7 +205,7 @@ void SellDialog::action(const gcn::ActionEvent &event) // Attempt sell MessageOut outMsg(mNetwork); - ShopItem* item = mShopItems->at(selectedItem); + ShopItem *item = mShopItems->at(selectedItem); int sellCount; mPlayerMoney += mAmountItems * mShopItems->at(selectedItem)->getPrice(); diff --git a/src/gui/setup_colors.cpp b/src/gui/setup_colors.cpp index 033ba372..073bbc1a 100644 --- a/src/gui/setup_colors.cpp +++ b/src/gui/setup_colors.cpp @@ -57,7 +57,7 @@ Setup_Colors::Setup_Colors() : mScroll = new ScrollArea(mColorBox); mScroll->setHorizontalScrollPolicy(gcn::ScrollArea::SHOW_NEVER); - mTextPreview = new TextPreview(&rawmsg); + mTextPreview = new TextPreview(rawmsg); mPreview = new BrowserBox(BrowserBox::AUTO_WRAP); mPreview->setOpaque(false); diff --git a/src/gui/shop.cpp b/src/gui/shop.cpp index 3cdc304c..300c91b8 100644 --- a/src/gui/shop.cpp +++ b/src/gui/shop.cpp @@ -51,7 +51,7 @@ void ShopItems::addItem(int id, int amount, int price) #ifdef EATHENA_SUPPORT void ShopItems::addItem(int inventoryIndex, int id, int quantity, int price) { - ShopItem* item = 0; + ShopItem *item = 0; if (mMergeDuplicates) { item = findItem(id); @@ -69,7 +69,7 @@ void ShopItems::addItem(int inventoryIndex, int id, int quantity, int price) } #endif -ShopItem* ShopItems::at(int i) const +ShopItem *ShopItems::at(int i) const { return mShopItems.at(i); } @@ -85,7 +85,7 @@ void ShopItems::clear() mShopItems.clear(); } -ShopItem* ShopItems::findItem(int id) +ShopItem *ShopItems::findItem(int id) { ShopItem *item; diff --git a/src/gui/skin.cpp b/src/gui/skin.cpp index d44c54a8..27a0fa3d 100644 --- a/src/gui/skin.cpp +++ b/src/gui/skin.cpp @@ -29,9 +29,9 @@ #include "../utils/dtor.h" #include "../utils/xml.h" -SkinLoader* skinLoader = NULL; +SkinLoader *skinLoader = NULL; -Skin::Skin(ImageRect skin, Image* close, std::string name): +Skin::Skin(ImageRect skin, Image *close, std::string name): instances(0), mName(name), border(skin), @@ -63,7 +63,7 @@ unsigned int Skin::getMinHeight() border.grid[6]->getHeight(); } -Skin* SkinLoader::load(const std::string &filename) +Skin *SkinLoader::load(const std::string &filename) { SkinIterator skinIterator = mSkins.find(filename); @@ -172,9 +172,9 @@ Skin* SkinLoader::load(const std::string &filename) logger->log("Finished loading Skin."); // Hard-coded for now until we update the above code to look for window buttons. - Image* closeImage = resman->getImage("graphics/gui/close_button.png"); + Image *closeImage = resman->getImage("graphics/gui/close_button.png"); - Skin* skin = new Skin(border, closeImage); + Skin *skin = new Skin(border, closeImage); mSkins[filename] = skin; return skin; diff --git a/src/gui/textdialog.cpp b/src/gui/textdialog.cpp index 05e43906..68a6f502 100644 --- a/src/gui/textdialog.cpp +++ b/src/gui/textdialog.cpp @@ -80,7 +80,7 @@ void TextDialog::action(const gcn::ActionEvent &event) } } -const std::string& TextDialog::getText() const +const std::string &TextDialog::getText() const { return textField->getText(); } diff --git a/src/gui/textdialog.h b/src/gui/textdialog.h index 8b4e2cc3..5583e189 100644 --- a/src/gui/textdialog.h +++ b/src/gui/textdialog.h @@ -51,7 +51,7 @@ public: /** * Get the text in the textfield */ - const std::string& getText() const; + const std::string &getText() const; /** * Set the OK button action id diff --git a/src/gui/textfield.cpp b/src/gui/textfield.cpp index 5c6e4f49..42279fc4 100644 --- a/src/gui/textfield.cpp +++ b/src/gui/textfield.cpp @@ -39,7 +39,7 @@ int TextField::instances = 0; float TextField::mAlpha = config.getValue("guialpha", 0.8); ImageRect TextField::skin; -TextField::TextField(const std::string& text): +TextField::TextField(const std::string &text): gcn::TextField(text), mNumeric(false), mListener(0) diff --git a/src/gui/textfield.h b/src/gui/textfield.h index 73824615..070d86ae 100644 --- a/src/gui/textfield.h +++ b/src/gui/textfield.h @@ -38,12 +38,13 @@ class TextFieldListener * * \ingroup GUI */ -class TextField : public gcn::TextField { +class TextField : public gcn::TextField +{ public: /** * Constructor, initializes the text field with the given string. */ - TextField(const std::string& text = ""); + TextField(const std::string &text = ""); /** * Destructor. diff --git a/src/gui/textrenderer.h b/src/gui/textrenderer.h index b69e72a7..712c1312 100644 --- a/src/gui/textrenderer.h +++ b/src/gui/textrenderer.h @@ -35,10 +35,14 @@ class TextRenderer /** * Renders a specified text. */ - static inline void renderText(gcn::Graphics *graphics, const std::string& - text, int x, int y, gcn::Graphics::Alignment align, - const gcn::Color color, gcn::Font *font, bool outline = false, - bool shadow = false, int alpha = 255) + static inline void renderText(gcn::Graphics *graphics, + const std::string &text, + int x, int y, + gcn::Graphics::Alignment align, + const gcn::Color &color, + gcn::Font *font, + bool outline = false, + bool shadow = false, int alpha = 255) { graphics->setFont(font); diff --git a/src/gui/truetypefont.h b/src/gui/truetypefont.h index 1cf6c2c8..cbe64368 100644 --- a/src/gui/truetypefont.h +++ b/src/gui/truetypefont.h @@ -55,7 +55,7 @@ class TrueTypeFont : public gcn::Font */ ~TrueTypeFont(); - virtual int getWidth(const std::string& text) const; + virtual int getWidth(const std::string &text) const; virtual int getHeight() const; diff --git a/src/gui/widgets/textpreview.cpp b/src/gui/widgets/textpreview.cpp index 01790a67..7d4fbd80 100644 --- a/src/gui/widgets/textpreview.cpp +++ b/src/gui/widgets/textpreview.cpp @@ -32,9 +32,9 @@ float TextPreview::mAlpha = config.getValue("guialpha", 0.8); -TextPreview::TextPreview(const std::string* text) +TextPreview::TextPreview(const std::string &text): + mText(text) { - mText = text; mTextAlpha = false; mFont = gui->getFont(); mTextColor = &guiPalette->getColor(Palette::TEXT); @@ -65,7 +65,7 @@ void TextPreview::draw(gcn::Graphics* graphics) if (mTextBGColor && typeid(*mFont) == typeid(TrueTypeFont)) { TrueTypeFont *font = static_cast(mFont); - int x = font->getWidth(*mText) + 1 + 2 * ((mOutline || mShadow) ? 1 :0); + int x = font->getWidth(mText) + 1 + 2 * ((mOutline || mShadow) ? 1 :0); int y = font->getHeight() + 1 + 2 * ((mOutline || mShadow) ? 1 : 0); graphics->setColor(gcn::Color((int) mTextBGColor->r, (int) mTextBGColor->g, @@ -74,7 +74,7 @@ void TextPreview::draw(gcn::Graphics* graphics) graphics->fillRectangle(gcn::Rectangle(1, 1, x, y)); } - TextRenderer::renderText(graphics, *mText, 2, 2, gcn::Graphics::LEFT, + TextRenderer::renderText(graphics, mText, 2, 2, gcn::Graphics::LEFT, gcn::Color(mTextColor->r, mTextColor->g, mTextColor->b, alpha), mFont, mOutline, mShadow, alpha); diff --git a/src/gui/widgets/textpreview.h b/src/gui/widgets/textpreview.h index e7b7db80..0ca343bf 100644 --- a/src/gui/widgets/textpreview.h +++ b/src/gui/widgets/textpreview.h @@ -32,14 +32,14 @@ class TextPreview : public gcn::Widget { public: - TextPreview(const std::string* text); + TextPreview(const std::string &text); /** * Sets the color the text is printed in. * * @param color the color to set */ - inline void setTextColor(const gcn::Color* color) + inline void setTextColor(const gcn::Color *color) { mTextColor = color; } @@ -49,7 +49,7 @@ class TextPreview : public gcn::Widget * * @param alpha whether to use alpha values for the text or not */ - inline void useTextAlpha(bool alpha) + inline void useTextAlpha(bool alpha) { mTextAlpha = alpha; } @@ -60,7 +60,7 @@ class TextPreview : public gcn::Widget * * @param color the color to set */ - inline void setTextBGColor(const gcn::Color* color) + inline void setTextBGColor(const gcn::Color *color) { mTextBGColor = color; } @@ -70,7 +70,7 @@ class TextPreview : public gcn::Widget * * @param color the color to set */ - inline void setBGColor(const gcn::Color* color) + inline void setBGColor(const gcn::Color *color) { mBGColor = color; } @@ -124,14 +124,14 @@ class TextPreview : public gcn::Widget * Gets opacity for this widget (whether or not the background color * is shown below the widget) */ - bool isOpaque() { return mOpaque; } + bool isOpaque() const { return mOpaque; } private: gcn::Font *mFont; - const std::string* mText; - const gcn::Color* mTextColor; - const gcn::Color* mBGColor; - const gcn::Color* mTextBGColor; + std::string mText; + const gcn::Color *mTextColor; + const gcn::Color *mBGColor; + const gcn::Color *mTextBGColor; static float mAlpha; bool mTextAlpha; bool mOpaque; diff --git a/src/gui/window.cpp b/src/gui/window.cpp index 4689c86a..83354a07 100644 --- a/src/gui/window.cpp +++ b/src/gui/window.cpp @@ -53,7 +53,7 @@ class WindowConfigListener : public ConfigListener } }; -Window::Window(const std::string& caption, bool modal, Window *parent, const std::string& skin): +Window::Window(const std::string &caption, bool modal, Window *parent, const std::string &skin): gcn::Window(caption), mGrip(0), mParent(parent), -- cgit v1.2.3-70-g09d2