🚀 go-pugleaf

mataus wrote:

>Can anyone help me make this fast?

1) You use AnsiString which is often slower than plain cstrings.
2) You Add() to Memo->Lines. This is probably slow.
3) You manually check _every_ file to see if it is a folder or an exe.
4) You ProcessMessages() way too often.
5) You are passed the path which you then mess with and later
ExtractFilePath(sPath) to get back to the passed path.

For #1, Don't be so quick to use AnsiString all over the place.
Use a char array[MAX_PATH+1] for local strings.

For #2, Append() might be faster.  BeginUpdate / EndUpdate around the
entire process might speed up the Memo.

For #3, I would break it into two sections.
First section, FindFirst/Next would specify FILE_ATTRIBUTE_DIRECTORY
in the search attributes. (Handle all folders first, quickly.)
Second section, new FindFirst/Next would specify \*.exe so you only
get those files instead of _every_ file.

For #4, eliminate it from the loop.
If you find you really do need it, then put a counter on it so it
only executes every ~50 times through the loop.

For #5, you could copy the passed path to a local char[] and ensure
it has the terminating \.  Then use that to build the other strings.