feedreader-development — Feedreader development list

[Feedreader-development] Properties dialog is "forgetting" proxy settings

[Miha R. <pb...@mi...>]

Proxy settings are currently completely ignored when dialog window closes.
There are several assignment commands missing in
TAppOptionsDialog.btnOKClick procedure. Here's patch to copy proxy settings
from dialog controls to global settings.

Index: AppOptionsDialog_FRM.pas
===================================================================
RCS file: /cvsroot/feedreader/feedreader/AppOptionsDialog_FRM.pas,v
retrieving revision 1.6
diff -r1.6 AppOptionsDialog_FRM.pas
374a375,379
>   gProperties.ProxyEnabled := enableproxy.checked;
>   gProperties.ProxyHost := proxyhost.text;
>   gProperties.ProxyPort := proxyport.text;
>   gProperties.ProxyUsername := proxyusername.text;
>


Also, ProxyPassword is not stored anywhere (currently, I can see that one
line is commented out). What are current plans about storing password to
registry?

--
Miha Remec

[Feedreader-development] patch to fix buffer overrun in MdHintWn.pas

[Miha R. <pb...@mi...>]

Hi!

Here's patch to fix possible buffer overruns that could occur in
TMdHintWindow/TMdBubbleHintWindow classes. I found three calls to StrPCopy
without any parameter length checking.


Index: MdHintWn.pas
===================================================================
RCS file: /cvsroot/feedreader/feedreader/MdHintWn.pas,v
retrieving revision 1.2
diff -r1.2 MdHintWn.pas
334c334
<   StrPCopy(CCaption, Caption);
---
>   StrLCopy(CCaption, PChar(Caption), SizeOf(CCaption) - 1);
435c435
<   StrPCopy(CCaption, Caption);
---
>   StrLCopy(CCaption, PChar(Caption), SizeOf(CCaption) - 1);
616c616
<   StrPCopy(CCaption, Caption);
---
>   StrLCopy(CCaption, PChar(Caption), SizeOf(CCaption) - 1);


--
Miha Remec

[Feedreader-development] [Fwd: FeedReader development]

[Toomas T. <to...@i-...>]

Welcoming new member...

[Feedreader-development] Changing DFM to text in CVS

[Kyle C. <ky...@ky...>]

Marcus Hettlage had asked for a tip on how to change those DFMs to text in
CVS.

Marcus, I tried to reply to you directly, but got an email error:

------------
A message that you sent could not be delivered to one or more of its
recipients. This is a permanent error. The following address(es) failed:

  hm...@so...
    unknown local-part "hmarcus" in domain "sourceforge.net"
--------------------

Here is the message I was trying to send:


Here is how to fix it.  You have to have appropriate rights in CVS, which
you probable already have.  Run this command, in your local cvs workspace
for the feedreader project:

cvs admin -kv filename.dfm

It is possible that the line ending will be messed up by this, they may get
an extra CR at the end.   If so, you'll need to do an "update", then fix
them (many text editors make this very easy) and a "commit".

There is a good page on SourceForge with lots of CVS information:

https://sourceforge.net/docman/display_doc.php?docid=768&group_id=1

Let me know if you have any trouble, I will try to help.

Kyle

[Feedreader-development] DFM files

[Kyle C. <ky...@ky...>]

I think I posted a few days ago, about the DFM files being marked binary in
CVS, making it not merge.

This bit me today; I had made some tweaks to the DFM here, but CVS would not
merge them with the incoming changes from the updated file in CVS.


