#4576: There should be a BTimer ----------------------------------+----------------------------------------- Reporter: joeyadams | Owner: nobody Type: enhancement | Status: new Priority: normal | Milestone: Unscheduled Component: Kits/Application Kit | Version: Keywords: | Blockedby: Patch: 0 | Platform: All Blocking: | ----------------------------------+----------------------------------------- Comment(by eml): I'm not a dev, but I had a quick look at the patch anyways. Seems to me that the implementation does what it should do, however I think you should review the coding guidelines: [http://www.haiku-os.org/development/coding- guidelines Haiku Coding Guidelines] Some examples of 'violations' include member variable names and that a functions return type should be on its own line before the actual declaration. Generally header files use the extension '.h', not '.hpp'. I would suggest putting the public interface first in your class definition as that is the general way it's done throughout the Haiku source code from what I gather. Even though this is a first draft as you say, I would suggest that you write your functions in source files and not in header files. -- Ticket URL: <http://dev.haiku-os.org/ticket/4576#comment:2> Haiku <http://dev.haiku-os.org> Haiku - the operating system.