[icedtea-web] Proposed formatting for C++ side of plugin

Omair Majid omajid at redhat.com
Mon Dec 31 07:08:57 PST 2012


On 12/28/2012 06:07 PM, Omair Majid wrote:
> On a quick look, one thing jumps out at me:
> 
> -bool
> -JavaRequestProcessor::newMessageOnBus(const char* message)
> -{
> +bool JavaRequestProcessor::newMessageOnBus(const char* message) {
> 
> The K&R indentation style places the opening brace in the line below the
> function prototype:
> 
> +bool JavaRequestProcessor::newMessageOnBus(const char* message)
> +{

I didn't mean to imply that this should be changed to conform to K&R
style, just that we should note that this is not exactly the same as K&R.

I like how this is similar enough to our Java style that you don't have
to think about formatting when switching from one language to the other.

Cheers,
Omair

-- 
PGP Key: 66484681 (http://pgp.mit.edu/)
Fingerprint = F072 555B 0A17 3957 4E95  0056 F286 F14F 6648 4681



More information about the distro-pkg-dev mailing list