[ Kyle Cordes * ky...@ky... * http://kylecordes.com ]
[ Consulting, Training, and Software development tips and   ]
[ techniques: Java, Delphi, ASTA, BDE Alternatives Guide,   ]
[ JB Open Tools, EJB, Web applications, methodologies, etc. ]

[Feedreader-development] DFM files

[Kyle C. <ky...@ky...>]

It looks like the DFM files are (correctly) stored as Text DFMs, but are
(incorrectly) marked in CVS as Binary.

That's a bad thing, it will make CVS handle merges poorly.

[ Kyle Cordes * ky...@ky... * http://kylecordes.com ]
[ Consulting, Training, and Software development tips and   ]
[ techniques: Java, Delphi, ASTA, BDE Alternatives Guide,   ]
[ JB Open Tools, EJB, Web applications, methodologies, etc. ]

[Feedreader-development] [Fwd: Re: A patch for multi-monitor bug]

[Toomas T. <to...@i-...>]

Forwared to list....

[Feedreader-development] patch to fix bug 198

[Kyle C. <ky...@ky...>]

Here is a patch for the multi-monitor bug I reported on the feedreader web
site, bug # 198:



cvs server: Diffing .
Index: MainForm_FRM.dfm
===================================================================
RCS file: /cvsroot/feedreader/feedreader/MainForm_FRM.dfm,v
retrieving revision 1.1
diff -r1.1 MainForm_FRM.dfm
15c15
<   Position = poDesktopCenter
---
>   Position = poScreenCenter




[ Kyle Cordes * ky...@ky... * http://kylecordes.com ]
[ Consulting, Training, and Software development tips and   ]
[ techniques: Java, Delphi, ASTA, BDE Alternatives Guide,   ]
[ JB Open Tools, EJB, Web applications, methodologies, etc. ]

Re: [Feedreader-development] Re: I'm ready?

[Toomas T. <to...@i-...>]

New version file location is

http://feedreader.sourceforge.net/version.txt

I guess I can send you new about and splash dialog descriptions today 
evening.


Greetings
Toomas

[Feedreader-development] Re: I'm ready?

[Toomas T. <to...@i-...>]

Marcus Hettlage wrote:

> Hi Toomas,
>  
> I think I've reached a point of developement where I can give the 
> source back to the public.
> I'd like to send it to you and you should put it back into the CVS.

I think that it's good for you to install some kind of wincvs software 
and commit them yourself. It's good because so we (and others) can start 
codevloping at the same time. I have installed tortoisecvs programm and 
it's really simple and straightforward to configure and use. Or if you 
are against this idea i will of course put files to cvs myself.

>  
> There are still some points that have to be done:
>  
> 1. Splashscreen needs updated copyright, etc.

I will do them

> 2. About-dialog needs updated copyright, etc.

Same...

> 3. You should update the version.txt file on your server: you now have 
> to use the complete version-number, i.e. "2.5.0.483"

At the moment this files is in www.postimees.ee but we should change 
code to get this file from feedreader.sourceforge.com/version.txt for 
example

> 4. bug fixing: there are still some exceptions occurring if you run it 
> inside the IDE that I haven't yet fixed

Ok

> 5. we should change the internal data-handling to use objects and 
> collections instead of records and arrays, that makes a lot of things 
> easier and
> reduces unnecessary copying of data and removes some memory-leaks that 
> are present in the code

These datastructures are new to me.  So maybe you can start the process 
and i can of course go on myself, if i know how to do it.

> 6. some forms still access the mainwindow directly, this should be 
> changed to a controlling mechanism that is initaited from the mainform

ok.

> 7. new features ?
>  

I guess, let's put this version up to sourceforge as download (binaries 
and install) and then start implementing new features. At the moment 
there is no download files and that drives people away from our project.

> What about the co-branding and releasing in general?
> I think we should offer a binary distribution with a setup that 
> displays the license, etc.
> This looks much better than just the binary, which could be offered as 
> well.
> And I still think in an installation should the translation files be 
> included.
>  

I think, that's ok. Where's the source? ;)

> What do you think?
>  
> Greetings
>   Marcus
>  
>  

[Feedreader-development] Re: Feedreader Co-Branding]

[Toomas T. <to...@i-...>]

What do you think, should we put your version to CVS before that? What 
comes to Splashscreen I'd like to have it like photoshop for example. 
Some kind of nice surreal graphics. I have a friend, who maybe can do it.

Greetings
Toomas

Marcus Hettlage wrote:

>can you please finish the about and splash-screen?
>I think you know better than me who has helped in the past to develop
>the software, etc.
>
>Greetings
>  Marcus
>
>
>----- Original Message -----
>From: "Toomas Toots" <to...@i-...>
>To: "Marcus Hettlage" <ma...@he...>
>Sent: Wednesday, February 12, 2003 9:43 PM
>Subject: Re: Feedreader Co-Branding]
>
>
>  
>
>>Marcus Hettlage wrote:
>>
>>    
>>
>>>Can you give me a URL with which I can test the amphetadesk
>>>      
>>>
>compatibility?
>  
>
>>>Greetings
>>> Marcus
>>>
>>>
>>>      
>>>
>>Here you go!
>>
>>http://www.disobey.com/amphetadesk/website_integration.html
>>
>>Toomas
>>
>>    
>>

Re: [Feedreader-development] HI ALL!

