On Mon, 22 Mar 2010 20:22:02 -0700 Kristen Carlson Accardi <[email protected]> wrote:
> On Mon, 22 Mar 2010 19:22:00 -0700 > Marcel Holtmann <[email protected]> wrote: > > > Hi Kristen, > > > > > > Changes since v2 - corrected whitespace. > > > > > > all the patches have been applied. > > > > > > So form now on please send smaller patches as soon as you have them > > > ready and tested. If you need comments or have questions, you can use > > > the mailing list as well. > > > > some additional comments from my side. > > > > If you have variables that are only valid inside the scope of a > > statement, then I prefer if you only declare them there. This makes the > > code a bit simpler to read. Check the patch I just pushed to give you an > > example. > > > > Also please find access to a 32-bit system if you are running 64-bit or > > vice versa. Our build system is strict and will turn turn all casting > > mistakes in an error. > > > > Please be present on #ofono IRC as much as possible so people can report > > such things. Remember that your code is now part of the tree and it can > > break the build. > > > > Regards > > > > Marcel > > > > > > Marcel - I just started reviewing the changes you made and the first > patch I hit I discovered that you broke something. Would you like to > revert the patches and let me review them first, or should I just send > a patch to the mailing list to fix what you broke? Here's the commits to revert: a09d38643b8da50c8703fd843be26553f294f0bb e396b7d5afcce6d90d5836d57f7b800a1b6b7b52 I'd be glad to clean them up for you later if you like. _______________________________________________ ofono mailing list [email protected] http://lists.ofono.org/listinfo/ofono
