Update for project files

[jeroen]

Can you imagine that it might make a difference for any other source file from the include hierarchy (Microsoft Runtime Library ...)?

Yes, I can imagine that. But as I said, with and without this switch in my Makefile the results are bit for bit the same.

Your header "JkDefragLib.h" will not be usable by other C/C++ projects to import its functions.

Sigh. Not true. Just try it, it works like a charm. There is an example VisualC project included in the sources.

[elfring]

Yes, I can imagine that. But as I said, with and without this switch in my Makefile the results are bit for bit the same.

Well, I just try to remind us that there is the danger that something will go wrong because of a missing preprocessor symbol. An API contract might be violated. Can software be developed in a way to guarantee its safety?

Not true. Just try it, it works like a charm. There is an example VisualC project included in the sources.

I know - it has got the same open issue (no option for a dynamic import) as the old Makefile. I've updated it for my source code branch.
I sent you my proposed changes. Are you going to accept any patches from this feedback?

[jaben]

2 items on this...

1) I agree that this is very overengineered.
2) C++ may need the DLL doe to the difference in the licensing... I cannot use the original if I want to distribute the ending app without also distributing my code (which I am sometimes prohibited from doing) for some clients).

[elfring]

But as I said, with and without this switch in my Makefile the results are bit for bit the same.

It is true at the moment. - I prefer an approach where I can be sure that the required parameters were passed to all tools in the software build process to achieve the correct result.

[jeroen]

Are you going to accept any patches from this feedback?

I have not had time yet to look at them. At the moment all my free time is consumed by answering questions here on the forum.

[elfring]

The parameter /LD is the required switch for the compiler (and indirect also for the linker) to mark a source file for DLL usage.

1) I agree that this is very overengineered.

Would you like to contribute a wording for the Makefile that does not look overengineered with all recommended markers?

[jaben]

You seem like a very knowledable person about makefiles (Let me also say that I dont use them a lot).  I am confused since I always thought engineering started with a clear concise problem statement...  It seems to me that you have proposed lots of solutions to problems that are very nebulous and therefore mostly unknown to me.  I don't understand the reason that you feel that you need a new makefile.

[elfring]

It seems to me that you have proposed lots of solutions to problems that are very nebulous and therefore mostly unknown to me.  I don't understand the reason that you feel that you need a new makefile.

I try once more to explain my reasoning.
1. The specification /LD is missing for the DLL compilation. Now there are different opinions if and when effects by a missing compiler parameter will be noticed in binary object and executable files.
2. The source files "JkDefragLib.cpp" and "ScanNtfs.cpp" must be compiled with the same and specific parameter set for DLL generation. The resulting file "ScanNtfs.obj" must not be reused for the link step to produce "JkDefrag.exe".
=> The Makefile structure should be adjusted.

[jeroen]

1. The specification /LD is missing for the DLL compilation.

You are wrong. Please try the DLL. It works perfectly.

[elfring]

It works perfectly.

... until missing compiler switches will result in unsupported requirements for DLL generation. How likely can it be noticed that an incomplete parameter set for the compilation might be the source of unexpected effects?

[jaben]

Has this happened to you?  

What compiler are you using?  

When I compiled the DLL with VS'05 it seems to work fine.  I did remove the memory leak of course.

[Lexar]

Looks like the VC++ solution needs updating, to include ScanFat.h and ScanFat.cpp for each project.