[Marcus H. <mar...@ep...>]

Hi!

We're currently in the process of refactoring the software.
I think I'm through with it in the next few days.
I already have a big list of things we could do.
I think we should wait 'til I'm through wit it.

Greetings
  Marcus


----- Original Message ----- 
From: "Brizuela Ezequiel Hector" <bri...@TE...>
To: <fee...@li...>
Sent: Wednesday, February 12, 2003 9:33 PM
Subject: [Feedreader-development] HI ALL!


> Hi!, i'm new here...
> I'm a developer and I really wanna help U tom.
> Please, can we coordinate something?.
> Bye.
> 
> QliX=D!
> 
> 
> -------------------------------------------------------
> This sf.net email is sponsored by:ThinkGeek
> Welcome to geek heaven.
> http://thinkgeek.com/sf
> _______________________________________________
> Feedreader-development mailing list
> Fee...@li...
> https://lists.sourceforge.net/lists/listinfo/feedreader-development

[Feedreader-development] HI ALL!

[Brizuela E. H. <bri...@TE...>]

Hi!, i'm new here...
I'm a developer and I really wanna help U tom.
Please, can we coordinate something?.
Bye.

QliX=D!

[Feedreader-development] Older files

[Marcus H. <ma...@he...>]

Hi!

I've fixed the problem with the wrong encoding of the old files.
If a conversion error occurrs the UTF-8 encoding is replaced with a =
windows-1250.
If that does not work, the user has bad luck.

Greetings
  Marcus

Re: [Feedreader-development] Problem

[Marcus H. <mar...@ep...>]

----- Original Message -----
From: "Toomas Toots" <to...@i-...>
To: "Feedreader Development" <fee...@li...>
Sent: Tuesday, February 11, 2003 10:54 PM
Subject: Re: [Feedreader-development] Problem


> Marcus Hettlage wrote:
>
> > The current developement version of FeedReader uses only the OpenXML
> > XDOM-Parser.
> > This parser is stricter than the ICXML-parser that was used before.
> > This results in the situation, that some XML-files can't be parsed,
> > i.e. loaded and used.
> > This applies for example to the german-translation file where "umlaut
> > U" and other
> > german specific characters are used that are not properly encoded
> > according to the
> > procssing instruction at the beginning of the file. The file says that
> > it's UTF-8
> > where in reality it's "Windows 12xx".
>
> This is not problem at all. Users download files from www.feedreader.com
> and downloading is being managed by php script  where at the moment is
> print "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n..... It's quite easy
> for me to change this. Only problem is that i do not know right
> encodings. I can put diferent encoding for every translation file, if
> somebody gives me right encodings.

This should not be too hard to find out. I'll check them and tell you what
encoding we need.

>
> > This applies also to the headlines.xml and other files.
> > What do you think is the best and easiest way to fix it?
>
> What do you think is the right encoding for headlines,*.xml files? One
> possibility is to check for parsing error and if parsing error occures,
> then manually do string replace on  xml stream? From utf-8 to something
> new so that xml parser will not complain?

That is the same solution I'd come up with. I think I'll do it that way.

>
> Can you tell me, what kind of encoding is utf-8? What are right and what
> are not right characters for this encoding. And if somebody has japanese
> characters and german characters at the same time, what kind of encoding
> is this?

have a look at http://www.cis.ohio-state.edu/cgi-bin/rfc/rfc2279.html

Greetings
  Marcus

Re: [Feedreader-development] One annoying thing

[Marcus H. <mar...@ep...>]

> Hi!
> 
> There is one more annoying thing I could not get rid of.
> 
> If there are some new headlines and "new headline notification" window 
> pops up in the right lower corner of desktop this form is visible on 
> taskbar, too. It should not be there. I tried and tried but could not 
> get popups showing without this taskbar presentation.
> 

I'll keep an eye on it.

Marcus

[Feedreader-development] One annoying thing

[Toomas T. <to...@i-...>]

Hi!

There is one more annoying thing I could not get rid of.

If there are some new headlines and "new headline notification" window 
pops up in the right lower corner of desktop this form is visible on 
taskbar, too. It should not be there. I tried and tried but could not 
get popups showing without this taskbar presentation.


Greetings
Toomas

Re: [Feedreader-development] Problem

[Toomas T. <to...@i-...>]

Marcus Hettlage wrote:

