Re: [pgadmin-hackers] Query tool layout issues

2010-03-04 Thread Dave Page 
On Wed, Mar 3, 2010 at 10:03 PM, Guillaume Lelarge
 wrote:

> Done too. See miscquery.patch.

The Query tool looks good now, but I get a crash if I try to open the
Options dialogue:

Thread 0 Crashed:  Dispatch queue: com.apple.main-thread
0   org.postgresql.pgadmin  0x001eddb0
pgSettingItemHashmap_wxImplementation_HashTable::DeleteNode(_wxHashTable_NodeBase*)
+ 28512
1   org.postgresql.pgadmin  0x001f21f9
pgSettingItemHashmap_wxImplementation_HashTable::DeleteNode(_wxHashTable_NodeBase*)
+ 45993
2   org.postgresql.pgadmin  0x0018c926
dlgSelDBNode::~dlgSelDBNode() + 342
3   libwx_base_carbonu-2.8.0.dylib  0x01375463
wxEvtHandler::ProcessEventIfMatches(wxEventTableEntryBase const&,
wxEvtHandler*, wxEvent&) + 131
4   libwx_base_carbonu-2.8.0.dylib  0x01375526
wxEvtHandler::SearchDynamicEventTable(wxEvent&) + 86
5   libwx_base_carbonu-2.8.0.dylib  0x01375df3
wxEvtHandler::ProcessEvent(wxEvent&) + 179
6   libwx_base_carbonu-2.8.0.dylib  0x01375da9
wxEvtHandler::ProcessEvent(wxEvent&) + 105
7   libwx_macu_core-2.8.0.dylib 0x00f50f80
wxMenuBase::SendEvent(int, int) + 144
8   libwx_macu_core-2.8.0.dylib 0x00e9b6ac
wxMenu::MacHandleCommandProcess(wxMenuItem*, int, wxWindow*) + 60
9   libwx_macu_core-2.8.0.dylib 0x00ecc3ab
wxMacWindowEventHandler(OpaqueEventHandlerCallRef*, OpaqueEventRef*,
void*) + 427
10  com.apple.HIToolbox 0x90e5ee29
DispatchEventToHandlers(EventTargetRec*, OpaqueEventRef*,
HandlerCallRec*) + 1567
11  com.apple.HIToolbox 0x90e5e0f0
SendEventToEventTargetInternal(OpaqueEventRef*, OpaqueEventTargetRef*,
HandlerCallRec*) + 411
12  com.apple.HIToolbox 0x90e80981 SendEventToEventTarget + 52
13  com.apple.HIToolbox 0x90eace3b
SendHICommandEvent(unsigned long, HICommand const*, unsigned long,
unsigned long, unsigned char, void const*, OpaqueEventTargetRef*,
OpaqueEventTargetRef*, OpaqueEventRef**) + 448
14  com.apple.HIToolbox 0x90ed1b20
SendMenuCommandWithContextAndModifiers + 66
15  com.apple.HIToolbox 0x90ed1ad7 SendMenuItemSelectedEvent + 
121
16  com.apple.HIToolbox 0x90ed19d3
FinishMenuSelection(SelectionData*, MenuResult*, MenuResult*) + 152
17  com.apple.HIToolbox 0x90ea1212
MenuSelectCore(MenuData*, Point, double, unsigned long,
OpaqueMenuRef**, unsigned short*) + 440
18  com.apple.HIToolbox 0x91053144 MenuSelect + 190
19  libwx_macu_core-2.8.0.dylib 0x00ec3af9
wxMacTopLevelMouseEventHandler(OpaqueEventHandlerCallRef*,
OpaqueEventRef*, void*) + 1193
20  libwx_macu_core-2.8.0.dylib 0x00e60b95
wxMacAppEventHandler(OpaqueEventHandlerCallRef*, OpaqueEventRef*,
void*) + 1061
21  com.apple.HIToolbox 0x90e5ee29
DispatchEventToHandlers(EventTargetRec*, OpaqueEventRef*,
HandlerCallRec*) + 1567
22  com.apple.HIToolbox 0x90e5e0f0
SendEventToEventTargetInternal(OpaqueEventRef*, OpaqueEventTargetRef*,
HandlerCallRec*) + 411
23  com.apple.HIToolbox 0x90e80981 SendEventToEventTarget + 52
24  com.apple.HIToolbox 0x90e924f7
ToolboxEventDispatcherHandler(OpaqueEventHandlerCallRef*,
OpaqueEventRef*, void*) + 1417
25  com.apple.HIToolbox 0x90e5f27a
DispatchEventToHandlers(EventTargetRec*, OpaqueEventRef*,
HandlerCallRec*) + 2672
26  com.apple.HIToolbox 0x90e5e0f0
SendEventToEventTargetInternal(OpaqueEventRef*, OpaqueEventTargetRef*,
HandlerCallRec*) + 411
27  com.apple.HIToolbox 0x90e80981 SendEventToEventTarget + 52
28  libwx_macu_core-2.8.0.dylib 0x00e5efe6
wxApp::MacHandleOneEvent(void*) + 38
29  libwx_macu_core-2.8.0.dylib 0x00e5f80b wxApp::MacDoOneEvent() + 123
30  libwx_macu_core-2.8.0.dylib 0x00e78fd3 wxEventLoop::Dispatch() + 35
31  libwx_macu_core-2.8.0.dylib 0x00f1f708 wxEventLoopManual::Run() + 
136
32  libwx_macu_core-2.8.0.dylib 0x00ef8bf3 wxAppBase::MainLoop() + 83
33  libwx_base_carbonu-2.8.0.dylib  0x0132b81a wxEntry(int&, wchar_t**) + 
106
34  org.postgresql.pgadmin  0x0003f4c8 0x1000 + 255176
35  org.postgresql.pgadmin  0x0003f1c9 0x1000 + 254409


