rbmj/612-code — comment on commit 8b083c467ed46dc907ec68450a8b12c1e6b17ad4
File: joysmooth.cpp line 1
By rbmj:

Do try and compile code before you commit it... there were three missing
semicolons, two missing braces, and an invalid declaration in the last
bit of code. I'm too lazy to see if you were the person who wrote it or
if you merged someone else's code, but even so, once you merge just run
a quick make. We want any valid commit in the tree to be a starting
place for a new build.

I'm going to check it during LS tomorrow. I'm going to bed.

It took quite a bit of self-discipline to keep from ranting... tired
+stressed + swim practice + compile error != happy me...

On 02/15/2012 09:12 PM, Anirudh B wrote:
> I have no idea how that happened... anyway that was part of the undefined reference problems. I also had to add a source file for launcher_counter.
>
> Sent from my Android device.
>
>
>
> -----Original Message-----
> From: "Robert Blair Mason Jr."
> To: Anirudh B
> Sent: Wed, 15 Feb 2012 8:59 PM
> Subject: Re: [612-code] added joysmooth class and moved registry declaration (8b083c4)
>
> Why did you put include guards on a .cpp file?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/rbmj/612-code/commit/8b083c467ed46dc907ec68450a8b12c1e6b17ad4#commitcomment-972255
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/rbmj/612-code/commit/8b083c467ed46dc907ec68450a8b12c1e6b17ad4#commitcomment-972278
>