> The current developement version of FeedReader uses only the OpenXML 
> XDOM-Parser.
> This parser is stricter than the ICXML-parser that was used before.
> This results in the situation, that some XML-files can't be parsed, 
> i.e. loaded and used.
> This applies for example to the german-translation file where "umlaut 
> U" and other
> german specific characters are used that are not properly encoded 
> according to the
> procssing instruction at the beginning of the file. The file says that 
> it's UTF-8
> where in reality it's "Windows 12xx".

This is not problem at all. Users download files from www.feedreader.com 
and downloading is being managed by php script  where at the moment is 
print "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n..... It's quite easy 
for me to change this. Only problem is that i do not know right 
encodings. I can put diferent encoding for every translation file, if 
somebody gives me right encodings.

> This applies also to the headlines.xml and other files.
> What do you think is the best and easiest way to fix it?

What do you think is the right encoding for headlines,*.xml files? One 
possibility is to check for parsing error and if parsing error occures, 
then manually do string replace on  xml stream? From utf-8 to something 
new so that xml parser will not complain?

Can you tell me, what kind of encoding is utf-8? What are right and what 
are not right characters for this encoding. And if somebody has japanese 
characters and german characters at the same time, what kind of encoding 
is this?

> But, please remember this code is used only once for older installations,
> where these files already exist, and only for the first time the new .EXE
> is started.
>  
>     Marcus

Greetings
Toomas

[Feedreader-development] Problem

[Marcus H. <ma...@he...>]

The current developement version of FeedReader uses only the OpenXML =
XDOM-Parser.
This parser is stricter than the ICXML-parser that was used before.=20
This results in the situation, that some XML-files can't be parsed, i.e. =
loaded and used.
This applies for example to the german-translation file where "umlaut U" =
and other=20
german specific characters are used that are not properly encoded =
according to the
procssing instruction at the beginning of the file. The file says that =
it's UTF-8
where in reality it's "Windows 12xx".
This applies also to the headlines.xml and other files.
What do you think is the best and easiest way to fix it?
But, please remember this code is used only once for older =
installations,
where these files already exist, and only for the first time the new =
.EXE
is started.

    Marcus

Re: [Feedreader-development] Further Developement

[Marcus H. <mar...@ep...>]

    The next things I'm going to do is change the handling of the
    display in the TreeView. I'd like to see an option to enable a bold =
font
    for feeds that have new messages, i.e. like Outlook Express.
  This is the thing I thought about, too. Only problem, is that I could =
not get standard treeview to do that. Extremely cool treeview component =
is here:http://www.delphi-gems.com/VirtualTreeview/VT.php , but it's not =
dropin replacement.

    OK, that's working...., but I've used the standard TreeView-control.

    Another thing I'd like to add is a kind of "import the browser =
favorites"
    and check them for updates as well. For me it would be sufficient
    if I knew that a page was changed. I don't need more special logic
    in this area. But maybe someone else?
  What do you think, if we modularize feedreader. So we have one module =
for "web page updates" and second module for example for nntp =
(newsgroups) servers and so on. Feedreader will only trigger update =
procedure and shows headlines from modules.

    This sounds interresting, but I think we should first finish the =
tasks I've started.
    That should give us an easier basis to implement these things.

    What compilers will we support?
    I've currently only Delphi 7 Pro available. What about you?

  I have Delphi 6 Ent,  but the nicest thing will be, if we could get =
Feedreader compiling on Delphi 7 personal. In my opinion users could  =
use it for free.

              I think it's a good idea to support Delphi 7 personal. It =
should be possible, I'll try it next week.

Re: [Feedreader-development] Language files

[Marcus H. <mar...@ep...>]

I think we should export them on a regular basis and install them
together with the application.
the user should be able to change the language through the =
options-dialog.
  ----- Original Message -----=20
  From: Toomas Toots=20
  Cc: fee...@li...=20
  Sent: Wednesday, February 05, 2003 12:32 PM
  Subject: Re: [Feedreader-development] Language files


  Marcus Hettlage wrote:

    What about the translations?
    Shouldn't these files be in the CVS as well?
    Or are they copyrighted and I missed it?

  These language files are situated on www.feedreader.com site. Problem =
is that they are in SQL database. There is a special page where =
translation can be inputed and modified. I guess, there is no need to =
put translation files to cvs.=20


  Greetings
  Toomas

Re: [Feedreader-development] Further Developement

[Toomas T. <to...@i-...>]

  Marcus Hettlage wrote:

> Hi,
>  
> I've started to reorganize the released Source-Code.
> I started by removing the ICOM-XML-parser. The code
> now uses only the XDOM-parser

>  
> Another component I've already exchanged is the Tray-Icon-component.
> I replaced it with CollTrayIcon, because this is available with Source.
>  

That's cool. Then I choosed tnotificationicon component, there was 
something that made me not choose cooltrayicon. I guess, it was this 
bubble notification. But at the moment, I have made feedreaders own 
notification dialog, there is problems any more.

> The next thing I did was to start restructuring the code. I started to
> eliminate unused code and started to restructure it, so that it can
> be extended a little bit easier.

Ehh... That's me. Actually this is quite only program I developed in 
Delphi, that's why I maybe did not use the best technicues. Second 
problem is lack of comments. Code is quite ;) understandable to me, but 
others are in little trouble i guess.

>  
> The next things I'm going to do is change the handling of the
> display in the TreeView. I'd like to see an option to enable a bold font
> for feeds that have new messages, i.e. like Outlook Express.

This is the thing I thought about, too. Only problem, is that I could 
not get standard treeview to do that. Extremely cool treeview component 
is here:http://www.delphi-gems.com/VirtualTreeview/VT.php , but it's not 
dropin replacement.

>  
> Another thing I'd like to add is a kind of "import the browser favorites"
> and check them for updates as well. For me it would be sufficient
> if I knew that a page was changed. I don't need more special logic
> in this area. But maybe someone else?

What do you think, if we modularize feedreader. So we have one module 
for "web page updates" and second module for example for nntp 
(newsgroups) servers and so on. Feedreader will only trigger update 
procedure and shows headlines from modules.

>  
> I've already implemented a kind of "Newspaper"-feature, that
> displays all unread messages on a single HTML-page.
> This feature is nearly identical to the one in NewzCrawler.

Actually Feedreader has internal webserver, so user could check all 
headlines from another computer (if on the road for example). It's nice 
if we integrate the look of both views (webbrowser view, and internal 
html-page view).

>  
> As I've already said I'm not yet ready with my planned restructuring.
> This will create a lot of new units. I think I should finish this before
> I put the Source back into the CVS. I hope that I'm through until the
> end of the month.

Cool.

>  
> What compilers will we support?
> I've currently only Delphi 7 Pro available. What about you?
>  

I have Delphi 6 Ent,  but the nicest thing will be, if we could get 
Feedreader compiling on Delphi 7 personal. In my opinion users could 
 use it for free.

Greetings!
Toomas

Re: [Feedreader-development] Language files

[Toomas T. <to...@i-...>]

Marcus Hettlage wrote:

> What about the translations?
> Shouldn't these files be in the CVS as well?
> Or are they copyrighted and I missed it?
>  

These language files are situated on www.feedreader.com site. Problem is 
that they are in SQL database. There is a special page where translation 
can be inputed and modified. I guess, there is no need to put 
translation files to cvs.


Greetings
Toomas

[Feedreader-development] Language files

[Marcus H. <ma...@he...>]

What about the translations?
Shouldn't these files be in the CVS as well?
Or are they copyrighted and I missed it?

[Feedreader-development] Further Developement

[Marcus H. <ma...@he...>]

Hi,

I've started to reorganize the released Source-Code.
I started by removing the ICOM-XML-parser. The code
now uses only the XDOM-parser.

Another component I've already exchanged is the Tray-Icon-component.
I replaced it with CollTrayIcon, because this is available with Source.

The next thing I did was to start restructuring the code. I started to=20
eliminate unused code and started to restructure it, so that it can
be extended a little bit easier.

The next things I'm going to do is change the handling of the
display in the TreeView. I'd like to see an option to enable a bold font
for feeds that have new messages, i.e. like Outlook Express.

Another thing I'd like to add is a kind of "import the browser =
favorites"
and check them for updates as well. For me it would be sufficient
if I knew that a page was changed. I don't need more special logic
in this area. But maybe someone else?

I've already implemented a kind of "Newspaper"-feature, that
displays all unread messages on a single HTML-page.
This feature is nearly identical to the one in NewzCrawler.

As I've already said I'm not yet ready with my planned restructuring.
This will create a lot of new units. I think I should finish this before
I put the Source back into the CVS. I hope that I'm through until the
end of the month.

What compilers will we support?
I've currently only Delphi 7 Pro available. What about you?

Any comments?

Greetings
    Marcus