-- 
Dave Page
EnterpriseDB UK: http://www.enterprisedb.com
PG East Conference: http://www.enterprisedb.com/community/nav-pg-east-2010.do

-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

[pgadmin-hackers] SVN Commit by guillaume: r8201 - in branches/REL-1_10_0_PATCHES/pgadmin3: . pgadmin/dlg

2010-03-04 Thread svn 
Author: guillaume

Date: 2010-03-04 19:56:17 + (Thu, 04 Mar 2010)

New Revision: 8201

Revision summary: http://svn.pgadmin.org/cgi-bin/viewcvs.cgi/?rev=8201&view=rev

Log:
Correctly save the position of a pgFrame, per a report from Andreas Laggner and 
Dan Halbert.



Modified:
   branches/REL-1_10_0_PATCHES/pgadmin3/CHANGELOG
   branches/REL-1_10_0_PATCHES/pgadmin3/pgadmin/dlg/dlgClasses.cpp

-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

[pgadmin-hackers] SVN Commit by guillaume: r8202 - branches/REL-1_10_0_PATCHES/pgadmin3

2010-03-04 Thread svn 
Author: guillaume

Date: 2010-03-04 19:56:38 + (Thu, 04 Mar 2010)

New Revision: 8202

Revision summary: http://svn.pgadmin.org/cgi-bin/viewcvs.cgi/?rev=8202&view=rev

Log:
Update the .pot file.



Modified:
   branches/REL-1_10_0_PATCHES/pgadmin3/pgadmin3.pot

-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

[pgadmin-hackers] SVN Commit by guillaume: r8203 - in trunk/pgadmin3: . pgadmin/dlg

2010-03-04 Thread svn 
Author: guillaume

Date: 2010-03-04 19:58:38 + (Thu, 04 Mar 2010)

New Revision: 8203

Revision summary: http://svn.pgadmin.org/cgi-bin/viewcvs.cgi/?rev=8203&view=rev

Log:
Correctly save the position of a pgFrame, per a report from Andreas Laggner and 
Dan Halbert.



Modified:
   trunk/pgadmin3/CHANGELOG
   trunk/pgadmin3/pgadmin/dlg/dlgClasses.cpp

-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

[pgadmin-hackers] [pgAdmin III] #147: Window Size SQL-query

