Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some feedback #1

Open
rcmaehl opened this issue Nov 27, 2021 · 1 comment
Open

Some feedback #1

rcmaehl opened this issue Nov 27, 2021 · 1 comment

Comments

@rcmaehl
Copy link

rcmaehl commented Nov 27, 2021

I'm off to grab lunch so I'll review more later but the project looks great. There's some additional feedback I have on reducing (or adjusting) AV detections and some personal knowledge having used AutoIt for about a decade now.

README

Unfortunately, you can't use AutoIt3.exe to execute an AutoIt script with includes in it since the included files won't exist on systems without AutoIt installed.

This is incorrect. Check out the Au3Stripper utility included in SciTE4Autoit. The output _stripped.au3 file should be executable without needing to compile to .a3x

ClipboardMonitor.au3

Swap to _ClipBoard_IsFormatAvailable / _ClipBoard_GetDataEx as this shows AutoIt can grab more than just text from the clipboard

Kelogger.au3

_IsPressed is intentionally nerfed. It works okay, but is not the best implementation. I have an example SOMEWHERE that I can send you privately

@V1V1
Copy link
Owner

V1V1 commented Nov 28, 2021

Hey there :)

Just checked out your profile and projects and I'd love to hear more of your feedback,
Thanks a lot for the tips! I'll check your recommendations out during the week and update the repo as soon as I can.

If you'd like to send me anything privately, you can reach me on gabriel[at]thevivi.net.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants