Ashish.As.Gupta at nokia.com wrote:
> Thanks for the comments. Please see inline response within [].
> 
> -----Original Message-----
> From: ext Greg Wright [mailto:gwright at real.com]
> Sent: 18 August, 2005 21:52
> To: Gupta Ashish.As (Nokia-TP-MSW/Irving);
> client-dev at helixcommunity.org; common
> Subject: Re: [Client-dev] CR:new network API implementation for Symbian
> 
> 
> This CR covers *ONLY* the diffs sent in a 4 of the 10 new files:
> 
>  > common/netio/platform/symbian/hxsymbiansocket.cpp
>  > common/netio/pub/platform/symbian/hxsymbiansocket.h
>  > client/netwksvc/platform/symbian/hxsymbianapmanao.cpp
>  > client/netwksvc/platform/symbian/hxsymbianapmanao.h>
> 
> We will get to the rest of these files tomorrow:
> 
>  > common/netio/platform/symbian/hxsymbiannet.cpp
>  > common/netio/platform/symbian/hxsymbianresolver.cpp
>  > common/netio/platform/symbian/hxsymbiansockaddr.cpp
>  > common/netio/pub/platform/symbian/hxsymbiannet.h
>  > common/netio/pub/platform/symbian/hxsymbianresolver.h
>  > common/netio/pub/platform/symbian/hxsymbiansockaddr.h
> 
> 
> 
> First, a general question. Will all these new files compile under
> all versions of Symbian, if not, which are the requirments?
> 
> We can certainly drop support for 6.1, I would like to keep support
> for 70s and above if we can. Any problems here supporting 70s? If
> so, how big are they and can we work around it?
> 
> [It compiles for 70s, 8, 9.1 and works. It will not compile for 6.1.
> ]

Cool, we don't have a problem dropping support for 6.1 on the HEAD.


> STDMETHODIMP
> CHXSymbianSocket::SetOption(HXSockOpt name, UINT32 val)
> {
>      DPRINTF(D_INFO, ("CHXSymbianSocket::SetOption name=%d val=%d\n", name, val));
>      // XXXAG not all options are implemented. add other options as required.
>      switch (name)
>      ...
> }

Ahhh, OK, that is fair. I thought we just committed a CR that
got hxlog working on symbian though. I will have to go look
through my mail. You might just try building it again yourself
as well.


Thanks for the new networking code,
--greg.