2010-03-04 Thread pgAdmin Trac 
#147: Window Size SQL-query
-+--
 Reporter:  Andreas Laggner  |   Owner:  gleu  
 Type:  bug  |  Status:  new   
 Priority:  minor|   Milestone:  1.10.2
Component:  pgadmin  | Version:  1.10  
 Keywords:  window   |Platform:  linux 
-+--
 i'm running 1.10.0 binary with openSuse 11.2 64bit. When i open a sql-
 query the window size is standard and not as i left it last time i used
 pgadmin3. How can i get pgadmin3 remembering the window size? Hint: this
 problem occurs on more 11.2 machines, but on the 11.1 ones pgadmin
 remembers the window size

-- 
Ticket URL: 
pgAdmin III 
pgAdmin III

-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

Re: [pgadmin-hackers] [pgAdmin III] #147: Window Size SQL-query

2010-03-04 Thread pgAdmin Trac 
#147: Window Size SQL-query
-+--
 Reporter:  Andreas Laggner  |   Owner:  gleu  
 Type:  bug  |  Status:  new   
 Priority:  minor|   Milestone:  1.10.2
Component:  pgadmin  | Version:  1.10  
 Keywords:  window   |Platform:  linux 
-+--

Comment(by gleu):

 I am seeing the same issue on Ubuntu 9.10 GNOME 64-bit with pgadmin
 1.10.0. The window sizes are not remembered, though the internal
 arrangements are (e.g. if you moved the scratchpad). 1.10.0 remembers
 window sizes on Windows just fine.

 Dan Halbert

-- 
Ticket URL: 
pgAdmin III 
pgAdmin III

-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

Re: [pgadmin-hackers] [pgAdmin III] #147: Window Size SQL-query

2010-03-04 Thread pgAdmin Trac 
#147: Window Size SQL-query
--+-
  Reporter:  Andreas Laggner  |   Owner:  gleu  
  Type:  bug  |  Status:  closed
  Priority:  minor|   Milestone:  1.10.2
 Component:  pgadmin  | Version:  1.10  
Resolution:  fixed|Keywords:  window
  Platform:  linux|  
--+-
Changes (by gleu):

  * status:  new => closed
  * resolution:  => fixed


Comment:

 Fixed in r8201 and r8203.

-- 
Ticket URL: 
pgAdmin III 
pgAdmin III

-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

Re: [pgadmin-hackers] Query tool layout issues

2010-03-04 Thread Guillaume Lelarge 
Le 04/03/2010 11:23, Dave Page a écrit :
> On Wed, Mar 3, 2010 at 10:03 PM, Guillaume Lelarge
>  wrote:
> 
>> Done too. See miscquery.patch.
> 
> The Query tool looks good now, but I get a crash if I try to open the
> Options dialogue:
> 

I don't get a crash when I open the frmOptions window. With which patch
did you get that crash?

