Jump to content


Photo

[module] Dlg 5.01


  • Please log in to reply
37 replies to this topic

#1 majkinetor

majkinetor
  • Fellows
  • 4511 posts

Posted 05 March 2007 - 01:16 PM

OUTDATED. Dlg is now maintained via Formsframework.

Common Dialogs

Download          Documentation



#2 majkinetor

majkinetor
  • Fellows
  • 4511 posts

Posted 05 March 2007 - 01:21 PM

One strange thing happening. If you check the code you will see lines:

VarSetCapacity(i, 4, 0)
InsertInteger(idx, i, 0)
r := DllCall("shell32.dll\PickIconDlg", "uint", hGui, "uint", &wIcon, "uint", size, "int", &i)
if I do this without VarSetCapacity and InsertInteger it will not work, the windows will receive ascii codes. Like, if you set idx := 1 31st icon will be selected as ASCII(1) = 31.

Now, this is strange, I expect dll call to handle this as I specify "uint" as an parameter.

#3 Sean

Sean
  • Members
  • 2462 posts

Posted 05 March 2007 - 01:37 PM

This is normal. You only give the address of i, and i is stored as a string. Use a pointer instead: "intP", i like in
<!-- m -->http://www.autohotke...topic16755.html<!-- m -->

#4 majkinetor

majkinetor
  • Fellows
  • 4511 posts

Posted 05 March 2007 - 02:02 PM

ah... my mistake.. i somehow confused uint with intp

Hmm.. something is wrong with the function. Suddenly it stoped returning new path upon selection.... The index is returned correctly but the path stays the same as on input for some reason, even if you change it.

#5 Sean

Sean
  • Members
  • 2462 posts

Posted 05 March 2007 - 02:43 PM

Looks like the script doesn't count the null terminator character at all. Then, lstrlen function might not function as expected. That could be a cause.

#6 majkinetor

majkinetor
  • Fellows
  • 4511 posts

Posted 05 March 2007 - 02:46 PM

Argh I must get some sleep..

I changed the func, and left something that was there in my initial runs.. count of chars that i removed latter...


Btw, it counts null char, that is the reason for 1025 ~ Max_PATH*2 + 1

ok, just redownload

#7 Sean

Sean
  • Members
  • 2462 posts

Posted 05 March 2007 - 03:09 PM

I didn't mean the buffer length, 1025 + 1 is far more than enough mostly even with Unicode function, I suppose. I meant the parameter like StrLen() etc. To be safer, they should be like StrLen() + 1 etc IMO.

#8 majkinetor

majkinetor
  • Fellows
  • 4511 posts

Posted 05 March 2007 - 03:30 PM

well, msdn explicitely states where it requires null char counted and where not.

#9 majkinetor

majkinetor
  • Fellows
  • 4511 posts

Posted 05 March 2007 - 07:37 PM

I reordered things a little

#10 n-l-i-d

n-l-i-d
  • Guests

Posted 05 March 2007 - 10:14 PM

Very cool! 8)

Icon doesn't insert correctly into your example for me.


Wondering... Does anybody know what the exact system requirements are for these emerging COM/OLE scripts? Does it need NT or IE or anything?

#11 majkinetor

majkinetor
  • Fellows
  • 4511 posts

Posted 05 March 2007 - 10:54 PM

I tried this and it showed me folder icon

s := "c:\windows\system32\shell32.dll"
idx := 4
ChooseIcon(s, idx)

About COM, nothing special, just IE.

#12 n-l-i-d

n-l-i-d
  • Guests

Posted 05 March 2007 - 10:57 PM

Ok, I see, your example does not add the actual selected icon, only the first.

I guess COM/OLE needs IE3, from reading a bit, and it should be available from Win95 on... Now I'm still wondering if all the COM scripts here actually would work on a Win95 system with IE3.

#13 Sean

Sean
  • Members
  • 2462 posts

Posted 05 March 2007 - 10:58 PM

Btw, it counts null char, that is the reason for 1025 ~ Max_PATH*2 + 1

Hmm, where did you get MAX_PATH is 512?
AFAIK, MAX_PATH is 260 = 3 + 256 + 1, and the last 1 is indeed for the terminating null (:the first 3 is for drive like C:\).

#14 Sean

Sean
  • Members
  • 2462 posts

Posted 05 March 2007 - 11:02 PM

I guess COM/OLE needs IE3, from reading a bit, and it should be available from Win95 on... Now I'm still wondering if all the COM scripts here actually would work on a Win95 system with IE3.

I'm afraid that you're a bit confused. There is no COM here, comdlg32.dll is Common Dialog (:there is indeed a custom comdlg32.ocx which is used by VB).

#15 majkinetor

majkinetor
  • Fellows
  • 4511 posts

Posted 06 March 2007 - 08:38 AM

well, max path is 260 TCHARS AFAIK. SO, in unicode terms it is double. Why I used double of double, don't ask... was very sleepy and crashed yesterday, I made bunch of trivial errors. I didn't use 260, but degree of 2 as it is the fastest to alocate buffers with such size.