RE: Phone detection changes
"Simon C" <[email protected]>
| Newsgroups | gmane.comp.mobile.bitpim.devel |
|---|---|
| Message-ID | <000d01c636af$964d41a0$0201a8c0@HOME> |
> >Are you going to do this? It is a little frustrating suggesting a > >feature, getting positive feedback from other team members, > >implementing it and then having it removed without any real > discussion. > > I think it's just a miscommunication. We did discuss it, and > you stated that you weren't able to make the build, so I went > ahead and worked it. Maybe I should have made it clear that > I was doing it. Good lesson learned for future builds. I never agreed that to change the way I implemented things. I know you preferred to see it done a different way, but it that was a style issue, there was nothing faulty with the way the feature was implemented there was no reason to believe that it would not work. The way it stands now I have no idea how to go ahead and make changes for this feature. Adding auto detect to the first time startup will cause the detection to run twice once as the code change you ended up making will be triggered as the main screen starts up, fixable with more code of course. I'm also not sure that if you don't like the way I implement it that you won't go ahead and change it anyway as you did here. Simon ------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Do you grep through log files for problems? Stop! Download the new AJAX search engine that makes searching your log files as easy as surfing the web. DOWNLOAD SPLUNK! http://sel.as-us.falkag.net/sel?cmd=lnk&kid=103432&bid=230486&dat=121642