> Thread 0 Crashed:  Dispatch queue: com.apple.main-thread
> 0   org.postgresql.pgadmin0x001eddb0
> pgSettingItemHashmap_wxImplementation_HashTable::DeleteNode(_wxHashTable_NodeBase*)
> + 28512
> 1   org.postgresql.pgadmin0x001f21f9
> pgSettingItemHashmap_wxImplementation_HashTable::DeleteNode(_wxHashTable_NodeBase*)
> + 45993
> 2   org.postgresql.pgadmin0x0018c926
> dlgSelDBNode::~dlgSelDBNode() + 342
> 3   libwx_base_carbonu-2.8.0.dylib0x01375463
> wxEvtHandler::ProcessEventIfMatches(wxEventTableEntryBase const&,
> wxEvtHandler*, wxEvent&) + 131
> 4   libwx_base_carbonu-2.8.0.dylib0x01375526
> wxEvtHandler::SearchDynamicEventTable(wxEvent&) + 86
> 5   libwx_base_carbonu-2.8.0.dylib0x01375df3
> wxEvtHandler::ProcessEvent(wxEvent&) + 179
> 6   libwx_base_carbonu-2.8.0.dylib0x01375da9
> wxEvtHandler::ProcessEvent(wxEvent&) + 105
> 7   libwx_macu_core-2.8.0.dylib   0x00f50f80
> wxMenuBase::SendEvent(int, int) + 144
> 8   libwx_macu_core-2.8.0.dylib   0x00e9b6ac
> wxMenu::MacHandleCommandProcess(wxMenuItem*, int, wxWindow*) + 60
> 9   libwx_macu_core-2.8.0.dylib   0x00ecc3ab
> wxMacWindowEventHandler(OpaqueEventHandlerCallRef*, OpaqueEventRef*,
> void*) + 427
> 10  com.apple.HIToolbox   0x90e5ee29
> DispatchEventToHandlers(EventTargetRec*, OpaqueEventRef*,
> HandlerCallRec*) + 1567
> 11  com.apple.HIToolbox   0x90e5e0f0
> SendEventToEventTargetInternal(OpaqueEventRef*, OpaqueEventTargetRef*,
> HandlerCallRec*) + 411
> 12  com.apple.HIToolbox   0x90e80981 SendEventToEventTarget + 52
> 13  com.apple.HIToolbox   0x90eace3b
> SendHICommandEvent(unsigned long, HICommand const*, unsigned long,
> unsigned long, unsigned char, void const*, OpaqueEventTargetRef*,
> OpaqueEventTargetRef*, OpaqueEventRef**) + 448
> 14  com.apple.HIToolbox   0x90ed1b20
> SendMenuCommandWithContextAndModifiers + 66
> 15  com.apple.HIToolbox   0x90ed1ad7 SendMenuItemSelectedEvent + 
> 121
> 16  com.apple.HIToolbox   0x90ed19d3
> FinishMenuSelection(SelectionData*, MenuResult*, MenuResult*) + 152
> 17  com.apple.HIToolbox   0x90ea1212
> MenuSelectCore(MenuData*, Point, double, unsigned long,
> OpaqueMenuRef**, unsigned short*) + 440
> 18  com.apple.HIToolbox   0x91053144 MenuSelect + 190
> 19  libwx_macu_core-2.8.0.dylib   0x00ec3af9
> wxMacTopLevelMouseEventHandler(OpaqueEventHandlerCallRef*,
> OpaqueEventRef*, void*) + 1193
> 20  libwx_macu_core-2.8.0.dylib   0x00e60b95
> wxMacAppEventHandler(OpaqueEventHandlerCallRef*, OpaqueEventRef*,
> void*) + 1061
> 21  com.apple.HIToolbox   0x90e5ee29
> DispatchEventToHandlers(EventTargetRec*, OpaqueEventRef*,
> HandlerCallRec*) + 1567
> 22  com.apple.HIToolbox   0x90e5e0f0
> SendEventToEventTargetInternal(OpaqueEventRef*, OpaqueEventTargetRef*,
> HandlerCallRec*) + 411
> 23  com.apple.HIToolbox   0x90e80981 SendEventToEventTarget + 52
> 24  com.apple.HIToolbox   0x90e924f7
> ToolboxEventDispatcherHandler(OpaqueEventHandlerCallRef*,
> OpaqueEventRef*, void*) + 1417
> 25  com.apple.HIToolbox   0x90e5f27a
> DispatchEventToHandlers(EventTargetRec*, OpaqueEventRef*,
> HandlerCallRec*) + 2672
> 26  com.apple.HIToolbox   0x90e5e0f0
> SendEventToEventTargetInternal(OpaqueEventRef*, OpaqueEventTargetRef*,
> HandlerCallRec*) + 411
> 27  com.apple.HIToolbox   0x90e80981 SendEventToEventTarget + 52
> 28  libwx_macu_core-2.8.0.dylib   0x00e5efe6
> wxApp::MacHandleOneEvent(void*) + 38
> 29  libwx_macu_core-2.8.0.dylib   0x00e5f80b wxApp::MacDoOneEvent() + 123
> 30  libwx_macu_core-2.8.0.dylib   0x00e78fd3 wxEventLoop::Dispatch() + 35
> 31  libwx_macu_core-2.8.0.dylib   0x00f1f708 wxEventLoopManual::Run() + 
> 136
> 32  libwx_macu_core-2.8.0.dylib   0x00ef8bf3 wxAppBase::MainLoop() + 83
> 33  libwx_base_carbonu-2.8.0.dylib0x0132b81a wxEntry(int&, wchar_t**) + 
> 106
> 34  org.postgresql.pgadmin0x0003f4c8 0x1000 + 255176
> 35  org.postgresql.pgadmin0x0003f1c9 0x1000 + 254409
> 
> 


