Re: [Evolution-hackers] vcard 2.1 implementation

[Armin Bauer <azrael desscon com>]

On Thu, 2005-01-27 at 09:09, Sivaiah Nallagatla wrote:
> On Wed, 2005-01-26 at 11:14 +0100, Armin Bauer wrote: 
> > On Wed, 2005-01-26 at 07:10, JP Rosevear wrote:
> > > On Tue, 2005-01-25 at 12:28 +0100, Armin Bauer wrote:
> > > > Hi,
> > > > 
> > > > i was just wondering if anyone is working on the task to implement vcard
> > > > 2.1 support in e-vcard.c? If not i will work on this.
> > > 
> > > I don't know of anyone currently working on this, so feel free to take a
> > > crack at this.
> > > 
> > 
> > Here is the patch. It adds the capability to generate 2.1 cards, encode
> > and decode quoted-printable and fixes a bug. it still has some quirks
> > (for example len in quoted_decode and encode is ignored). i will keep
> > you updated when i improve this.
> > 
> > Attached is also a vcard that made the vcard parser run in a loop. thats
> > what the lp++; is for.
> 
> Great Work !!, Probably you can use stuff under addressbook/tests/vcard
> as a test suite as you do your work. Vcard parsing going to loop is
> http://bugzilla.ximian.com/show_bug.cgi?id=70918.  Apart from the
> missing lp++ we need to add some skip_to_next_line calls in
> read_attribute_params to take care of vcards like the second one
> attached in the bug report. Will have a look at your patch
> 

Is there a specific reason e-vcard.c can only parse vcards (besides the
name)? I wanted to use it to parse other stuff as well (like vcal,
vnotes etc), but it only seems to handle vcards even though the BNR of
all is the same:

contentline  = [group "."] name *(";" param ) ":" value CRLF

Why not implement it as a generic parser that understands the vformat
and implement some more convinient "interfaces" to this parser for the
vcard, vcal and vnote standard that would then handle the specific
stuff?

> Thanks, 
> Siva
> 
> > > > Did you think about making a vcard parsing library out of this? There is
> > > > currently no usable vcard lib.
> > > 
> > > Standalone you mean? I don't think we'd really thought about that much.
> > > 
> > > -JP
> _______________________________________________
> evolution-hackers maillist  -  evolution-hackers lists ximian com
> http://lists.ximian.com/mailman/listinfo/evolution-hackers