Bug in UDF's, lots of them..

[corz]

I have to admit, I hardly use the UDF's (which is wrong, I know, but that's just me), but I thought I might try out something today. Oh dear!

I'm not going to name functions, nor names, but it wouldn't work, nay, it just messed everything up, until I did this..

AutoItSetOption("GUIOnEventMode", 0)
_SomeGroovyUDFFunction($my_edit_box)
AutoItSetOption("GUIOnEventMode", 1)

I realise that my experience of AutoIt is still limited, and I may be missing something obvious here, but it just seems like it would save a lot of frustration and wasted man-hours if the fix was applied right at source. This goes for tray modes, etc., etc.; anything the UDF needs.

Please put me straight, or otherwise!

;o)

(or

[GaryFrost]

I'm not going to vote, I hate these polls in threads.

If there are "Bugs" in the UDFs that are supplied with AutoIt install, we need to know about them.

We work diligently to fix bugs that are found.

It would help to point out the UDFs that you are referring to.

[corz]

I use the word "bugs" to get folk here. It's probably as much a conceptual issue as any other. Though the end result is exactly the same; you drop the function in your application, and your application stops working.

I'd prefer not to name particular functions, because on looking, I can see that many are affected, but if you insist on an actual example, of course, I'll provide that. Is that necessary?

As for the poll, I hate them, too, but I figured it might be a good way for people to chime in whithout offending anyone - because all the big names are affected!

;o)

(or

[edit: actually, there's only a few in the official "released" UDF's - though many of the common UDF's here on-site also suffer]

Edited October 13, 2007 by corz

[Jos]

I use the word "bugs" to get folk here. It's probably as much a conceptual issue as any other. Though the end result is exactly the same; you drop the function in your application, and your application stops working.

I'd prefer not to name particular functions, because on looking, I can see that many are affected, but if you insist on an actual example, of course, I'll provide that. Is that necessary?

As for the poll, I hate them, too, but I figured it might be a good way for people to chime in whithout offending anyone - because all the big names are affected!

;o)

(or

You already have by stating it the way you did...

constructive ideas/criticism is welcome ... vague polls/proposals are going to lead nowhere ..

my 2 cents

[big_daddy]

You're not going to hurt anyone's feelings by pointing out a bug in their function.

[corz]

"You already have by stating it the way you did... " <- I don't understand that, you quoted my whole post!

I don't think I'm being vague, just kind. I could easily look inside all my own common functions and see if they set options, and I figured I'd leave it at that. I was more interested in knowning if this behaviour was intentional or not. Okay then, a specific example..

_GUICtrlEdit_Find()

It sets the Event mode to 0 for its own purposes, which will break an application using event mode 1.

The Array degugging functions also do this.

These are just ones I've come across in the release UDFs. I haven't looked extensively, or checked for other types of options thjat functions might set, which would be just wasting my time if this behaviour was, in fact, intentional.

;o)

(or

[GaryFrost]

"You already have by stating it the way you did... " <- I don't understand that, you quoted my whole post!

I don't think I'm being vague, just kind. I could easily look inside all my own common functions and see if they set options, and I figured I'd leave it at that. I was more interested in knowning if this behaviour was intentional or not. Okay then, a specific example..

_GUICtrlEdit_Find()

It sets the Event mode to 0 for its own purposes, which will break an application using event mode 1.

The Array degugging functions also do this.

These are just ones I've come across in the release UDFs. I haven't looked extensively, or checked for other types of options thjat functions might set, which would be just wasting my time if this behaviour was, in fact, intentional.

;o)

(or

That's a beta not a release function, anyways, I've looked at both versions, and I don't see anywhere where the Event mode is set.

I personally don't much care for the OnEvent modes, so I don't use them, don't change them.

Now I may be overlooking what you are talking about, and If that is the case I gladly look at changing that, I have no desire to set the event mode.

[big_daddy]

That's a beta not a release function, anyways, I've looked at both versions, and I don't see anywhere where the Event mode is set.

I personally don't much care for the OnEvent modes, so I don't use them, don't change them.

Now I may be overlooking what you are talking about, and If that is the case I gladly look at changing that, I have no desire to set the event mode.

I understand what he is saying. The function he pointed out is using a message loop, but if the OP is using "OnEventMode" the function will fail. As he has already stated the proper way to handle this would be to save the OP's mode setting, set event mode to false, then restore the saved setting before returning.

[corz]

Well, it does this..

$msgFind = GUIGetMsg()

So it *should* be setting the event mode, no?

At any rate, it's likely that line which messes things up if the calling app is running in event mode 1.

Specifically, GUIGetMsg() will always return zero, and you will lose control of the window.

You might be able to kill the app if you right-click the tray icon, if it has one.

The code in my first post fixes the issue.

My suggestion from the first post fixes it better.

;o)

(or

[corz]

Right, BD!

;o)

(or

[GaryFrost]

Easy enough to fix that's 1 out of thousands of functions so far.

[corz]

Your incredible contribution to AutoIt is beyond question, GaryFrost.

Change is inevitable!

I ignorantly assumed that GUIGetMsg() would implicitly set the event mode, but as I was reading the manual (so that I could be specific in my earlier reply) I discovered that it did no such thing, though of course, the end result is the same.

I'm glad to be of help!

;o)

(or

[big_daddy]

Easy enough to fix that's 1 out of thousands of functions so far.

I did a search for "GUIGetMsg" on the latest beta includes directory and came up with two results.

_ArrayDisplay() which already utilizes the proposed fix, and _GUICtrlEdit_Find().

[GaryFrost]

I did a search for "GUIGetMsg" on the latest beta includes directory and came up with two results.

_ArrayDisplay() which already utilizes the proposed fix, and _GUICtrlEdit_Find().

There is an update that has been submitted to fix the problem in the GuiEdit.au3 along with some other fixes in the UDFs.

[corz]

If only Microsoft responded to bugs this quickly!

;o)

(or