C / C++ / MFC

Michael Dunn30-Dec-01 19:16

Michael Dunn

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

Michael Dunn

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