-- 
Guillaume.
 http://www.postgresqlfr.org
 http://dalibo.com

-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

Re: [pgadmin-hackers] Query tool layout issues

2010-03-04 Thread Dave Page 
On Thu, Mar 4, 2010 at 8:42 PM, Guillaume Lelarge
 wrote:
> Le 04/03/2010 11:23, Dave Page a écrit :
>> On Wed, Mar 3, 2010 at 10:03 PM, Guillaume Lelarge
>>  wrote:
>>
>>> Done too. See miscquery.patch.
>>
>> The Query tool looks good now, but I get a crash if I try to open the
>> Options dialogue:
>>
>
> I don't get a crash when I open the frmOptions window. With which patch
> did you get that crash?

All of them on top of the latest SVN code. I tried a make clean which
didn't help.



-- 
Dave Page
EnterpriseDB UK: http://www.enterprisedb.com
PG East Conference: http://www.enterprisedb.com/community/nav-pg-east-2010.do

-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

Re: [pgadmin-hackers] Query tool layout issues

2010-03-04 Thread Guillaume Lelarge 
Le 04/03/2010 21:45, Dave Page a écrit :
> On Thu, Mar 4, 2010 at 8:42 PM, Guillaume Lelarge
>  wrote:
>> Le 04/03/2010 11:23, Dave Page a écrit :
>>> On Wed, Mar 3, 2010 at 10:03 PM, Guillaume Lelarge
>>>  wrote:
>>>
 Done too. See miscquery.patch.
>>>
>>> The Query tool looks good now, but I get a crash if I try to open the
>>> Options dialogue:
>>>
>>
>> I don't get a crash when I open the frmOptions window. With which patch
>> did you get that crash?
> 
> All of them on top of the latest SVN code. I tried a make clean which
> didn't help.
> 

I have them too, and still don't crash. Can you try with empty prefs?


-- 
Guillaume.
 http://www.postgresqlfr.org
 http://dalibo.com

-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

[pgadmin-hackers] Strange error message when linking

2010-03-04 Thread Guillaume Lelarge 
Hi,

I'm still working on a ctlColourPicker widget, our custom
wxColourPickerCtrl to work around issues found on Mac OS X by Dave. I
was obviously wrong when I said that coding this widget would be simple :-/

The patch attached is my current work. I'm pretty ashamed of it, but
can't find a way to fix my linking issue. Here is the error I got at the
linking step:

pgAdmin3.o: In function `ctlColourPickerXmlHandler':
/home/guillaume/freeprojects/git.pgadmin/pgadmin/../pgadmin/include/ctl/xh_ctlcolourpicker.h:25:
undefined reference to `vtable for ctlColourPickerXmlHandler'
collect2: ld returned 1 exit status

I have absolutely no idea what this could mean. Search engines didn't
help me on this. I don't know what to do. If you have any idea, I
welcome them.

If I don't find any way to fix this before the end of the week, I think
I'll fix the first issue another way: replacing wxColourPickerCtrl with
wxBitmapButton for each control (which means duplicated code, but much
simpler work). Simpler work means quicker work, so more time to code new
interesting 9.0 features. And I'll get back to this widget for next dev
cycle. But I would definitely prefer having a solution to my linking issue.


-- 
Guillaume.
 http://www.postgresqlfr.org
 http://dalibo.com
