ML errors

[zedd151]

Looks OK.
You can get rid of the following:

Was only getting the random number generation out of WM_PAINT. Will leave the other fixes to tda0626.

[daydreamer]

Could as well show workaround for one ml error i found out in my program
Its about to compare 4 bytes with one single .if eax==01010101h

.data
Hand1 db 0,0,0,0,0
lea esi,[hand1+ebx*2] ;workaround below error
;mov eax,[hand1+ebx*2];error A2070
mov eax,[esi]

[NoCforMe]

Well, if this threw an error:

Code Select Expand

mov eax,[hand1+ebx*2];error A2070

then how about

Code Select Expand

mov eax,DWORD PTR [hand1+ebx*2]

That should fix it.

[jj2007]

David's solution is correct.

Code Select Expand

.data
Hand1 db 0,0,0,0,0
.code
mov eax, dword ptr [hand1+ebx*2]
What happens here is that you try to a BYTE variable into a DWORD size register. That is technically possible but Masm throws an error, thus warning you that you are trying to do unorthodox things.

Now, if you are convinced that you really want to do that, you can override the error using dword ptr [whatever].

[TimoVJL]

Looks OK.
You can get rid of the following:

Code Select Expand


; Totally unnecessary: CreateWindowEx() will take care of this:
    invoke ShowWindow, hwnd, SW_SHOWNORMAL
    invoke UpdateWindow, hwnd

; but add this style when you create the window:
    WS_OVERLAPPEDWINDOW or WS_VISIBLE,\


Without startup code ShowWindow isn't important.

Example of startup code in C

Code Select Expand

    GetStartupInfo(&si);
    ExitProcess(WinMain(GetModuleHandle(NULL), NULL, szCmdLine,
        (si.dwFlags & STARTF_USESHOWWINDOW) ? si.wShowWindow : SW_SHOWDEFAULT));


[daydreamer]

Thanks david
.data
Ace2 db 0 ;copy of ace here
Twos db 0,0,0,0,0,0,0,0,0,0,0,0
Ace db 0
.code
...previous code checks statistics on number of twos to aces and write to twos db to aces db
Mov ebx,0
M2m ace2,ace
L1:
Mov eax, dword ptr [ace2+ebx]
Mov dl,[ace2+5+ebx]

[jj2007]

M2m ace2,ace

No error? Which macro is that?

[zedd151]

Tim, did you mean to add something? All I see in your post is an oddly  truncated quote from myself, and nothing from you. 

[daydreamer]

M2m ace2,ace

No error? Which macro is that?

sorry forgot m2m memory is made out of push/pop,means it doesnt work with bytes
 

[tda0626]

First, I put the rand routines in WM_CREATE for the initial random number.
Then I also put it in WM_CHAR using the space bar to initiate next random number(s)... all of that plus commented out the random routine from WM_PAINT.

I did not change the positioning of the text, I'll leave it for you. 

There are most likely other ways to do the same. I just offer one example here.    It is not perfect, but should give you some ideas.

Sorry for the long delay, Tim. Real life interferes and I had forgotten about this til just now.
Btw, no ml errors here nor any Defender issues (I'm running Windows 7). 

This gives me some good ideas. I appreciate this. Thank you.


Tim

[tda0626]

Tim, did you mean to add something? All I see in your post is an oddly  truncated quote from myself, and nothing from you. 

Not sure how that happened but asked for deletion.

[NoCforMe]

Not sure how that happened but asked for deletion.

In the future, you can simply edit your posts, at any time after you make them.

[stoo23]

Not sure how that happened but asked for deletion.

In the future, you can simply edit your posts, at any time after you make them.

Yes, in fact I'd have thought you may already have been aware of that ?? 

From the choices (which should be visible in the Lower Right of you Post, (shown below), you can choose either "Quick Edit" or "More", which will allow you to choose 'Modify' to give you Full access to the Post Editor available when you initially wrote your post 

You cannot view this attachment.

[NoCforMe]

[stoo23]

Beat me to it,  ... I was on the phone. ... Thanks