C / C++ / MFC

Michael Dunn30-Dec-01 19:16

30-Dec-01 19:16

What is the return value from Create()?

--Mike--
My really out-of-date homepage
Buffy's on. Gotta go, bye!
Sonork - 100.10414 AcidHelm
Big fan of Alyson Hannigan.

Jason Hooper31-Dec-01 14:17

Jason Hooper

31-Dec-01 14:17

Michael Dunn wrote:
What is the return value from Create()?

Seems to be 161.

// MessageId: ERROR_BAD_PATHNAME
//
// MessageText:
//
//  The specified path is invalid.
//
#define ERROR_BAD_PATHNAME               161L

Perhaps Software needs to be capitalized? Is the Windows XP Registry case sensitive?

BTW here is the code:

CRegKey autoRunKey;
long ret;

if ((ret = autoRunKey.Create(HKEY_CURRENT_USER, "\\Software\\Microsoft\\Windows\\CurrentVersion\\Run")) != ERROR_SUCCESS)
{
    CString f;
    f.Format("Couldn't open key.. return code is %ld", ret);

    AfxMessageBox(f);
}

-Jason
nirgle.bitdevil.com

Michael Dunn31-Dec-01 14:33

31-Dec-01 14:33

hmm, two things to change. 1, remove the backslash before "software". (Case doesn't matter with registry names). 2, never use KEY_ALL_ACCESS with registry functions because they will fail if the user is not an administrator. The default access for the ATL functions I've encountered is KEY_ALL_ACCESS, which in hindsight was a bad design decision. Anyway, don't take the default access, specify exactly the access you need (KEY_WRITE or whatever). You didn't say if the XP user is running as admin, but if they're not, then the access problem is definitely making the call fail.

--Mike--
My really out-of-date homepage
Buffy's on. Gotta go, bye!
Sonork - 100.10414 AcidHelm
Big fan of Alyson Hannigan.

Jason Hooper31-Dec-01 14:39

Jason Hooper

31-Dec-01 14:39

Thanks for the quick response. I tried removing the \\ on a hunch, and it ended up working. Good idea. I'll look into the permissions thing too.

Thanks!

Cool | :cool:

-Jason
nirgle.bitdevil.com

STL and member functions ???

Roger Wright (new)31-Dec-01 14:25

Roger Wright (new)

31-Dec-01 14:25

I haven't yet used XP, but it is supposed to be Win2K warmed over. This may be a permissions issue. Does your friend run the program under a user account that has permission to alter the Registry?

Christian Graus30-Dec-01 15:17

30-Dec-01 15:17

I'm working on a trace utility, mainly to extend my knowledge of IOStreams and STL. I have a map to keep track of a number of edit windows, which all take the same position and are sown based on a tab control.

map<cstring, ctracewnd*=""> m_TraceMap;

Now, I hide all my windows before showing the one selected, like this:

void HideWindow(std::pair<cstring, ctracewnd="" *=""> p)
{
CTraceWnd * pWnd = p.second;
pWnd->ShowWindow(SW_HIDE);
}

void CTabView::OnSelchangeTab(NMHDR* pNMHDR, LRESULT* pResult)
{
std::for_each(m_TraceMap.begin(), m_TraceMap.end(), HideWindow);
// etc, etc.

Now I have two questions.

1. How can I make HideWindow a member function of CTabView ? I've tried mem_funxxx stuff, to no avail, and spent a good amount of time reading 'Generic Programming and the STL', also without any success.

2. Better yet, is there a way I can call ShowWindow(SW_HIDE) directly, OR add a HideWindow function that does this to CTraceWnd, and call that ?

Thanks to anyone who answers.

Christian

After all, there's nothing wrong with an elite as long as I'm allowed to be part of it!! - Mike Burston Oct 23, 2001

Sonork ID 100.10002:MeanManOzI live in Bob's HungOut now

Todd Smith30-Dec-01 20:39

Todd Smith

30-Dec-01 20:39

class MyClass : public CTabView
{
public:
MyClass() {}

void HideWindow() { CTabView::ShowWindow(SW_SHOW); };
};

// this seems like it should be a template of some kind
void HideWindow(const std::pair<int, MyClass>& v)
{
MyClass& blah = const_cast<MyClass&>(v.second);

blah.HideWindow();
}

void main()
{
std::map<int, MyClass> blah;

std::for_each(blah.begin(), blah.end(), HideWindow);
}

Christian Graus31-Dec-01 1:20

This is what I'm trying to achieve...

31-Dec-01 1:20

Thanks, I'm sure this will work, and the way I'm doing it now works, too, but I want to get rid of the global function altogether and replace it with a member function, either of the class held in the second part of the pair, or in the class calling the function.

Christian

After all, there's nothing wrong with an elite as long as I'm allowed to be part of it!! - Mike Burston Oct 23, 2001

Sonork ID 100.10002:MeanManOzI live in Bob's HungOut now

Christian Graus31-Dec-01 1:24

Re: This is what I'm trying to achieve...

31-Dec-01 1:24

void CTabView::HideWindow(std::pair<cstring, ctracewnd="" *=""> p)
{
CTraceWnd * pWnd = p.second;
pWnd->ShowWindow(SW_HIDE);
}

void CTabView::OnSelchangeTab(NMHDR* pNMHDR, LRESULT* pResult)
{
std::for_each(m_TraceMap.begin(), m_TraceMap.end(), std::mem_fun_ref(&CTabView::HideWindow));

but I get this error:

error C2784: 'class std::mem_fun_ref_t<_R,_Ty> __cdecl std::mem_fun_ref(_R (__thiscall _Ty::*)(void))' : could not deduce template argument for '<unknown>' from 'void (__thiscall CTabView::*)(struct std::p
air<class cstring,class="" ctracewnd="" *="">)'

I thought maybe mem_fun1_ref, but things just got worse. I tried adding the function to CTraceWnd, but that was worse again....

Christian

After all, there's nothing wrong with an elite as long as I'm allowed to be part of it!! - Mike Burston Oct 23, 2001

Sonork ID 100.10002:MeanManOzI live in Bob's HungOut now

Tim Smith31-Dec-01 2:25

Tim Smith

31-Dec-01 2:25

Try making a "void HideWindow ()" method in CTraceWnd. Then "std::mem_fun_ref (CTraceWnd::HideWindow)" should work.

Tim Smith
Descartes Systems Sciences, Inc.

Re: This is what I'm trying to achieve...

Christian Graus31-Dec-01 10:24

31-Dec-01 10:24

I had done that previously, and I get this:

void CTabView::OnSelchangeTab(NMHDR* pNMHDR, LRESULT* pResult)
{
std::for_each(m_TraceMap.begin(), m_TraceMap.end(), std::mem_fun_ref(CTraceWnd::HideWindow));

void CTraceWnd::HideWindow(std::pair<cstring, ctracewnd*=""> p)
{
p.second->ShowWindow(SW_HIDE);
}

error C2784: 'class std::mem_fun_ref_t<_R,_Ty> __cdecl std::mem_fun_ref(_R (__thiscall _Ty::*)(void))' : could not deduce template argument for '<unknown>' from 'void (__thiscall CTraceWnd::*)(struct std::
pair<class cstring,class="" ctracewnd="" *="">)'

Tried mem_fun1_ref as well. I don't think this can work as I need to pull second out of the pair first, which requires some non-standard stuff found in STLPort, etc. I will install STLPort tonight.

Christian

After all, there's nothing wrong with an elite as long as I'm allowed to be part of it!! - Mike Burston Oct 23, 2001

Sonork ID 100.10002:MeanManOzI live in Bob's HungOut now

Alexander Berthold31-Dec-01 2:01

Alexander Berthold

31-Dec-01 2:01

Create a from std::unary_function derived helper class then it should work. For map for_each functors, it should look like this, remember the first pair argument must be const (i think). I assume your maps' pairs' first argument is an int.

class hidewindow_functor : public std::unary_function< std::pair<int const,CWnd*>, void>
{
public:
void operator() (std::pair<int const,CWnd*>& target) const
{
target.second->ShowWindow(SW_HIDE);
}
};

Cheers,
Alexander Berthold

Christian Graus31-Dec-01 10:35

CListCtrl - header notification ???

31-Dec-01 10:35

Thanks - although not quite what I was after, I'm now convinced I need some non standard STL stuff from STLPort to make it work the way I want. However, it only works if I make CWnd above CTraceWnd, can you tell me why STL cannot do the cast ?

Thanks.

Christian

After all, there's nothing wrong with an elite as long as I'm allowed to be part of it!! - Mike Burston Oct 23, 2001

Sonork ID 100.10002:MeanManOzI live in Bob's HungOut now

Christian Graus30-Dec-01 15:13

30-Dec-01 15:13

I have a CListCtrl derived class and I want to recieve notification when a user resizes the columns so I can make sure they dynamically resize and never have a scrolling portion. But none of my attempts to catch this message are working. I found an article that subclasses the CHeaderCtrl - is this the only way, or does anyone know of another way ? I just want tis one stupid message....

Christian

After all, there's nothing wrong with an elite as long as I'm allowed to be part of it!! - Mike Burston Oct 23, 2001

Sonork ID 100.10002:MeanManOzI live in Bob's HungOut now

Michael Dunn30-Dec-01 19:20

30-Dec-01 19:20

Check out HDN_BEGINTRACK, HDN_TRACK, HDN_ENDTRACK, HDN_DIVIDERDBLCLICK

--Mike--
My really out-of-date homepage
Buffy's on. Gotta go, bye!
Sonork - 100.10414 AcidHelm
Big fan of Alyson Hannigan.

Christian Graus31-Dec-01 1:05

31-Dec-01 1:05

I know about the messages, but overriding them using class wizard does *nothing*. Or have I got something else wrong and am just blaming this ? I went into class wizard, chose HDN_TRACK, put an AfxMessageBox in there and started sliding the things about to try and get a response. I'm using a splitter SDI app.

Christian

After all, there's nothing wrong with an elite as long as I'm allowed to be part of it!! - Mike Burston Oct 23, 2001

Sonork ID 100.10002:MeanManOzI live in Bob's HungOut now

Michael Dunn31-Dec-01 7:33

31-Dec-01 7:33

hmm... I just tried it with an Explorer-style app, handling the HDN_* messages in the list view, and it looks like ClassWizard has a bug. It makes ON_NOTIFY_REFLECT macros for the header messages, which is totally wrong. For example, change:

ON_NOTIFY_REFLECT(HDN_DIVIDERDBLCLICK, OnDividerdblclick)

ON_NOTIFY(HDN_DIVIDERDBLCLICK, 0, OnDividerdblclick)

The header control always has ID 0, that's the reason for the 0 above.

--Mike--
My really out-of-date homepage
Buffy's on. Gotta go, bye!
Sonork - 100.10414 AcidHelm
Big fan of Alyson Hannigan.

Christian Graus1-Jan-02 0:59

1-Jan-02 0:59

Thanks Mike, but I did this, handling the message in my CListCtrl derived class and changing the macro to ON_NOTIFY, inserting a 0 between the other two arguments. It is never called.

Christian

After all, there's nothing wrong with an elite as long as I'm allowed to be part of it!! - Mike Burston Oct 23, 2001

Sonork ID 100.10002:MeanManOzI live in Bob's HungOut now

Jonas Larsson1-Jan-02 23:20

Jonas Larsson

1-Jan-02 23:20

I did something like this about 6 months ago, so my memory is a bit rusty.

This worked for me anyway, but with some flickering of the scrollbars (I never could figure out how to get a shot at resizing the headers before the listctrl tried to draw them)

Here's a skeleton of the code I used.

BOOL CNumbersListCtrl::OnNotify(WPARAM wParam, LPARAM lParam, LRESULT* pResult) 
{
    NMHDR* pNMHDR = (NMHDR*)lParam;  	// Trap Header control notifications here 
	
    HD_NOTIFY *phdn = (HD_NOTIFY *)lParam; 
    switch(phdn->hdr.code)
 	{ 		
        case HDN_BEGINTRACK:
        {
            // Trap the HDN_BEGINTRACK message
            TRACE(_T("CNumbersListCtrl::OnNotify -- HDN_BEGINTRACK Trapped\n"));
            return(FALSE);  // return FALSE to allow tracking of the divider 
            break; 		
        }
        case HDN_ENDTRACK:
	{
            TRACE(_T("CNumbersListCtrl::OnNotify -- HDN_ENDTRACK Trapped\n"));
            if (phdn->iItem == 0)
	    {
                TRACE(_T("CNumbersListCtrl::OnNotify -- Resising right to fit\n"));
            }
            else if (phdn->iItem == 1)
	    {
                TRACE(_T("CNumbersListCtrl::OnNotify -- Resising left to fit\n"));
            }	
            break;
	}
    }	 	
    return CListCtrl::OnNotify(wParam, lParam, pResult); 

}

Christian Graus2-Jan-02 0:14

2-Jan-02 0:14

Thanks, but this did not work. Right now I am doing this in OnNotify:

GetAsyncKeyState(VK_LBUTTON);

if (GetAsyncKeyState(VK_LBUTTON) && 4294967284 == pnmh->code)
{
CRect rc;

GetClientRect(&rc);

SetColumnWidth(1, rc.Width() - GetColumnWidth(0));
}

if I change that big number to HDM_ITEMCHANGINGA, HDM_ITEMCHANGEDA, or the W equivelants ( or drop the A or W ) it either does nothing or crashes. I am certain the two messages I get are ITEMCHANGED and ITEMCHANGING. The HDM_TRACK message is a phantom - it is NEVER sent ASAIK.

I've tried several things and in the end I've concluded that CListCtrl is rubbish. It works for me as it stands, although it flickers and jumps when the mouse button is raised. I'm interested if you have any comments.

Christian

After all, there's nothing wrong with an elite as long as I'm allowed to be part of it!! - Mike Burston Oct 23, 2001

Sonork ID 100.10002:MeanManOzI live in Bob's HungOut now

Jonas Larsson2-Jan-02 0:36

Jonas Larsson

2-Jan-02 0:36

Christian Graus wrote:
Thanks, but this did not work. Right now I am doing this in OnNotify:

Ok. I remeber that I had alot of trouble catching the HDN_BEGINTRACK, not to mention the HDN_TRACK (AFAIR I never got it). I know I got the above code to work and Ive glanced through the rest of the code to see if I did anything special somewhere else, but I cant find anything suspicious. Also, that project was for WinCE, but the page that explained how to catch the header notification was for the desktop so I assumed it would work there too. I wish I still had that link, but it was like 6 months ago so it probably went in the trashbin some time ago. Ill do a bit of research after work today and post here if I find anything usefull.

Christian Graus wrote:
and in the end I've concluded that CListCtrl is rubbish.

So true. Smile | :)

Jonas Larsson4-Jan-02 4:26

Jonas Larsson

4-Jan-02 4:26

I did some testing yesterday. I create a subclassed CListCtrl in a childview and the only extra stuff I have is this in OnNotify (see below). It keeps the total column width to 240 the whole time and doesnt flicker at all (as opposed to the CE version of CListCtrl, which decides to draw the bottom scrollbar each time before I get a chance to resize the columns)

The only diffrence with this and the earlier version is that I look for HDN_xxxA and HDN_xxxW instead of just HDN_xxx.

Let me know if you still cant get it to work.

BOOL CNumbersListCtrl::OnNotify(WPARAM wParam, LPARAM lParam, LRESULT* pResult) 
{
    HD_NOTIFY *phdn = (HD_NOTIFY *)lParam; 
    switch(phdn->hdr.code)
    { 		
        case HDN_BEGINTRACKA:
        case HDN_BEGINTRACKW:
        {
            // Trap the HDN_BEGINTRACK message
            TRACE(_T("CNumbersListCtrl::OnNotify -- HDN_BEGINTRACK Trapped\n"));
            SetRedraw(false);
            // *pResult = 1;  // Set to 1 and return TRUE to prevent tracking 
            return(FALSE);  // return FALSE to allow tracking of the divider 
            break; 		
        }
        case HDN_ENDTRACKA:
        case HDN_ENDTRACKW:
        {
            int windowsize = 225;   
            TRACE(_T("CNumbersListCtrl::OnNotify -- HDN_ENDTRACK Trapped\n"));

            SCROLLINFO si; // check if scrollbar is visible
            if (GetScrollInfo(SB_VERT, &si))
            {
                if ((int) si.nPage <= (si.nMax-si.nMin))
                    windowsize = 225;
                else
                    windowsize = 240;
            }
            else
                windowsize = 240;
			
            if (phdn->iItem == 0)
	    {
                int left = phdn->pitem->cxy;
                TRACE(_T("CNumbersListCtrl::OnNotify -- Resize right to fit\n"));
                SetColumnWidth(1, windowsize - left);
            }
            else if (phdn->iItem == 1)
            {
                int right = phdn->pitem->cxy;
                TRACE(_T("CNumbersListCtrl::OnNotify -- Resize left to fit\n"));
                SetColumnWidth(0, windowsize - right);
            }	
            SetRedraw(true);
            Invalidate();
            break;
        }
    }	 	
    return CListCtrl::OnNotify(wParam, lParam, pResult); 

}

Christian Graus4-Jan-02 10:36