On Mon, 2010-07-26 at 11:05 -0700, Denis Kenzior wrote: 
> Hi Inaky,
> 
> >>> +SMS / Messaging interface
> >>> +=========================
> >>> +
> >>> +Service          org.ofono
> >>> +Interface        org.ofono.SMSMessage
> >>
> >> oFono API is full CamelCase, so this should be SmsMessage.
> > 
> > I disagree. It would be like that if SMS was a word, but it is an
> > acronym. It makes it quite confusing, if you ask me. If that's still how
> > you like it, I'll change it, but I think it is wrong to capitalize an
> > acronym.
> > 
> 
> Your objection has been noted.  However, pure camel case looks nicer and
> that is how most of the API already is (e.g. SmsManager, SimManager,
> UnlockPin, etc)  So please change it or name it simply 'Message'.

Changed to SmsMessage as discussed over IRC.

_______________________________________________
ofono mailing list
ofono@ofono.org
http://lists.ofono.org/listinfo/ofono

Reply via email to