diff --git a/pgadmin/ctl/ctlColourPicker.cpp b/pgadmin/ctl/ctlColourPicker.cpp
new file mode 100644
index 000..00c4897
--- /dev/null
+++ b/pgadmin/ctl/ctlColourPicker.cpp
@@ -0,0 +1,42 @@
+//
+//
+// pgAdmin III - PostgreSQL Tools
+// RCS-ID:  $Id$
+// Copyright (C) 2002 - 2010, The pgAdmin Development Team
+// This software is released under the BSD Licence
+//
+// ctlColourPicker.cpp - Colour Picker with a wxBitmapButton
+//
+//
+
+// wxWindows headers
+#include 
+
+// App headers
+#include "pgAdmin3.h"
+#include "ctl/ctlColourPicker.h"
+
+
+/*
+BEGIN_EVENT_TABLE(ctlColourPicker, wxBitmapButton)
+EVT_BUTTON(ctlColourPicker::OnClick)
+END_EVENT_TABLE()
+*/
+
+
+ctlColourPicker::ctlColourPicker(wxWindow* parent, wxWindowID id, const wxBitmap& bitmap)
+: wxBitmapButton(parent, id, bitmap)
+{
+}
+
+wxColour ctlColourPicker::GetColour()
+{
+}
+
+void ctlColourPicker::SetColour(wxColour colour)
+{
+}
+
+void ctlColourPicker::OnClick(wxCommandEvent &ev)
+{
+}
diff --git a/pgadmin/ctl/module.mk b/pgadmin/ctl/module.mk
index e0cd6d7..0ef6bea 100644
--- a/pgadmin/ctl/module.mk
+++ b/pgadmin/ctl/module.mk
@@ -12,6 +12,7 @@
 pgadmin3_SOURCES += \
 	$(srcdir)/ctl/calbox.cpp \
 $(srcdir)/ctl/ctlCheckTreeView.cpp \
+$(srcdir)/ctl/ctlColourPicker.cpp \
 $(srcdir)/ctl/ctlComboBox.cpp \
 $(srcdir)/ctl/ctlListView.cpp \
 $(srcdir)/ctl/ctlMenuToolbar.cpp \
diff --git a/pgadmin/ctl/xh_ctlcolourpicker.cpp b/pgadmin/ctl/xh_ctlcolourpicker.cpp
new file mode 100644
index 000..2e566e1
--- /dev/null
+++ b/pgadmin/ctl/xh_ctlcolourpicker.cpp
@@ -0,0 +1,33 @@
+//
+//
+// pgAdmin III - PostgreSQL Tools
+// RCS-ID:  $Id$
+// Copyright (C) 2002 - 2010, The pgAdmin Development Team
+// This software is released under the BSD Licence
+//
+// xh_ctlcolourpicker.cpp - ctlColourPicker handler
+//
+//
+ 
+#include "pgAdmin3.h"
+
+#include "wx/wx.h"
+#include "ctl/ctlColourPicker.h"
+#include "ctl/xh_ctlcolourpicker.h"
+
+IMPLEMENT_DYNAMIC_CLASS(ctlColourPickerXmlHandler, wxBitmapButtonXmlHandler)
+
+
+wxObject *ctlColourPickerXmlHandler::DoCreateResource()
+{ 
+ctlColourPicker *ctl=new ctlColourPicker(m_parentAsWindow, GetID(), wxNullBitmap);
+
+SetupWindow(ctl);
+   
+return ctl;
+}
+
+bool ctlColourPickerXmlHandler::CanHandle(wxXmlNode *node)
+{
+return IsOfClass(node, wxT("ctlColourPicker"));
+}
diff --git a/pgadmin/include/ctl/ctlColourPicker.h b/pgadmin/include/ctl/ctlColourPicker.h
new file mode 100644
index 000..f2f06f2
--- /dev/null
+++ b/pgadmin/include/ctl/ctlColourPicker.h
@@ -0,0 +1,34 @@
+//
+//
+// pgAdmin III - PostgreSQL Tools
+// RCS-ID:  $Id$
+// Copyright (C) 2002 - 2010, The pgAdmin Development Team
+// This software is released under the BSD Licence
+//
+// ctlColourPicker.cpp - TreeView with Checkboxes
+//
+//
+
+
+#ifndef _CTLCOLOURPICKER_H
+#define _CTLCOLOURPICKER_H
+
+// wxWindows headers
+#include 
+#include "utils/misc.h"
+
+
+
+class ctlColourPicker : public wxBitmapButton
+{
+public:
+ctlColourPicker(wxWindow* parent, wxWindowID id, const wxBitmap& bitmap);
+
+wxColour GetColour();
+void SetColour(wxColour colour);
+
+private:
+void OnClick(wxCommandEvent &evt);
+};
+
+#endif
diff --git a/pgadmin/include/ctl/module.mk b/pgadmin/include/ctl/modu

