On Thu, 07 Sep 2006 16:40:05 +0200
Jouke Rensma <[EMAIL PROTECTED]> wrote:

> Hello, 
> 
> So I got things working now! The code is in the enclosed (zipped)
> patch. 
> 
> I realize it needs some finetuning, but the basic logic does what I
> had in mind. 
> 
> Now I think it needs some testing, in order to get it ready for the
> world. Some questions and remarks: 
> 
>  - I had to create an extra form for the projectoptions. Can it stay
> like this or are there other possibilities/ideas?
>  - I created a check for the targetos, so it should not interfere
> non-win32 systems, but I cannot test that right now.
>  - the rewriting of the .rc file is not protected with try yet.
> That's my first change... 

Thanks. Applied.

 
> Any other remarks and suggestions are welcome. 

Project.Modified is a property. You can not give it to a 'var'
parameter. It's a compiler bug to allow that. I replaced
Project.Modified. 
Possible improvement: Add a Modified property to the version info.

Minors:
The lazarus code should have a maximum line length of 80.
Try to avoid nested comments.


Mattias

_________________________________________________________________
     To unsubscribe: mail [EMAIL PROTECTED] with
                "unsubscribe" as the Subject
   archives at http://www.lazarus.freepascal.org/mailarchives

Reply via email to