M util/fbrun/FbRun.cc → util/fbrun/FbRun.cc

@@ -19,12 +19,13 @@ // LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
 // FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
 // DEALINGS IN THE SOFTWARE.
 
-// $Id: FbRun.cc,v 1.8 2002/11/27 21:56:56 fluxgen Exp $
+// $Id: FbRun.cc,v 1.9 2002/12/05 00:07:39 fluxgen Exp $
 
 #include "FbRun.hh"
 
 #include "App.hh"
 #include "EventManager.hh"
+#include "Color.hh"
 
 #include <X11/Xlib.h>
 #include <X11/keysym.h>
@@ -37,248 +38,208 @@ #include <fstream>
 
 using namespace std;
 FbRun::FbRun(int x, int y, size_t width):
-m_font("fixed"),
-m_win(None),
-m_display(FbTk::App::instance()->display()),
-m_bevel(4),
-m_gc(DefaultGC(m_display, DefaultScreen(m_display))),
-m_end(false),
-m_current_history_item(0) {
-	createWindow(x, y, width + m_bevel, m_font.height());
+    m_font("fixed"),
+    m_win((int)0, x, y,  //screen num and position
+          width + m_bevel, m_font.height(),  // size
+          KeyPressMask|ExposureMask), // eventmask
+    m_display(FbTk::App::instance()->display()),
+    m_bevel(4),
+    m_gc(DefaultGC(m_display, DefaultScreen(m_display))),
+    m_end(false),
+    m_current_history_item(0) {
+    // setting nomaximize in local resize
+    resize(width, m_font.height());
+    FbTk::EventManager::instance()->registerEventHandler(*this, m_win.window());
 }
 
 FbRun::~FbRun() {
-	hide();
-	FbTk::EventManager::instance()->unregisterEventHandler(m_win);
-	XDestroyWindow(m_display, m_win);
+    hide();
+    FbTk::EventManager::instance()->unregisterEventHandler(m_win.window());
 }
 
 void FbRun::run(const std::string &command) {
-	//fork and execute program
-	if (!fork()) {
-		setsid();
-		execl("/bin/sh", "/bin/sh", "-c", command.c_str(), 0);
-		exit(0); //exit fork
-	}
+    //fork and execute program
+    if (!fork()) {
+        setsid();
+        execl("/bin/sh", "/bin/sh", "-c", command.c_str(), 0);
+        exit(0); //exit child
+    }
 
-	hide(); // hide gui
+    hide(); // hide gui
 	
-	// save command history to file
-	if (m_runtext.size() != 0) { // no need to save empty command
-		// open file in append mode
-		ofstream outfile(m_history_file.c_str(), ios::app);
-		if (outfile)
-			outfile<<m_runtext<<endl;
-		else 
-			cerr<<"FbRun Warning: Can't write command history to file: "<<m_history_file<<endl;
-	}
-	FbTk::App::instance()->end(); // end application
-	m_end = true; // mark end of processing
+    // save command history to file
+    if (m_runtext.size() != 0) { // no need to save empty command
+        // open file in append mode
+        ofstream outfile(m_history_file.c_str(), ios::app);
+        if (outfile)
+            outfile<<m_runtext<<endl;
+        else 
+            cerr<<"FbRun Warning: Can't write command history to file: "<<m_history_file<<endl;
+    }
+    FbTk::App::instance()->end(); // end application
+    m_end = true; // mark end of processing
 }
 
 bool FbRun::loadHistory(const char *filename) {
-	if (filename == 0)
-		return false;
-	ifstream infile(filename);
-	if (!infile) {
-		//even though we fail to load file, we should try save to it
-		m_history_file = filename;
-		return false;
-	}
-	// clear old history and load new one from file
-	m_history.clear();
-	// each line is a command
-	string line;
-	while (!infile.eof()) {
-		getline(infile, line);
-		if (line.size()) // don't add empty lines
-			m_history.push_back(line);
-	}
-	// set no current histor to display
-	m_current_history_item = m_history.size();
-	// set history file
-	m_history_file = filename;
-	return true;
+    if (filename == 0)
+        return false;
+    ifstream infile(filename);
+    if (!infile) {
+        //even though we fail to load file, we should try save to it
+        m_history_file = filename;
+        return false;
+    }
+    // clear old history and load new one from file
+    m_history.clear();
+    // each line is a command
+    string line;
+    while (!infile.eof()) {
+        getline(infile, line);
+        if (line.size()) // don't add empty lines
+            m_history.push_back(line);
+    }
+    // set no current histor to display
+    m_current_history_item = m_history.size();
+    // set history file
+    m_history_file = filename;
+    return true;
 }
 
 bool FbRun::loadFont(const string &fontname) {
-	if (!m_font.load(fontname.c_str()))
-		return false;
+    if (!m_font.load(fontname.c_str()))
+        return false;
 
-	// resize to fit new font height
-	resize(m_width, m_font.height() + m_bevel);
-	return true;
+    // resize to fit new font height
+    resize(m_win.width(), m_font.height() + m_bevel);
+    return true;
 }
 
-void FbRun::setForeground(const XColor &color) {
-	XSetForeground(m_display, m_gc, color.pixel);
-	redrawLabel();
+void FbRun::setForeground(const FbTk::Color &color) {
+    XSetForeground(m_display, m_gc, color.pixel());
+    redrawLabel();
 }
 
-void FbRun::setBackground(const XColor &color) {
-	XSetWindowBackground(m_display, m_win, color.pixel);
-	redrawLabel();
+void FbRun::setBackground(const FbTk::Color &color) {
+    m_win.setBackgroundColor(color);
+    redrawLabel();
 }
 
 
 void FbRun::setText(const string &text) {
-	m_runtext = text;
-	redrawLabel();
+    m_runtext = text;
+    redrawLabel();
 }
 
 void FbRun::setTitle(const string &title) {
-	assert(m_win);
-	XStoreName(m_display, m_win, const_cast<char *>(title.c_str()));	
+    m_win.setName(title.c_str());
 }
 
 void FbRun::move(int x, int y) {
-	XMoveWindow(m_display, m_win, x, y);
+    m_win.move(x, y);
 }
 
 void FbRun::resize(size_t width, size_t height) {
-	assert(m_win);
-	XResizeWindow(m_display, m_win, width, height);
-	m_width = width;
-	m_height = height;
-	setNoMaximize();
+    m_win.resize(width, height);	
+    setNoMaximize();
 }
 
 void FbRun::show() {
-	assert(m_win);
-	XMapWindow(m_display, m_win);
+    m_win.show();
 }
 
 void FbRun::hide() {
-	assert(m_win);
-	XUnmapWindow(m_display, m_win);
+    m_win.hide();
 }
 
 void FbRun::redrawLabel() {
-	assert(m_win);
-
-	XClearWindow(m_display, m_win);
-	drawString(m_bevel/2, m_font.ascent() + m_bevel/2,
-		m_runtext.c_str(), m_runtext.size());
+    m_win.clear();
+    drawString(m_bevel/2, m_font.ascent() + m_bevel/2,
+               m_runtext.c_str(), m_runtext.size());
 
 }
 
 void FbRun::drawString(int x, int y,
-	const char *text, size_t len) {
-	assert(m_win);
-	assert(m_gc);
-	// check right boundary
-	// and adjust text drawing
-	size_t text_width = m_font.textWidth(text, len);
-	size_t startpos = 0;
-	if (text_width > m_width) {
-		for (; startpos < len; ++startpos) {
-			if (m_font.textWidth(text+startpos, len-startpos) < m_width)
-				break;
-		}		
-	}
-
-	m_font.drawText(m_win, DefaultScreen(m_display), m_gc, text + startpos, len-startpos, x, y);
-}
-
-
-void FbRun::createWindow(int x, int y, size_t width, size_t height) {
-	m_win = XCreateSimpleWindow(m_display, // display
-		DefaultRootWindow(m_display), // parent windows
-		x, y,
-		width, height,
-		1,  // border_width
-		0,  // border
-		WhitePixel(m_display, DefaultScreen(m_display))); // background
-
-	if (m_win == None)
-		throw string("Failed to create FbRun window!");
-
-	XSelectInput(m_display, m_win, KeyPressMask|ExposureMask);
-	
-	FbTk::EventManager::instance()->registerEventHandler(*this, m_win);
-	
-	setNoMaximize();
+                       const char *text, size_t len) {
+    assert(m_gc);
 
-	m_width = width;
-	m_height = height;
+    // check right boundary and adjust text drawing
+    size_t text_width = m_font.textWidth(text, len);
+    size_t startpos = 0;
+    if (text_width > m_win.width()) {
+        for (; startpos < len; ++startpos) {
+            if (m_font.textWidth(text+startpos, len-startpos) < m_win.width())
+                break;
+        }		
+    }
 
+    m_font.drawText(m_win.window(), DefaultScreen(m_display), m_gc, text + startpos, len-startpos, x, y);
 }
 
 void FbRun::keyPressEvent(XKeyEvent &ke) {
-	KeySym ks;
-	char keychar[1];
-	XLookupString(&ke, keychar, 1, &ks, 0);
-	if (ks == XK_Escape) {
-		m_end = true;
-		hide();
-		return; // no more processing
-	} else if (ks == XK_Return) {
-		run(m_runtext);
-		m_runtext = ""; // clear text
-	} else if (ks == XK_BackSpace) {
-		if (m_runtext.size() != 0) { // we can't erase what we don't have ;)
-			m_runtext.erase(m_runtext.size()-1);
-			redrawLabel();
-		}
-	} else if (! IsModifierKey(ks) && !IsCursorKey(ks)) {
-		m_runtext+=keychar[0]; // append character
-		redrawLabel(); 
-	} else if (IsCursorKey(ks)) {
+    KeySym ks;
+    char keychar[1];
+    XLookupString(&ke, keychar, 1, &ks, 0);
+    if (ks == XK_Escape) {
+        m_end = true;
+        hide();
+        FbTk::App::instance()->end(); // end program
+        return; // no more processing
+    } else if (ks == XK_Return) {
+        run(m_runtext);
+        m_runtext = ""; // clear text
+    } else if (ks == XK_BackSpace) {
+        if (m_runtext.size() != 0) { // we can't erase what we don't have ;)
+            m_runtext.erase(m_runtext.size()-1);
+            redrawLabel();
+        }
+    } else if (! IsModifierKey(ks) && !IsCursorKey(ks)) {
+        m_runtext+=keychar[0]; // append character
+        redrawLabel(); 
+    } else if (IsCursorKey(ks)) {
 		
-		switch (ks) {
-		case XK_Up:
-			prevHistoryItem();
-		break;
-		case XK_Down:
-			nextHistoryItem();
-		break;
-		}
-		redrawLabel();
-	}
+        switch (ks) {
+        case XK_Up:
+            prevHistoryItem();
+            break;
+        case XK_Down:
+            nextHistoryItem();
+            break;
+        }
+        redrawLabel();
+    }
 }
 
 void FbRun::exposeEvent(XExposeEvent &ev) {
-	redrawLabel();
+    redrawLabel();
 }
 
-void FbRun::getSize(size_t &width, size_t &height) {
-	XWindowAttributes attr;
-	XGetWindowAttributes(m_display, m_win, &attr);
-	width = attr.width;
-	height = attr.height;
-}
 
 void FbRun::setNoMaximize() {
-
-	size_t width, height;
-
-	getSize(width, height);
-
-	// we don't need to maximize this window
-	XSizeHints sh;
-	sh.flags = PMaxSize | PMinSize;
-	sh.max_width = width;
-	sh.max_height = height;
-	sh.min_width = width;
-	sh.min_height = height;
-	XSetWMNormalHints(m_display, m_win, &sh);
+    // we don't need to maximize this window
+    XSizeHints sh;
+    sh.flags = PMaxSize | PMinSize;
+    sh.max_width = m_win.width();
+    sh.max_height = m_win.height();
+    sh.min_width = m_win.width();
+    sh.min_height = m_win.height();
+    XSetWMNormalHints(m_display, m_win.window(), &sh);
 }
 
 void FbRun::prevHistoryItem() {
 
-	if (m_current_history_item > 0 && m_history.size() > 0)
-		m_current_history_item--;
-	if (m_current_history_item < m_history.size())
-		m_runtext = m_history[m_current_history_item];
+    if (m_current_history_item > 0 && m_history.size() > 0)
+        m_current_history_item--;
+    if (m_current_history_item < m_history.size())
+        m_runtext = m_history[m_current_history_item];
 }
 
 void FbRun::nextHistoryItem() {
-	m_current_history_item++;
-	if (m_current_history_item >= m_history.size()) {
-		m_current_history_item = m_history.size();
-		m_runtext = "";
-		return;
-	} else 
-		m_runtext = m_history[m_current_history_item];
+    m_current_history_item++;
+    if (m_current_history_item >= m_history.size()) {
+        m_current_history_item = m_history.size();
+        m_runtext = "";
+        return;
+    } else 
+        m_runtext = m_history[m_current_history_item];
 
 }

M util/fbrun/FbRun.hh → util/fbrun/FbRun.hh

@@ -19,79 +19,79 @@ // LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
 // FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
 // DEALINGS IN THE SOFTWARE.
 
-// $Id: FbRun.hh,v 1.7 2002/11/27 21:56:02 fluxgen Exp $
+// $Id: FbRun.hh,v 1.8 2002/12/05 00:07:38 fluxgen Exp $
 
 #ifndef FBRUN_HH
 #define FBRUN_HH
 
 #include "EventHandler.hh"
 #include "Font.hh"
+#include "FbWindow.hh"
 
 #include <string>
 #include <vector>
 
 /**
-	Creates and managed a run window
+   Creates and managed a run window
 */
 class FbRun: public FbTk::EventHandler {
 public:
-	FbRun(int x = 0, int y = 0, size_t width = 200);
-	~FbRun();
-	void handleEvent(XEvent * const ev);
-	void setText(const std::string &text);
-	void setTitle(const std::string &title);
-	void move(int x, int y);
-	void resize(size_t width, size_t height);
-	size_t height() const { return m_height; }
-	size_t width() const { return m_width; }
-	/// hide window
-	void hide();
-	/// show window
-	void show();
-	/// load and reconfigure for new font
-	bool loadFont(const std::string &fontname);
-	void setForeground(const XColor &color);
-	void setBackground(const XColor &color);
-	void setAntialias(bool val) { m_font.setAntialias(val); }
-	const FbTk::Font &font() const { return m_font; }
-	/// execute command and exit
-	void run(const std::string &execstring);
-	/// is this application done?
-	bool end() const { return m_end; }
-	/**
-	  loads history file.
-	  @return true on success, else false
-	*/
-	bool loadHistory(const char *filename);
-	/**
-		@name events
-	*/
-	///@{
-	void exposeEvent(XExposeEvent &ev);
-	void keyPressEvent(XKeyEvent &ev);
-	///@}
+    FbRun(int x = 0, int y = 0, size_t width = 200);
+    ~FbRun();
+    void handleEvent(XEvent * const ev);
+    void setText(const std::string &text);
+    void setTitle(const std::string &title);
+    void move(int x, int y);
+    void resize(size_t width, size_t height);
+    size_t height() const { return m_win.height(); }
+    size_t width() const { return m_win.width(); }
+    /// hide window
+    void hide();
+    /// show window
+    void show();
+    /// load and reconfigure for new font
+    bool loadFont(const std::string &fontname);
+    void setForeground(const FbTk::Color &color);
+    void setBackground(const FbTk::Color &color);
+    void setAntialias(bool val) { m_font.setAntialias(val); }
+    const FbTk::Font &font() const { return m_font; }
+    /// execute command and exit
+    void run(const std::string &execstring);
+    /// is this application done?
+    bool end() const { return m_end; }
+    /**
+       loads history file.
+       @return true on success, else false
+    */
+    bool loadHistory(const char *filename);
+    /**
+       @name events
+    */
+    ///@{
+    void exposeEvent(XExposeEvent &ev);
+    void keyPressEvent(XKeyEvent &ev);
+    ///@}
 	
 private:
-	void nextHistoryItem();
-	void prevHistoryItem();
-	void drawString(int x, int y, const char *text, size_t len);
-	void getSize(size_t &width, size_t &height);
-	void createWindow(int x, int y, size_t width, size_t height);	
-	void redrawLabel();	
-	/// set no maximizable for this window
-	void setNoMaximize();
+    void nextHistoryItem();
+    void prevHistoryItem();
+    void drawString(int x, int y, const char *text, size_t len);
+    void getSize(size_t &width, size_t &height);
+    void createWindow(int x, int y, size_t width, size_t height);	
+    void redrawLabel();	
+    /// set no maximizable for this window
+    void setNoMaximize();
 
-	FbTk::Font m_font; ///< font used to draw command text
-	Window m_win;  ///< toplevel window 
-	Display *m_display;  ///< display connection
-	std::string m_runtext; ///< command to execute
-	size_t m_width, m_height; ///< size of window
-	int m_bevel; ///< distance to window edge from font in pixels
-	GC m_gc; ///< graphic context
-	bool m_end; ///< marks when this object is done
-	std::vector<std::string> m_history; ///< history list of commands
-	size_t m_current_history_item; ///< holds current position in command history
-	std::string m_history_file; ///< holds filename for command history file
+    FbTk::Font m_font; ///< font used to draw command text
+    FbTk::FbWindow m_win;  ///< toplevel window 
+    Display *m_display;  ///< display connection
+    std::string m_runtext; ///< command to execute
+    int m_bevel; ///< distance to window edge from font in pixels
+    GC m_gc; ///< graphic context
+    bool m_end; ///< marks when this object is done
+    std::vector<std::string> m_history; ///< history list of commands
+    size_t m_current_history_item; ///< holds current position in command history
+    std::string m_history_file; ///< holds filename for command history file
 };
 
 #endif // FBRUN_HH

M util/fbrun/main.cc → util/fbrun/main.cc

@@ -19,11 +19,12 @@ // LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
 // FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
 // DEALINGS IN THE SOFTWARE.
 
-// $Id: main.cc,v 1.7 2002/11/27 21:54:11 fluxgen Exp $
+// $Id: main.cc,v 1.8 2002/12/05 00:07:39 fluxgen Exp $
 
 #include "FbRun.hh"
 #include "App.hh"
 #include "StringUtil.hh"
+#include "Color.hh"
 
 #include <string>
 #include <iostream>
@@ -31,134 +32,118 @@
 using namespace std;
 
 void showUsage(const char *progname) {
-	cerr<<"fbrun 1.1.2 : (c) 2002 Henrik Kinnunen"<<endl;
-	cerr<<"Usage: "<<
-		progname<<" [arguments]"<<endl<<
-		"Arguments: "<<endl<<
-		"   -font [font name]           Text font"<<endl<<
-		"   -title [title name]         Set title"<<endl<<
-		"   -text [text]                Text input"<<endl<<
-		"   -w [width]                  Window width in pixels"<<endl<<
-		"   -h [height]                 Window height in pixels"<<endl<<
-		"   -display [display string]   Display name"<<endl<<
-		"   -pos [x] [y]                Window position in pixels"<<endl<<
-		"   -fg [color name]            Foreground text color"<<endl<<
-		"   -bg [color name]            Background color"<<endl<<
-		"   -a                          Antialias"<<endl<<
-		"   -hf [history file]          History file to load (default ~/.fluxbox/history)"<<endl<<
-		"   -help                       Show this help"<<endl<<endl<<
-		"Example: fbrun -fg black -bg white -text xterm -title \"run xterm\""<<endl;
+    cerr<<"fbrun 1.1.2 : (c) 2002 Henrik Kinnunen"<<endl;
+    cerr<<"Usage: "<<
+        progname<<" [arguments]"<<endl<<
+        "Arguments: "<<endl<<
+        "   -font [font name]           Text font"<<endl<<
+        "   -title [title name]         Set title"<<endl<<
+        "   -text [text]                Text input"<<endl<<
+        "   -w [width]                  Window width in pixels"<<endl<<
+        "   -h [height]                 Window height in pixels"<<endl<<
+        "   -display [display string]   Display name"<<endl<<
+        "   -pos [x] [y]                Window position in pixels"<<endl<<
+        "   -fg [color name]            Foreground text color"<<endl<<
+        "   -bg [color name]            Background color"<<endl<<
+        "   -a                          Antialias"<<endl<<
+        "   -hf [history file]          History file to load (default ~/.fluxbox/fbrun_history)"<<endl<<
+        "   -help                       Show this help"<<endl<<endl<<
+        "Example: fbrun -fg black -bg white -text xterm -title \"run xterm\""<<endl;
 }
 
 int main(int argc, char **argv) {
-	int x = 0, y = 0; // default pos of window
-	size_t width = 200, height = 32; // default size of window
-	bool set_height = false, set_width=false; // use height/width of font by default
-	bool set_pos = false; // set position
-	bool antialias = false; // antialias text
-	string fontname; // font name
-	string title("Run program"); // default title
-	string text;         // default input text
-	string foreground("black");   // text color
-	string background("white");   // text background color
-	string display_name; // name of the display connection
-	string history_file("~/.fluxbox/fbrun_history"); // command history file
-	// parse arguments
-	for (int i=1; i<argc; i++) {
-		if (strcmp(argv[i], "-font") == 0 && i+1 < argc) {
-			fontname = argv[++i];
-		} else if (strcmp(argv[i], "-title") == 0 && i+1 < argc) {
-			title = argv[++i];
-		} else if (strcmp(argv[i], "-text") == 0 && i+1 < argc) {
-			text = argv[++i];
-		} else if (strcmp(argv[i], "-w") == 0 && i+1 < argc) {
-			width = atoi(argv[++i]);			
-			set_width = true;
-		} else if (strcmp(argv[i], "-h") == 0 && i+1 < argc) {
-			height = atoi(argv[++i]);
-			set_height = true; // mark true else the height of font will be used
-		} else if (strcmp(argv[i], "-display") == 0 && i+1 < argc) {
-			display_name = argv[++i];
-		} else if (strcmp(argv[i], "-pos") == 0 && i+2 < argc) {
-			x = atoi(argv[++i]);
-			y = atoi(argv[++i]);
-			set_pos = true;
-		} else if (strcmp(argv[i], "-fg") == 0 && i+1 < argc) {
-			foreground = argv[++i];
-		} else if (strcmp(argv[i], "-bg") == 0 && i+1 < argc) {
-			background = argv[++i];
-		} else if (strcmp(argv[i], "-a") == 0) {
-			antialias = true;
-		} else if (strcmp(argv[i], "-hf") == 0 && i+1 < argc) {
-			history_file = argv[++i];
-		} else if (strcmp(argv[i], "-h") == 0 || strcmp(argv[i], "-help") == 0) {
-			showUsage(argv[0]);
-			exit(0);
-		} else {
-			cerr<<"Invalid argument: "<<argv[i]<<endl;
-			showUsage(argv[0]);
-			exit(0);
-		}
+    int x = 0, y = 0; // default pos of window
+    size_t width = 200, height = 32; // default size of window
+    bool set_height = false, set_width=false; // use height/width of font by default
+    bool set_pos = false; // set position
+    bool antialias = false; // antialias text
+    string fontname; // font name
+    string title("Run program"); // default title
+    string text;         // default input text
+    string foreground("black");   // text color
+    string background("white");   // text background color
+    string display_name; // name of the display connection
+    string history_file("~/.fluxbox/fbrun_history"); // command history file
+    // parse arguments
+    for (int i=1; i<argc; i++) {
+        if (strcmp(argv[i], "-font") == 0 && i+1 < argc) {
+            fontname = argv[++i];
+        } else if (strcmp(argv[i], "-title") == 0 && i+1 < argc) {
+            title = argv[++i];
+        } else if (strcmp(argv[i], "-text") == 0 && i+1 < argc) {
+            text = argv[++i];
+        } else if (strcmp(argv[i], "-w") == 0 && i+1 < argc) {
+            width = atoi(argv[++i]);			
+            set_width = true;
+        } else if (strcmp(argv[i], "-h") == 0 && i+1 < argc) {
+            height = atoi(argv[++i]);
+            set_height = true; // mark true else the height of font will be used
+        } else if (strcmp(argv[i], "-display") == 0 && i+1 < argc) {
+            display_name = argv[++i];
+        } else if (strcmp(argv[i], "-pos") == 0 && i+2 < argc) {
+            x = atoi(argv[++i]);
+            y = atoi(argv[++i]);
+            set_pos = true;
+        } else if (strcmp(argv[i], "-fg") == 0 && i+1 < argc) {
+            foreground = argv[++i];
+        } else if (strcmp(argv[i], "-bg") == 0 && i+1 < argc) {
+            background = argv[++i];
+        } else if (strcmp(argv[i], "-a") == 0) {
+            antialias = true;
+        } else if (strcmp(argv[i], "-hf") == 0 && i+1 < argc) {
+            history_file = argv[++i];
+        } else if (strcmp(argv[i], "-h") == 0 || strcmp(argv[i], "-help") == 0) {
+            showUsage(argv[0]);
+            exit(0);
+        } else {
+            cerr<<"Invalid argument: "<<argv[i]<<endl;
+            showUsage(argv[0]);
+            exit(0);
+        }
 
-	}
+    }
 
-	try {
+    try {
 		
-		FbTk::App application(display_name.c_str());
-		Display *disp = application.display();
-				
-		FbRun fbrun;
+        FbTk::App application(display_name.c_str());
+        FbRun fbrun;
 
-		if (fontname.size() != 0) {
-			if (!fbrun.loadFont(fontname.c_str())) {
-				cerr<<"Failed to load font: "<<fontname<<endl;
-				cerr<<"Falling back to \"fixed\""<<endl;
-			}
-		}
+        if (fontname.size() != 0) {
+            if (!fbrun.loadFont(fontname.c_str())) {
+                cerr<<"Failed to load font: "<<fontname<<endl;
+                cerr<<"Falling back to \"fixed\""<<endl;
+            }
+        }
 
-		// get color
-		XColor xc_foreground, xc_background;
-		if (XParseColor(disp, DefaultColormap(disp, DefaultScreen(disp)),
-			foreground.c_str(),
-			&xc_foreground) == 0) {
-			cerr<<"Faild to lookup color: "<<foreground<<endl;
-		}
-
-		if (XParseColor(disp, DefaultColormap(disp, DefaultScreen(disp)),
-			background.c_str(),
-			&xc_background) == 0) {
-			cerr<<"Faild to lookup color: "<<background<<endl;
-		}
-
-		XAllocColor(disp, DefaultColormap(disp, DefaultScreen(disp)),
-			&xc_foreground);
-		XAllocColor(disp, DefaultColormap(disp, DefaultScreen(disp)),
-			&xc_background);
+        // get color
+        XColor xc_foreground, xc_background;
+        FbTk::Color fg_color(foreground.c_str(), 0);
+        FbTk::Color bg_color(background.c_str(), 0);
 		
-		fbrun.setForeground(xc_foreground);
-		fbrun.setBackground(xc_background);
+        fbrun.setForeground(fg_color);
+        fbrun.setBackground(bg_color);
 
-		if (set_height)
-			fbrun.resize(fbrun.width(), height);
-		if (set_width)
-			fbrun.resize(width, fbrun.height());
-		if (antialias)
-			fbrun.setAntialias(antialias);
-		// expand and load command history
-		string expanded_filename = StringUtil::expandFilename(history_file);
-		if (!fbrun.loadHistory(expanded_filename.c_str()))
-			cerr<<"FbRun Warning: Failed to load history file: "<<expanded_filename<<endl;
+        if (set_height)
+            fbrun.resize(fbrun.width(), height);
+        if (set_width)
+            fbrun.resize(width, fbrun.height());
+        if (antialias)
+            fbrun.setAntialias(antialias);
+        // expand and load command history
+        string expanded_filename = StringUtil::expandFilename(history_file);
+        if (!fbrun.loadHistory(expanded_filename.c_str()))
+            cerr<<"FbRun Warning: Failed to load history file: "<<expanded_filename<<endl;
 
-		fbrun.setTitle(title);
-		fbrun.setText(text);
-		fbrun.show();
+        fbrun.setTitle(title);
+        fbrun.setText(text);
+        fbrun.show();
 		
-		if (set_pos)
-			fbrun.move(x, y);
+        if (set_pos)
+            fbrun.move(x, y);
 		
-		application.eventLoop();
+        application.eventLoop();
 
-	} catch (string errstr) {
-		cerr<<"Error: "<<errstr<<endl;
-	}
+    } catch (string errstr) {
+        cerr<<"Error: "<<errstr<<endl;
+    }
 }