Re: [pgadmin-hackers] Strange error message when linking

2010-03-04 Thread Guillaume Lelarge 
Le 05/03/2010 00:52, Guillaume Lelarge a écrit :
> [...]
> I'm still working on a ctlColourPicker widget, our custom
> wxColourPickerCtrl to work around issues found on Mac OS X by Dave. I
> was obviously wrong when I said that coding this widget would be simple :-/
> 
> The patch attached is my current work. I'm pretty ashamed of it, but
> can't find a way to fix my linking issue. Here is the error I got at the
> linking step:
> 
> pgAdmin3.o: In function `ctlColourPickerXmlHandler':
> /home/guillaume/freeprojects/git.pgadmin/pgadmin/../pgadmin/include/ctl/xh_ctlcolourpicker.h:25:
> undefined reference to `vtable for ctlColourPickerXmlHandler'
> collect2: ld returned 1 exit status
> 
> I have absolutely no idea what this could mean. Search engines didn't
> help me on this. I don't know what to do. If you have any idea, I
> welcome them.
> 

Just tried on Windows, where it compiles and links. As it doesn't do
anything right now, I can't say if it works.


-- 
Guillaume.
 http://www.postgresqlfr.org
 http://dalibo.com

-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

Re: [pgadmin-hackers] Strange error message when linking

2010-03-04 Thread Ashesh Vashi 
Hi Guillaume,

Very small mistake. (I saw this error earlier.)
You forget to add "$(srcdir)/ctl/xh_ctlcolourpicker.cpp" in "pgadmin/ctl/
module.mk" file.
Please find the attached patch.

--
Thanks & Regards,
Ashesh Vashi

On Fri, Mar 5, 2010 at 5:44 AM, Guillaume Lelarge wrote:

> Le 05/03/2010 00:52, Guillaume Lelarge a écrit :
> > [...]
> > I'm still working on a ctlColourPicker widget, our custom
> > wxColourPickerCtrl to work around issues found on Mac OS X by Dave. I
> > was obviously wrong when I said that coding this widget would be simple
> :-/
> >
> > The patch attached is my current work. I'm pretty ashamed of it, but
> > can't find a way to fix my linking issue. Here is the error I got at the
> > linking step:
> >
> > pgAdmin3.o: In function `ctlColourPickerXmlHandler':
> >
> /home/guillaume/freeprojects/git.pgadmin/pgadmin/../pgadmin/include/ctl/xh_ctlcolourpicker.h:25:
> > undefined reference to `vtable for ctlColourPickerXmlHandler'
> > collect2: ld returned 1 exit status
> >
> > I have absolutely no idea what this could mean. Search engines didn't
> > help me on this. I don't know what to do. If you have any idea, I
> > welcome them.
> >
>
> Just tried on Windows, where it compiles and links. As it doesn't do
> anything right now, I can't say if it works.
>
>
> --
> Guillaume.
>  http://www.postgresqlfr.org
>  http://dalibo.com
>
> --
> Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>


ctlColourPicker_v2.patch
Description: Binary data

-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers

Re: [pgadmin-hackers] Strange error message when linking

2010-03-04 Thread Guillaume Lelarge 
Hi Ashesh,

Le 05/03/2010 07:31, Ashesh Vashi a écrit :
> [...]
> Very small mistake. (I saw this error earlier.)
> You forget to add "$(srcdir)/ctl/xh_ctlcolourpicker.cpp" in "pgadmin/ctl/
> module.mk" file.
> Please find the attached patch.
> 

Wow, thanks a lot. I thought I updated all module.mk file, but I was wrong.

Thanks again, I'll continue my work on this widget today.


-- 
Guillaume.
 http://www.postgresqlfr.org
 http://dalibo.com

-- 
Sent via pgadmin-hackers mailing list (pgadmin-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers