kallisti5@xxxxxxxxxxx wrote: > + */ > + > #include <stdlib.h> Two blank lines. > + fOnOff = new BButton(r, "onoff", "Disable", > + new BMessage(ONOFF_INTERFACE_MSG), > + B_FOLLOW_BOTTOM | B_FOLLOW_LEFT); Wrong indentation. > + } else { > + owner->SetHighColor(owner->LowColor()); > + } Still superfluous brackets. > +/* > + * Copyright 2004-2011 Haiku Inc. All rights reserved. > + * Distributed under the terms of the MIT License. > + * > + */ > + > #ifndef NETWORKSETUPADDON_H > #define NETWORKSETUPADDON_H No blank line here! > + */ > + > #include <stdio.h> But two here. Your driver contains also lots of those, btw. Bye, Axel.