Squashed 'third_party/allwpilib_2016/' changes from 7f61816..3ce6feb
3ce6feb Updated release number for the new release
e054bbc This adds StopMotor() to the SpeedController interface for C++ and Java. For Java, this is as simple as just adding it, as all motors already have an implementation from MotorSafety that is correctly resolved. For C++, I had to override StopMotor in the classes that descend from SafePWM and explicitly call the SafePWM version. RobotDrive now calls StopMotor on each of its SpeedControllers, instead of calling Disable or setting the motor to 0.0 as it was doing previously.
a15b9dc Merge "More updates to the Gyro test fixing potential null pointer exception"
21b7213 Added Config routine to allow enabling/disabling of limit switch and soft limits. This improves upon the ConfigLimitMode routine, which does not allow certain combinations of enable/disabled limit features. Also keeps parity with LV and Java.
1b45237 Merge "Add an additional member variable for "stopped" which indicates the CAN motor controller has been explicitly stopped, but not disabled by the user (main use case is MotorSafety tripping). When Set() is called the next time the controller will be re-enabled automatically."
1096b15 Add an additional member variable for "stopped" which indicates the CAN motor controller has been explicitly stopped, but not disabled by the user (main use case is MotorSafety tripping). When Set() is called the next time the controller will be re-enabled automatically.
7da21fa More updates to the Gyro test fixing potential null pointer exception
ede5862 Rate-limit duplicate error messages to avoid flooding console.
c22284d Merge "artf4818: Fix CAN Talon JNI references with underscores."
cac3741 Merge "Updated PDP port of Talon and disabled PDP tests for Victor and Jaguar since the Victor and Jaguar don't draw enough current for the PDP to read above 0. PDP tests for both java and cpp only test the Talon now."
097aa8c Fixed the gyro deviation over time test
1d647b3 artf4818: Fix CAN Talon JNI references with underscores.
368cfc7 Merge "Fixed the motor tests by reducing speed to within the limits of the encoders we use. Also fixed java pid tolerances since getAvgError() was broken. It is now fixed and works properly. Added tests for both java and cpp that test if pid tolerances are working using fake input output pairs."
833e459 Updated PDP port of Talon and disabled PDP tests for Victor and Jaguar since the Victor and Jaguar don't draw enough current for the PDP to read above 0. PDP tests for both java and cpp only test the Talon now.
6c096a3 Fixed the motor tests by reducing speed to within the limits of the encoders we use. Also fixed java pid tolerances since getAvgError() was broken. It is now fixed and works properly. Added tests for both java and cpp that test if pid tolerances are working using fake input output pairs.
dd7eb0f Fixed robot drive for C++ Simulation
258a622 Merge "Update version number for Release 3 Print distinctive message on robot program startup Change-Id: Ic91b81bd298ee6730503933cf0e733702e4b4405"
b1386c6 Update version number for Release 3 Print distinctive message on robot program startup Change-Id: Ic91b81bd298ee6730503933cf0e733702e4b4405
a58de40 Merge "Removed publishing of java sim jar"
792d0d3 PDP Classes should support any PDP address
35df955 Merge "Remove maven local as a possible search location"
a0ce9ee Another improvement to HAL-joy getting to ensure it works in future RIO image updates.
0f02c31 Removed publishing of java sim jar
8435ac7 DriverStation::GetJoystickName(): Make work for stick>0.
b4cf4f4 Remove maven local as a possible search location
c3000c3 Merge "Fix HALGetJoystickDescriptor()."
4dec0b4 Merge "Fixed Simulation C++ API"
abc9c27 Fixed Simulation C++ API
b8ae9ec Fix HALGetJoystickDescriptor().
a60f874 Artf4800: Fixes HALGetJoystick*** Segfault
010b584 Merge "fix sim_ds launch script"
4429e16 Merge "Added build dir specification for sim javadoc to not overwrite athena javadoc"
ec9349b Initialized the m_sensors variable to fix artf4798.
9745af8 Added build dir specification for sim javadoc to not overwrite athena javadoc
4da8702 fix sim_ds launch script
05acf79 Fix C++ PIDController SetToleranceBuffer and OnTarget locking.
94a6b05 Merge "Fix onTarget() so that it returns false until there are any values retrieved"
d06053d Fix onTarget() so that it returns false until there are any values retrieved
74927cc Correctly set smart dashboard type for AnalogGyro and ADXRS450_Gyro.
070752f Merge "Fixed sim_ds script library path"
21a8bab Merge "PIDController feed forward term can now be calculated by the end user"
56bd6da Fixed sim_ds script library path
07710f1 Merge "Fixing install script... again"
e1cb61f Use absolute path for NT persistent storage.
09c7482 Fixing install script... again
a3b8bec PIDController feed forward term can now be calculated by the end user
790adb0 artf2612: Update license in source files.
042671c Merge "Removed gz_msgs from wpilibcSim"
c111690 Ultrasonic: replace linked list with std::set.
d71a8ed Removed gz_msgs from wpilibcSim
37259f7 Merge "Replaced linked list in Notifier with std::list"
cd17e7a Merge "Renamed Gyro to AnalogGyro to match athena API"
c5c8a87 Replaced linked list in Notifier with std::list
89405d8 Renamed Gyro to AnalogGyro to match athena API
4a19490 Merge "Adds CANTalon to LiveWindow"
c4a3567 Merge "Fixing the frcsim installer script"
295648f Adds CANTalon to LiveWindow
1b964a2 Merge "Fixes CAN devices in C++ library not showing in the livewindow"
7ba5cee Merge "HAL: Use extern "C" in implementation files."
d17d242 Fixes CAN devices in C++ library not showing in the livewindow
25a771a Added linear digital filters
7349c2c Fixing the frcsim installer script
c82122c Merge "Default bufLength for PIDController in Java should be 1"
58f3f97 Merge "Adds WaitResult to Java waitForInterrupt"
bc8ed12 HAL: Use extern "C" in implementation files.
4cac89e Default bufLength for PIDController in Java should be 1
64fcdcc Keep track of FPGA time rollovers with 64-bit time.
d30b283 Merge "Change C++ Notifier to allow std::function callback."
6ee3052 Merge "Rewrite C++ Notifier to use HAL multi-notifier support."
f5d09e2 Merge "Rewrite Java Notifier and update Interrupt JNI."
d0274aa Merge "Readded styleguide accidentally removed in the reorg"
68311ad Merge "Artf4179: Allow alternate I2C addresses for ADXL345_I2C"
dee12d4 Readded styleguide accidentally removed in the reorg
fa100df Fixed some typos in the comments of MotorEncoderFixture.java, a method name in CANMotorEncoderFixture.java, and the README files
3397b5c Adds WaitResult to Java waitForInterrupt
5f0dffd Artf4177: Use read byte count for ReadString
8564f33 Artf4179: Allow alternate I2C addresses for ADXL345_I2C
e52b52d Change C++ Notifier to allow std::function callback.
40b29e7 Rewrite C++ Notifier to use HAL multi-notifier support.
d126f45 Rewrite Java Notifier and update Interrupt JNI.
557805a Merge "finishing up FRCSim installer"
911b64b finishing up FRCSim installer
e24fe6f Merge "Artf4776 Fixes First DIO PWM usage errors"
f8f9284 Merge "Artf4774 Fixes HAL getHALErrorMessage missing error"
84428d5 Merge "Prevent double free in DigitalGlitchFilter"
a00a5ff Merge "Set correct error message"
9aeee98 Prevent double free in DigitalGlitchFilter
5d2186c working on install process for FRCSim 2016
7fdb616 Merge "This commit adds documentation generation, including grabbing ntcore sources, for both Java and C++. This will need changes made in the wpilib promotion tasks to copy the generatd documentation to the correct places."
f67ebca Improved READMEs
8a7c019 Artf4776 Fixes First DIO PWM usage errors
1cd2f9a Added libnipalu to make vision programs link properly
375b92a This commit adds documentation generation, including grabbing ntcore sources, for both Java and C++. This will need changes made in the wpilib promotion tasks to copy the generatd documentation to the correct places.
8d0a990 Set correct error message
b65401f Artf4774 Fixes HAL getHALErrorMessage missing error
5f918be Condition java sim build on -PmakeSim flag
53bd180 Merge "Add SPARK and SD540 motor controllers"
66cbe69 Fixed double free of DriverStation.
431f345 Repaired simulation build on linux
611593c Add Cmake wrappers and unzip desktop ntcore builds
51a18cd Add SPARK and SD540 motor controllers
c05e883 Merge changes I55ce71c6,I803680c1
2b80029 Rewrite CANTalon JNI layer.
ef4c45b Last feature addition for CANTalon java/C++ user-facing API.
Change-Id: Ia3a124978a426991890b6f8abbe07d34d75ba38d
git-subtree-dir: third_party/allwpilib_2016
git-subtree-split: 3ce6feb8acdeca46e93a55280fb6ace3a4d5bcd6
diff --git a/styleguide/cppguide.html b/styleguide/cppguide.html
new file mode 100644
index 0000000..f37c836
--- /dev/null
+++ b/styleguide/cppguide.html
@@ -0,0 +1,2470 @@
+<!DOCTYPE html>
+<html>
+<head>
+<meta http-equiv="content-type" content="text/html; charset=UTF-8">
+<title>WPILib C++ Style Guide</title>
+<link rel="stylesheet" type="text/css" href="include/styleguide.css">
+<script language="javascript" src="include/styleguide.js"></script>
+</head>
+<body onload="initStyleGuide();">
+<div id="content">
+<h1>WPILib C++ Style Guide (Based on the <a href=http://google-styleguide.googlecode.com/svn/trunk/cppguide.html>Google C++ Style Guide</a>)</h1>
+<div class="horizontal_toc" id="tocDiv"></div>
+
+<div class="main_body">
+
+<h2 class="ignoreLink" id="Background">Background</h2>
+
+<p><strong>This guide is a work in progress.</strong>
+We are currently working on getting this guide updated to
+a point where it is useful for WPILib developers to use.</p>
+
+<p>C++ is one of the two main languages (Java being the other)
+used in WPILib; in order to maintain consistency and keep the
+maintenance of the code manageable, we use this style guide.</p>
+
+<p>There are two main overarching purposes to this guide. The first
+is to act as a normal C++ style guide (both in terms fo formatting
+and programming practices) for C++ developers of WPILib.
+The other purpose is to help Java programmers who may
+know a moderate amount of C++ but may not be fully
+up to date with things like C++11 and so may not even
+realize that certain C++ features exist.</p>
+
+<p>This style guide is a heavily modified version of the
+<a href=http://google-styleguide.googlecode.com/svn/trunk/cppguide.html>
+Google C++ Style Guide</a>. The Google Style Guide has
+a lot of good points and is a good read, but in order
+to cut the style guide down to a more readable size and to
+focus mroe on WPILib-specific information, we have
+altetered the original style guide in several ways.</p>
+
+<p>One way in which we <em>haven't</em> done much to
+alter the original style guide is to keep the vast
+majority of the formatting/naming/etc. related
+information intact. This is both so that we
+do not have to write up our own standards and so
+that existing tools such as clang-format and
+the Google eclipse format configuration files
+can work out of the box. All of these things
+should be relatively non-controversial and do not
+require much discussion.</p>
+
+<p>Where we deviate more from the original guide is
+in the style of the code itself. At the moment (ie,
+when we first created this modified version), we
+deleted all of the sections of the original guide
+which mandate particular programming practices
+such as forbidding exceptions, multiple inheritance,
+etc. However, as time goes on, we gradually add in more
+information along this lines, either by copying
+directly from Google's Style Guide or by writing
+our own decisions and best practices, some of which
+may be very WPILib-specific.</p>
+
+<p>As the original guide makes very clear, consistency
+is extremely important to keeping the code base
+manageable, and so we encourage that, wherever
+reasonable, that you keep everything consistent
+with whatever the standard style is.</p>
+
+<p>Along with just C++ style, it is also important
+to keep in mind that WPILib consists of both a C++
+and Java half. In order to keep things consistent
+and easier for users, we ask that, in general,
+Java and C++ be kept as consistent with one another
+as reasonable. This includes everything from using
+two spaces for indentation in both language to
+keeping the inheritance structure essentially the
+same in both. Although the two do not have to be
+precisely the same, it does mean that if there is
+something that you are doing which will be imposssible
+to reproduce in some way in Java, then you may
+want to reconsider.</p>
+
+<p>One final thing to remember is that High School
+students with relatively little experience programming
+are the main user for this code, and throwing the full
+brunt of C++ at a student just learning how to program
+is likely not the best of ideas. As such, any
+user-facing APIs should minimize the use of any
+more complicated C++ features. As always,
+use your judgement and ask others in cases where
+there is something which may violate anything
+in this guide.</p>
+
+<h2 id="Programming_Guidelines">Programming Guidelines</h2>
+<p>C++ is a large, complicated language, and in order
+to ensure that we stay consistent and maintain certain
+best practices, we have certain rules. For the most part
+these are common sense rules and in some cases exist
+solely to point out features of C++ that someone more
+familiar with Java may not realize even exist.</p>
+
+<h3 id="Pointers">Pointers</h3>
+<p>In general, we strongly discourage the use of
+raw pointers in C++ code; instead, references or
+STL pointers should be used where appropriate.
+There are two exceptions to this rule:</p>
+<ul>
+ <li>When interfacing with lower-level C code or
+ with any libraries which force you to use raw pointers.</li>
+ <li>In order to keep user interfaces consistent,
+ we may keep around deprecated functions which
+ take raw pointers. Any user-facing functions
+ which take raw pointers should be deprecated
+ using the
+ <a href=https://en.wikipedia.org/wiki/C%2B%2B14#The_attribute_.5B.5Bdeprecated.5D.5D><code>[[deprecated]]</code></a>
+ attribute and replaced with either references
+ or STL pointers.</li>
+</ul>
+<p>As of C++11, the following are options in the
+place of raw pointers:</p>
+<ul>
+ <li><code>std::unique_ptr</code> Should be used
+ when you still need to use a pointer, but you
+ only need one entity to own the pointer. The
+ <code>std::unique_ptr</code> will automatically
+ be deleted when there are no more references to
+ it.</li>
+ <li><code>std::shared_ptr</code> Should be used
+ when you still need to use a pointer and you
+ need many references to the object. When
+ there are zero remaining references to the
+ object, it will be deleted. Use <code>std::weak_ptr</code>
+ where necessary to avoid circular dependencies
+ or other potential issues.</li>
+ <li>L-value references (the traditional sort
+ of reference that has been around since before C++11)
+ should be used when you want to pass around a
+ reference to an object and want to guarantee
+ that it won't be null. Use const references
+ if you want to avoid copying a large object
+ but don't want to modify it.</li>
+ <li>R-value references were introduced in C++11
+ and allow for the use of <code>std::move</code>.
+ R-value references should be used where it makes
+ sense that a parameter to a function is having
+ its ownership passed from one place to another.
+ In general, R-value references are not inherently
+ bad, but they do introduce additional complexity
+ that may confuse people who are not familiar
+ with them.</li>
+</ul>
+
+<h3 id="Deprecation">Deprecation</h3>
+<p>When updating APIs, make liberal use of the
+<code>[[deprecated]]</code> attribute (although if
+it is reasonable to simply remove any old interfaces
+then do so) to indicate that users should no longer
+use the function. Currently, this will cause warnings
+in user code and errors in the WPILib build.</p>
+
+<pre>
+[[deprecated("This is a deprecated function; this text will be displayed when"
+ " the compiler throws a warning.")]]
+void foo() {}
+class [[deprecated("This is a deprecated class.")]] Foo {};
+int bar [[deprecated("This is a deprecated variable.")]];
+</pre>
+
+<p>See <a href=http://josephmansfield.uk/articles/marking-deprecated-c++14.html>
+here</a> for more information on deprecation.</p>
+
+<h2 id="Header_Files">Header Files</h2>
+
+<p>In general, every <code>.cc</code> file should have an
+associated <code>.h</code> file. There are some common
+exceptions, such as unittests and
+small <code>.cpp</code> files containing just a
+<code>main()</code> function.</p>
+
+<p>Correct use of header files can make a huge difference to
+the readability, size and performance of your code.</p>
+
+<p>The following rules will guide you through the various
+pitfalls of using header files.</p>
+
+<a id="The_-inl.h_Files"></a>
+<h3 id="Self_contained_Headers">Self-contained Headers</h3>
+
+<div class="summary">
+<p>Header files should be self-contained and end in <code>.h</code>. Files that
+are meant for textual inclusion, but are not headers, should end in
+<code>.inc</code>. Separate <code>-inl.h</code> headers are disallowed.</p>
+</div>
+
+<div class="stylebody">
+<p>All header files should be self-contained. In other
+words, users and refactoring tools should not have to adhere to special
+conditions in order to include the header. Specifically, a
+header should have <a href="#The__define_Guard">header guards</a>,
+should include all other headers it needs, and should not require any
+particular symbols to be defined.</p>
+
+<p>There are rare cases where a file is not meant to be self-contained, but
+instead is meant to be textually included at a specific point in the code.
+Examples are files that need to be included multiple times or
+platform-specific extensions that essentially are part of other headers. Such
+files should use the file extension <code>.inc</code>.</p>
+
+<p>If a template or inline function is declared in a <code>.h</code> file,
+define it in that same file. The definitions of these constructs must
+be included into every <code>.cc</code> file that uses them, or the
+program may fail to link in some build configurations. Do not move these
+definitions to separate <code>-inl.h</code> files.</p>
+
+<p>As an exception, a function template that is explicitly
+instantiated for all relevant sets of template arguments, or
+that is a private member of a class, may
+be defined in the only <code>.cc</code> file that
+instantiates the template.</p>
+
+</div>
+
+<h3 id="The__define_Guard">The #define Guard</h3>
+
+<div class="summary">
+<p>All header files should have <code>#define</code> guards to
+prevent multiple inclusion. The format of the symbol name
+should be
+<code><i><PROJECT></i>_<i><PATH></i>_<i><FILE></i>_H_</code>.</p>
+</div>
+
+<div class="stylebody">
+
+
+
+<p>To guarantee uniqueness, they should
+be based on the full path in a project's source tree. For
+example, the file <code>foo/src/bar/baz.h</code> in
+project <code>foo</code> should have the following
+guard:</p>
+
+<pre>#ifndef FOO_BAR_BAZ_H_
+#define FOO_BAR_BAZ_H_
+
+...
+
+#endif // FOO_BAR_BAZ_H_
+</pre>
+
+
+
+
+</div>
+
+<h3 id="Forward_Declarations">Forward Declarations</h3>
+
+<div class="summary">
+<p>You may forward declare ordinary classes in order to avoid
+unnecessary <code>#include</code>s.</p>
+</div>
+
+<div class="stylebody">
+
+<div class="definition">
+<p>A "forward declaration" is a declaration of a class,
+function, or template without an associated definition.
+<code>#include</code> lines can often be replaced with
+forward declarations of whatever symbols are actually
+used by the client code.</p>
+</div>
+
+<div class="pros">
+<ul>
+ <li>Unnecessary <code>#include</code>s force the
+ compiler to open more files and process more
+ input.</li>
+
+ <li>They can also force your code to be recompiled more
+ often, due to changes in the header.</li>
+</ul>
+</div>
+
+<div class="cons">
+<ul>
+ <li>It can be difficult to determine the correct form
+ of a forward declaration in the presence of features
+ like templates, typedefs, default parameters, and using
+ declarations.</li>
+
+ <li>It can be difficult to determine whether a forward
+ declaration or a full <code>#include</code> is needed
+ for a given piece of code, particularly when implicit
+ conversion operations are involved. In extreme cases,
+ replacing an <code>#include</code> with a forward
+ declaration can silently change the meaning of
+ code.</li>
+
+ <li>Forward declaring multiple symbols from a header
+ can be more verbose than simply
+ <code>#include</code>ing the header.</li>
+
+ <li>Forward declarations of functions and templates can
+ prevent the header owners from making
+ otherwise-compatible changes to their APIs; for
+ example, widening a parameter type, or adding a
+ template parameter with a default value.</li>
+ <li>Forward declaring symbols from namespace
+ <code>std::</code> usually yields undefined
+ behavior.</li>
+
+ <li>Structuring code to enable forward declarations
+ (e.g. using pointer members instead of object members)
+ can make the code slower and more complex.</li>
+
+ <li>The practical efficiency benefits of forward
+ declarations are unproven.</li>
+</ul>
+</div>
+
+<div class="decision">
+<ul>
+ <li>When using a function declared in a header file,
+ always <code>#include</code> that header.</li>
+
+ <li>When using a class template, prefer to
+ <code>#include</code> its header file.</li>
+
+ <li>When using an ordinary class, relying on a forward
+ declaration is OK, but be wary of situations where a
+ forward declaration may be insufficient or incorrect;
+ when in doubt, just <code>#include</code> the
+ appropriate header.</li>
+
+ <li>Do not replace data members with pointers just to
+ avoid an <code>#include</code>.</li>
+</ul>
+
+<p>Please see <a href="#Names_and_Order_of_Includes">Names and Order
+of Includes</a> for rules about when to #include a header.</p>
+</div>
+
+</div>
+
+<h3 id="Inline_Functions">Inline Functions</h3>
+
+<div class="summary">
+<p>Define functions inline only when they are small, say, 10
+lines or less.</p>
+</div>
+
+<div class="stylebody">
+
+<div class="definition">
+<p>You can declare functions in a way that allows the compiler to expand
+them inline rather than calling them through the usual
+function call mechanism.</p>
+</div>
+
+<div class="pros">
+<p>Inlining a function can generate more efficient object
+code, as long as the inlined function is small. Feel free
+to inline accessors and mutators, and other short,
+performance-critical functions.</p>
+</div>
+
+<div class="cons">
+<p>Overuse of inlining can actually make programs slower.
+Depending on a function's size, inlining it can cause the
+code size to increase or decrease. Inlining a very small
+accessor function will usually decrease code size while
+inlining a very large function can dramatically increase
+code size. On modern processors smaller code usually runs
+faster due to better use of the instruction cache.</p>
+</div>
+
+<div class="decision">
+<p>A decent rule of thumb is to not inline a function if
+it is more than 10 lines long. Beware of destructors,
+which are often longer than they appear because of
+implicit member- and base-destructor calls!</p>
+
+<p>Another useful rule of thumb: it's typically not cost
+effective to inline functions with loops or switch
+statements (unless, in the common case, the loop or
+switch statement is never executed).</p>
+
+<p>It is important to know that functions are not always
+inlined even if they are declared as such; for example,
+virtual and recursive functions are not normally inlined.
+Usually recursive functions should not be inline. The
+main reason for making a virtual function inline is to
+place its definition in the class, either for convenience
+or to document its behavior, e.g., for accessors and
+mutators.</p>
+</div>
+
+</div>
+
+<h3 id="Function_Parameter_Ordering">Function Parameter Ordering</h3>
+
+<div class="summary">
+<p>When defining a function, parameter order is: inputs, then
+outputs.</p>
+</div>
+
+<div class="stylebody">
+<p>Parameters to C/C++ functions are either input to the
+function, output from the function, or both. Input
+parameters are usually values or <code>const</code>
+references, while output and input/output parameters will
+be non-<code>const</code> pointers. When ordering
+function parameters, put all input-only parameters before
+any output parameters. In particular, do not add new
+parameters to the end of the function just because they
+are new; place new input-only parameters before the
+output parameters.</p>
+
+<p>This is not a hard-and-fast rule. Parameters that are
+both input and output (often classes/structs) muddy the
+waters, and, as always, consistency with related
+functions may require you to bend the rule.</p>
+
+</div>
+
+<h3 id="Names_and_Order_of_Includes">Names and Order of Includes</h3>
+
+<div class="summary">
+<p>Use standard order for readability and to avoid hidden
+dependencies: Related header, C library, C++ library, other libraries'
+<code>.h</code>, your project's <code>.h</code>.</p>
+</div>
+
+<div class="stylebody">
+<p>
+All of a project's header files should be
+listed as descendants of the project's source
+directory without use of UNIX directory shortcuts
+<code>.</code> (the current directory) or <code>..</code>
+(the parent directory). For example,
+
+<code>google-awesome-project/src/base/logging.h</code>
+should be included as:</p>
+
+<pre>#include "base/logging.h"
+</pre>
+
+<p>In <code><var>dir/foo</var>.cc</code> or
+<code><var>dir/foo_test</var>.cc</code>, whose main
+purpose is to implement or test the stuff in
+<code><var>dir2/foo2</var>.h</code>, order your includes
+as follows:</p>
+
+<ol>
+ <li><code><var>dir2/foo2</var>.h</code>.</li>
+
+ <li>C system files.</li>
+
+ <li>C++ system files.</li>
+
+ <li>Other libraries' <code>.h</code>
+ files.</li>
+
+ <li>
+ Your project's <code>.h</code>
+ files.</li>
+</ol>
+
+<p>With the preferred ordering, if
+<code><var>dir2/foo2</var>.h</code> omits any necessary
+includes, the build of <code><var>dir/foo</var>.cc</code>
+or <code><var>dir/foo</var>_test.cc</code> will break.
+Thus, this rule ensures that build breaks show up first
+for the people working on these files, not for innocent
+people in other packages.</p>
+
+<p><code><var>dir/foo</var>.cc</code> and
+<code><var>dir2/foo2</var>.h</code> are usually in the same
+directory (e.g. <code>base/basictypes_test.cc</code> and
+<code>base/basictypes.h</code>), but may sometimes be in different
+directories too.</p>
+
+
+
+<p>Within each section the includes should be ordered
+alphabetically. Note that older code might not conform to
+this rule and should be fixed when convenient.</p>
+
+<p>You should include all the headers that define the symbols you rely
+upon (except in cases of <a href="#Forward_Declarations">forward
+declaration</a>). If you rely on symbols from <code>bar.h</code>,
+don't count on the fact that you included <code>foo.h</code> which
+(currently) includes <code>bar.h</code>: include <code>bar.h</code>
+yourself, unless <code>foo.h</code> explicitly demonstrates its intent
+to provide you the symbols of <code>bar.h</code>. However, any
+includes present in the related header do not need to be included
+again in the related <code>cc</code> (i.e., <code>foo.cc</code> can
+rely on <code>foo.h</code>'s includes).</p>
+
+<p>For example, the includes in
+
+<code>google-awesome-project/src/foo/internal/fooserver.cc</code>
+might look like this:</p>
+
+
+<pre>#include "foo/server/fooserver.h"
+
+#include <sys/types.h>
+#include <unistd.h>
+#include <hash_map>
+#include <vector>
+
+#include "base/basictypes.h"
+#include "base/commandlineflags.h"
+#include "foo/server/bar.h"
+</pre>
+
+<p class="exception">Sometimes, system-specific code needs
+conditional includes. Such code can put conditional
+includes after other includes. Of course, keep your
+system-specific code small and localized. Example:</p>
+
+<pre>#include "foo/public/fooserver.h"
+
+#include "base/port.h" // For LANG_CXX11.
+
+#ifdef LANG_CXX11
+#include <initializer_list>
+#endif // LANG_CXX11
+</pre>
+
+</div>
+
+<h2 id="Naming">Naming</h2>
+
+<p>The most important consistency rules are those that govern
+naming. The style of a name immediately informs us what sort of
+thing the named entity is: a type, a variable, a function, a
+constant, a macro, etc., without requiring us to search for the
+declaration of that entity. The pattern-matching engine in our
+brains relies a great deal on these naming rules.
+</p>
+
+<p>Naming rules are pretty arbitrary, but
+ we feel that
+consistency is more important than individual preferences in this
+area, so regardless of whether you find them sensible or not,
+the rules are the rules.</p>
+
+<h3 id="General_Naming_Rules">General Naming Rules</h3>
+
+<div class="summary">
+<p>Function names, variable names, and filenames should be
+descriptive; eschew abbreviation.</p>
+</div>
+
+<div class="stylebody">
+<p>Give as descriptive a name as possible, within reason.
+Do not worry about saving horizontal space as it is far
+more important to make your code immediately
+understandable by a new reader. Do not use abbreviations
+that are ambiguous or unfamiliar to readers outside your
+project, and do not abbreviate by deleting letters within
+a word.</p>
+
+<pre>int price_count_reader; // No abbreviation.
+int num_errors; // "num" is a widespread convention.
+int num_dns_connections; // Most people know what "DNS" stands for.
+</pre>
+
+<pre class="badcode">int n; // Meaningless.
+int nerr; // Ambiguous abbreviation.
+int n_comp_conns; // Ambiguous abbreviation.
+int wgc_connections; // Only your group knows what this stands for.
+int pc_reader; // Lots of things can be abbreviated "pc".
+int cstmr_id; // Deletes internal letters.
+</pre>
+
+</div>
+
+<h3 id="File_Names">File Names</h3>
+
+<div class="summary">
+<p>Filenames should be all lowercase and can include
+underscores (<code>_</code>) or dashes (<code>-</code>).
+Follow the convention that your
+
+project uses. If there is no consistent
+local pattern to follow, prefer "_".</p>
+</div>
+
+<div class="stylebody">
+
+<p>Examples of acceptable file names:</p>
+
+<ul>
+ <li><code>my_useful_class.cc</code></li>
+ <li><code>my-useful-class.cc</code></li>
+ <li><code>myusefulclass.cc</code></li>
+ <li><code>myusefulclass_test.cc // _unittest and _regtest are deprecated.</code></li>
+</ul>
+
+<p>C++ files should end in <code>.cc</code> and header files should end in
+<code>.h</code>. Files that rely on being textually included at specific points
+should end in <code>.inc</code> (see also the section on
+<a href="#Self_contained_Headers">self-contained headers</a>).</p>
+
+<p>Do not use filenames that already exist in
+<code>/usr/include</code>, such as <code>db.h</code>.</p>
+
+<p>In general, make your filenames very specific. For
+example, use <code>http_server_logs.h</code> rather than
+<code>logs.h</code>. A very common case is to have a pair
+of files called, e.g., <code>foo_bar.h</code> and
+<code>foo_bar.cc</code>, defining a class called
+<code>FooBar</code>.</p>
+
+<p>Inline functions must be in a <code>.h</code> file. If
+your inline functions are very short, they should go
+directly into your <code>.h</code> file. </p>
+
+</div>
+
+<h3 id="Type_Names">Type Names</h3>
+
+<div class="summary">
+<p>Type names start with a capital letter and have a capital
+letter for each new word, with no underscores:
+<code>MyExcitingClass</code>, <code>MyExcitingEnum</code>.</p>
+</div>
+
+<div class="stylebody">
+
+<p>The names of all types — classes, structs, typedefs,
+and enums — have the same naming convention. Type names
+should start with a capital letter and have a capital letter
+for each new word. No underscores. For example:</p>
+
+<pre>// classes and structs
+class UrlTable { ...
+class UrlTableTester { ...
+struct UrlTableProperties { ...
+
+// typedefs
+typedef hash_map<UrlTableProperties *, string> PropertiesMap;
+
+// enums
+enum UrlTableErrors { ...
+</pre>
+
+</div>
+
+<h3 id="Variable_Names">Variable Names</h3>
+
+<div class="summary">
+<p>The names of variables and data members are all lowercase, with
+underscores between words. Data members of classes (but not structs)
+additionally are prefixed with "m_". For instance:
+<code>a_local_variable</code>, <code>a_struct_data_member</code>,
+<code>m_a_class_data_member</code>.</p>
+</div>
+
+<div class="stylebody">
+
+<h4 class="stylepoint_subsection">Common Variable names</h4>
+
+<p>For example:</p>
+
+<pre>string table_name; // OK - uses underscore.
+string tablename; // OK - all lowercase.
+</pre>
+
+<pre class="badcode">string tableName; // Bad - mixed case.
+</pre>
+
+<h4 class="stylepoint_subsection">Class Data Members</h4>
+
+<p>Data members of classes, both static and non-static, are
+named like ordinary nonmember variables, but prefixed with a
+"m_".</p>
+
+<pre>class TableInfo {
+ ...
+ private:
+ string m_table_name; // OK - m_ at beginning.
+ string m_tablename; // OK.
+ static Pool<TableInfo>* m_pool; // OK.
+};
+</pre>
+
+<h4 class="stylepoint_subsection">Struct Data Members</h4>
+
+<p>Data members of structs, both static and non-static,
+are named like ordinary nonmember variables. They do not have
+the preceding "m_" that data members in classes have.</p>
+
+<pre>struct UrlTableProperties {
+ string name;
+ int num_entries;
+ static Pool<UrlTableProperties>* pool;
+};
+</pre>
+
+
+<p>See <a href="#Structs_vs._Classes">Structs vs.
+Classes</a> for a discussion of when to use a struct
+versus a class.</p>
+
+<h4 class="stylepoint_subsection">Global Variables</h4>
+
+<p>There are no special requirements for global
+variables, which should be rare in any case, but if you
+use one, consider prefixing it with <code>g_</code> or
+some other marker to easily distinguish it from local
+variables.</p>
+
+</div>
+
+<h3 id="Constant_Names">Constant Names</h3>
+
+<div class="summary">
+<p>Use a <code>k</code> followed by mixed case, e.g.,
+<code>kDaysInAWeek</code>, for constants defined globally or within a class.</p>
+</div>
+
+<div class="stylebody">
+
+<p>As a convenience to the reader, compile-time constants of global or class scope
+follow a different naming convention from other variables.
+Use a <code>k</code> followed by words with uppercase first letters:</p>
+
+<pre>const int kDaysInAWeek = 7;
+</pre>
+
+<p>This convention may optionally be used for compile-time constants of local scope;
+otherwise the usual variable naming rules apply.
+
+</p></div>
+
+<h3 id="Function_Names">Function Names</h3>
+
+<div class="summary">
+<p>Regular functions have mixed case; accessors and mutators
+match the name of the variable:
+<code>MyExcitingFunction()</code>,
+<code>MyExcitingMethod()</code>,
+<code>my_exciting_member_variable()</code>,
+<code>set_my_exciting_member_variable()</code>.</p>
+</div>
+
+<div class="stylebody">
+
+<h4 class="stylepoint_subsection">Regular Functions</h4>
+
+<p>Functions should start with a capital letter and have
+a capital letter for each new word. No underscores.</p>
+
+<p>If your function crashes upon an error, you should
+append OrDie to the function name. This only applies to
+functions which could be used by production code and to
+errors that are reasonably likely to occur during normal
+operation.</p>
+
+ <pre>AddTableEntry()
+DeleteUrl()
+OpenFileOrDie()
+</pre>
+
+<h4 class="stylepoint_subsection">Accessors and Mutators</h4>
+
+<p>Accessors and mutators (get and set functions) should
+match the name of the variable they are getting and
+setting. This shows an excerpt of a class whose instance
+variable is <code>num_entries_</code>.</p>
+
+<pre>class MyClass {
+ public:
+ ...
+ int num_entries() const { return num_entries_; }
+ void set_num_entries(int num_entries) { num_entries_ = num_entries; }
+
+ private:
+ int num_entries_;
+};
+</pre>
+
+<p>You may also use lowercase letters for other very
+short inlined functions. For example if a function were
+so cheap you would not cache the value if you were
+calling it in a loop, then lowercase naming would be
+acceptable.</p>
+
+</div>
+
+<h3 id="Namespace_Names">Namespace Names</h3>
+
+<div class="summary">
+
+
+<p>Namespace names are all lower-case,
+and based on project names and possibly their directory
+structure: <code>google_awesome_project</code>.</p>
+</div>
+
+<div class="stylebody">
+
+<p>See <a href="#Namespaces">Namespaces</a> for a
+discussion of namespaces and how to name them.</p>
+
+</div>
+
+<h3 id="Enumerator_Names">Enumerator Names</h3>
+
+<div class="summary">
+<p>Enumerators should be named <i>either</i> like
+<a href="#Constant_Names">constants</a> or like
+<a href="#Macro_Names">macros</a>: either <code>kEnumName</code> or
+<code>ENUM_NAME</code>.</p>
+</div>
+
+<div class="stylebody">
+
+<p>Preferably, the individual enumerators should be named
+like <a href="#Constant_Names">constants</a>. However, it
+is also acceptable to name them like
+<a href="Macro_Names">macros</a>. The enumeration name,
+<code>UrlTableErrors</code> (and
+<code>AlternateUrlTableErrors</code>), is a type, and
+therefore mixed case.</p>
+
+<pre>enum UrlTableErrors {
+ kOK = 0,
+ kErrorOutOfMemory,
+ kErrorMalformedInput,
+};
+enum AlternateUrlTableErrors {
+ OK = 0,
+ OUT_OF_MEMORY = 1,
+ MALFORMED_INPUT = 2,
+};
+</pre>
+
+<p>Until January 2009, the style was to name enum values
+like <a href="#Macro_Names">macros</a>. This caused
+problems with name collisions between enum values and
+macros. Hence, the change to prefer constant-style naming
+was put in place. New code should prefer constant-style
+naming if possible. However, there is no reason to change
+old code to use constant-style names, unless the old
+names are actually causing a compile-time problem.</p>
+
+
+
+</div>
+
+<h3 id="Macro_Names">Macro Names</h3>
+
+<div class="summary">
+<p>You're not really going to <a href="#Preprocessor_Macros">
+define a macro</a>, are you? If you do, they're like this:
+<code>MY_MACRO_THAT_SCARES_SMALL_CHILDREN</code>.</p>
+</div>
+
+<div class="stylebody">
+
+<p>Please see the <a href="#Preprocessor_Macros">description
+of macros</a>; in general macros should <em>not</em> be used.
+However, if they are absolutely needed, then they should be
+named with all capitals and underscores.</p>
+
+<pre>#define ROUND(x) ...
+#define PI_ROUNDED 3.0
+</pre>
+
+</div>
+
+<h3 id="Exceptions_to_Naming_Rules">Exceptions to Naming Rules</h3>
+
+<div class="summary">
+<p>If you are naming something that is analogous to an
+existing C or C++ entity then you can follow the existing
+naming convention scheme.</p>
+</div>
+
+<div class="stylebody">
+
+<dl>
+ <dt><code>bigopen()</code></dt>
+ <dd>function name, follows form of <code>open()</code></dd>
+
+ <dt><code>uint</code></dt>
+ <dd><code>typedef</code></dd>
+
+ <dt><code>bigpos</code></dt>
+ <dd><code>struct</code> or <code>class</code>, follows
+ form of <code>pos</code></dd>
+
+ <dt><code>sparse_hash_map</code></dt>
+ <dd>STL-like entity; follows STL naming conventions</dd>
+
+ <dt><code>LONGLONG_MAX</code></dt>
+ <dd>a constant, as in <code>INT_MAX</code></dd>
+</dl>
+
+</div>
+
+<h2 id="Comments">Comments</h2>
+
+<p>Though a pain to write, comments are absolutely vital to
+keeping our code readable. The following rules describe what
+you should comment and where. But remember: while comments are
+very important, the best code is self-documenting. Giving
+sensible names to types and variables is much better than using
+obscure names that you must then explain through comments.</p>
+
+<p>When writing your comments, write for your audience: the
+next
+contributor who will need to
+understand your code. Be generous — the next
+one may be you!</p>
+
+<h3 id="Comment_Style">Comment Style</h3>
+
+<div class="summary">
+<p>Use either the <code>//</code> or <code>/* */</code>
+syntax, as long as you are consistent.</p>
+</div>
+
+<div class="stylebody">
+
+<p>You can use either the <code>//</code> or the <code>/*
+*/</code> syntax; however, <code>//</code> is
+<em>much</em> more common. Be consistent with how you
+comment and what style you use where.</p>
+
+</div>
+
+<h3 id="File_Comments">File Comments</h3>
+
+<div class="summary">
+<p> Start each file with license
+boilerplate, followed by a description of its
+contents.</p>
+</div>
+
+<div class="stylebody">
+
+<h4 class="stylepoint_subsection">Legal Notice and Author
+Line</h4>
+
+
+
+<p>Every file should contain license
+boilerplate. Choose the appropriate boilerplate for the
+license used by the project (for example, Apache 2.0,
+BSD, LGPL, GPL).</p>
+
+<p>If you make significant changes to a file with an
+author line, consider deleting the author line.</p>
+
+<h4 class="stylepoint_subsection">File Contents</h4>
+
+<p>Every file should have a comment at the top describing
+its contents.</p>
+
+<p>Generally a <code>.h</code> file will describe the
+classes that are declared in the file with an overview of
+what they are for and how they are used. A
+<code>.cc</code> file should contain more information
+about implementation details or discussions of tricky
+algorithms. If you feel the implementation details or a
+discussion of the algorithms would be useful for someone
+reading the <code>.h</code>, feel free to put it there
+instead, but mention in the <code>.cc</code> that the
+documentation is in the <code>.h</code> file.</p>
+
+<p>Do not duplicate comments in both the <code>.h</code>
+and the <code>.cc</code>. Duplicated comments
+diverge.</p>
+
+</div>
+
+<h3 id="Class_Comments">Class Comments</h3>
+
+<div class="summary">
+<p>Every class definition should have an accompanying comment
+that describes what it is for and how it should be used.</p>
+</div>
+
+<div class="stylebody">
+
+<pre>// Iterates over the contents of a GargantuanTable. Sample usage:
+// GargantuanTableIterator* iter = table->NewIterator();
+// for (iter->Seek("foo"); !iter->done(); iter->Next()) {
+// process(iter->key(), iter->value());
+// }
+// delete iter;
+class GargantuanTableIterator {
+ ...
+};
+</pre>
+
+<p>If you have already described a class in detail in the
+comments at the top of your file feel free to simply
+state "See comment at top of file for a complete
+description", but be sure to have some sort of
+comment.</p>
+
+<p>Document the synchronization assumptions the class
+makes, if any. If an instance of the class can be
+accessed by multiple threads, take extra care to document
+the rules and invariants surrounding multithreaded
+use.</p>
+
+</div>
+
+<h3 id="Function_Comments">Function Comments</h3>
+
+<div class="summary">
+<p>Declaration comments describe use of the function; comments
+at the definition of a function describe operation.</p>
+</div>
+
+<div class="stylebody">
+
+<h4 class="stylepoint_subsection">Function Declarations</h4>
+
+<p>Every function declaration should have comments
+immediately preceding it that describe what the function
+does and how to use it. These comments should be
+descriptive ("Opens the file") rather than imperative
+("Open the file"); the comment describes the function, it
+does not tell the function what to do. In general, these
+comments do not describe how the function performs its
+task. Instead, that should be left to comments in the
+function definition.</p>
+
+<p>Types of things to mention in comments at the function
+declaration:</p>
+
+<ul>
+ <li>What the inputs and outputs are.</li>
+
+ <li>For class member functions: whether the object
+ remembers reference arguments beyond the duration of
+ the method call, and whether it will free them or
+ not.</li>
+
+ <li>If the function allocates memory that the caller
+ must free.</li>
+
+ <li>Whether any of the arguments can be a null
+ pointer.</li>
+
+ <li>If there are any performance implications of how a
+ function is used.</li>
+
+ <li>If the function is re-entrant. What are its
+ synchronization assumptions?</li>
+ </ul>
+
+<p>Here is an example:</p>
+
+<pre>// Returns an iterator for this table. It is the client's
+// responsibility to delete the iterator when it is done with it,
+// and it must not use the iterator once the GargantuanTable object
+// on which the iterator was created has been deleted.
+//
+// The iterator is initially positioned at the beginning of the table.
+//
+// This method is equivalent to:
+// Iterator* iter = table->NewIterator();
+// iter->Seek("");
+// return iter;
+// If you are going to immediately seek to another place in the
+// returned iterator, it will be faster to use NewIterator()
+// and avoid the extra seek.
+Iterator* GetIterator() const;
+</pre>
+
+<p>However, do not be unnecessarily verbose or state the
+completely obvious. Notice below that it is not necessary
+ to say "returns false otherwise" because this is
+implied.</p>
+
+<pre>// Returns true if the table cannot hold any more entries.
+bool IsTableFull();
+</pre>
+
+<p>When commenting constructors and destructors, remember
+that the person reading your code knows what constructors
+and destructors are for, so comments that just say
+something like "destroys this object" are not useful.
+Document what constructors do with their arguments (for
+example, if they take ownership of pointers), and what
+cleanup the destructor does. If this is trivial, just
+skip the comment. It is quite common for destructors not
+to have a header comment.</p>
+
+<h4 class="stylepoint_subsection">Function Definitions</h4>
+
+<p>If there is anything tricky about how a function does
+its job, the function definition should have an
+explanatory comment. For example, in the definition
+comment you might describe any coding tricks you use,
+give an overview of the steps you go through, or explain
+why you chose to implement the function in the way you
+did rather than using a viable alternative. For instance,
+you might mention why it must acquire a lock for the
+first half of the function but why it is not needed for
+the second half.</p>
+
+<p>Note you should <em>not</em> just repeat the comments
+given with the function declaration, in the
+<code>.h</code> file or wherever. It's okay to
+recapitulate briefly what the function does, but the
+focus of the comments should be on how it does it.</p>
+
+</div>
+
+<h3 id="Variable_Comments">Variable Comments</h3>
+
+<div class="summary">
+<p>In general the actual name of the variable should be
+descriptive enough to give a good idea of what the variable
+is used for. In certain cases, more comments are required.</p>
+</div>
+
+<div class="stylebody">
+
+<h4 class="stylepoint_subsection">Class Data Members</h4>
+
+<p>Each class data member (also called an instance
+variable or member variable) should have a comment
+describing what it is used for. If the variable can take
+sentinel values with special meanings, such as a null
+pointer or -1, document this. For example:</p>
+
+
+<pre>private:
+ // Keeps track of the total number of entries in the table.
+ // Used to ensure we do not go over the limit. -1 means
+ // that we don't yet know how many entries the table has.
+ int num_total_entries_;
+</pre>
+
+<h4 class="stylepoint_subsection">Global Variables</h4>
+
+<p>As with data members, all global variables should have
+a comment describing what they are and what they are used
+for. For example:</p>
+
+<pre>// The total number of tests cases that we run through in this regression test.
+const int kNumTestCases = 6;
+</pre>
+
+</div>
+
+<h3 id="Implementation_Comments">Implementation Comments</h3>
+
+<div class="summary">
+<p>In your implementation you should have comments in tricky,
+non-obvious, interesting, or important parts of your code.</p>
+</div>
+
+<div class="stylebody">
+
+<h4 class="stylepoint_subsection">Explanatory Comments</h4>
+
+<p>Tricky or complicated code blocks should have comments
+before them. Example:</p>
+
+<pre>// Divide result by two, taking into account that x
+// contains the carry from the add.
+for (int i = 0; i < result->size(); i++) {
+ x = (x << 8) + (*result)[i];
+ (*result)[i] = x >> 1;
+ x &= 1;
+}
+</pre>
+
+<h4 class="stylepoint_subsection">Line Comments</h4>
+
+<p>Also, lines that are non-obvious should get a comment
+at the end of the line. These end-of-line comments should
+be separated from the code by 2 spaces. Example:</p>
+
+<pre>// If we have enough memory, mmap the data portion too.
+mmap_budget = max<int64>(0, mmap_budget - index_->length());
+if (mmap_budget >= data_size_ && !MmapData(mmap_chunk_bytes, mlock))
+ return; // Error already logged.
+</pre>
+
+<p>Note that there are both comments that describe what
+the code is doing, and comments that mention that an
+error has already been logged when the function
+returns.</p>
+
+<p>If you have several comments on subsequent lines, it
+can often be more readable to line them up:</p>
+
+<pre>DoSomething(); // Comment here so the comments line up.
+DoSomethingElseThatIsLonger(); // Two spaces between the code and the comment.
+{ // One space before comment when opening a new scope is allowed,
+ // thus the comment lines up with the following comments and code.
+ DoSomethingElse(); // Two spaces before line comments normally.
+}
+vector<string> list{// Comments in braced lists describe the next element ..
+ "First item",
+ // .. and should be aligned appropriately.
+ "Second item"};
+DoSomething(); /* For trailing block comments, one space is fine. */
+</pre>
+
+<h4 class="stylepoint_subsection">nullptr/NULL, true/false, 1, 2, 3...</h4>
+
+<p>When you pass in a null pointer, boolean, or literal
+integer values to functions, you should consider adding a
+comment about what they are, or make your code
+self-documenting by using constants. For example,
+compare:</p>
+
+<pre class="badcode">bool success = CalculateSomething(interesting_value,
+ 10,
+ false,
+ NULL); // What are these arguments??
+</pre>
+
+<p>versus:</p>
+
+<pre>bool success = CalculateSomething(interesting_value,
+ 10, // Default base value.
+ false, // Not the first time we're calling this.
+ NULL); // No callback.
+</pre>
+
+<p>Or alternatively, constants or self-describing variables:</p>
+
+<pre>const int kDefaultBaseValue = 10;
+const bool kFirstTimeCalling = false;
+Callback *null_callback = NULL;
+bool success = CalculateSomething(interesting_value,
+ kDefaultBaseValue,
+ kFirstTimeCalling,
+ null_callback);
+</pre>
+
+<h4 class="stylepoint_subsection">Don'ts</h4>
+
+<p>Note that you should <em>never</em> describe the code
+itself. Assume that the person reading the code knows C++
+better than you do, even though he or she does not know
+what you are trying to do:</p>
+
+<pre class="badcode">// Now go through the b array and make sure that if i occurs,
+// the next element is i+1.
+... // Geez. What a useless comment.
+</pre>
+
+</div>
+
+<h3 id="Punctuation,_Spelling_and_Grammar">Punctuation, Spelling and Grammar</h3>
+
+<div class="summary">
+<p>Pay attention to punctuation, spelling, and grammar; it is
+easier to read well-written comments than badly written
+ones.</p>
+</div>
+
+<div class="stylebody">
+
+<p>Comments should be as readable as narrative text, with
+proper capitalization and punctuation. In many cases,
+complete sentences are more readable than sentence
+fragments. Shorter comments, such as comments at the end
+of a line of code, can sometimes be less formal, but you
+should be consistent with your style.</p>
+
+<p>Although it can be frustrating to have a code reviewer
+point out that you are using a comma when you should be
+using a semicolon, it is very important that source code
+maintain a high level of clarity and readability. Proper
+punctuation, spelling, and grammar help with that
+goal.</p>
+
+</div>
+
+<h3 id="TODO_Comments">TODO Comments</h3>
+
+<div class="summary">
+<p>Use <code>TODO</code> comments for code that is temporary,
+a short-term solution, or good-enough but not perfect.</p>
+</div>
+
+<div class="stylebody">
+
+<p><code>TODO</code>s should include the string
+<code>TODO</code> in all caps, followed by the
+
+name, e-mail address, or other
+identifier of the person
+ with the best context
+about the problem referenced by the <code>TODO</code>. The
+main purpose is to have a consistent <code>TODO</code> that
+can be searched to find out how to get more details upon
+request. A <code>TODO</code> is not a commitment that the
+person referenced will fix the problem. Thus when you create
+a <code>TODO</code>, it is almost always your
+
+name
+that is given.</p>
+
+
+
+<div>
+<pre>// TODO(kl@gmail.com): Use a "*" here for concatenation operator.
+// TODO(Zeke) change this to use relations.
+</pre>
+</div>
+
+<p>If your <code>TODO</code> is of the form "At a future
+date do something" make sure that you either include a
+very specific date ("Fix by November 2005") or a very
+specific event ("Remove this code when all clients can
+handle XML responses.").</p>
+
+</div>
+
+<h3 id="Deprecation_Comments">Deprecation Comments</h3>
+
+<div class="summary">
+<p>Mark deprecated interface points with <code>DEPRECATED</code>
+comments.</p>
+</div>
+
+<div class="stylebody">
+
+<p>You can mark an interface as deprecated by writing a
+comment containing the word <code>DEPRECATED</code> in
+all caps. The comment goes either before the declaration
+of the interface or on the same line as the
+declaration.</p>
+
+
+
+<p>After the word
+<code>DEPRECATED</code>, write your name, e-mail address,
+or other identifier in parentheses.</p>
+
+<p>A deprecation comment must include simple, clear
+directions for people to fix their callsites. In C++, you
+can implement a deprecated function as an inline function
+that calls the new interface point.</p>
+
+<p>Marking an interface point <code>DEPRECATED</code>
+will not magically cause any callsites to change. If you
+want people to actually stop using the deprecated
+facility, you will have to fix the callsites yourself or
+recruit a crew to help you.</p>
+
+<p>New code should not contain calls to deprecated
+interface points. Use the new interface point instead. If
+you cannot understand the directions, find the person who
+created the deprecation and ask them for help using the
+new interface point.</p>
+
+
+
+</div>
+
+<h2 id="Formatting">Formatting</h2>
+
+<p>Coding style and formatting are pretty arbitrary, but a
+
+project is much easier to follow
+if everyone uses the same style. Individuals may not agree with every
+aspect of the formatting rules, and some of the rules may take
+some getting used to, but it is important that all
+
+project contributors follow the
+style rules so that
+they can all read and understand
+everyone's code easily.</p>
+
+
+
+<p>To help you format code correctly, we've
+created a
+<a href="http://google-styleguide.googlecode.com/svn/trunk/google-c-style.el">
+settings file for emacs</a>.</p>
+
+<h3 id="Line_Length">Line Length</h3>
+
+<div class="summary">
+<p>Each line of text in your code should be at most 80
+characters long.</p>
+</div>
+
+<div class="stylebody">
+
+
+
+ <p>We recognize that this rule is
+controversial, but so much existing code already adheres
+to it, and we feel that consistency is important.</p>
+
+<div class="pros">
+<p>Those who favor this rule
+argue that it is rude to force them to resize
+their windows and there is no need for anything longer.
+Some folks are used to having several code windows
+side-by-side, and thus don't have room to widen their
+windows in any case. People set up their work environment
+assuming a particular maximum window width, and 80
+columns has been the traditional standard. Why change
+it?</p>
+</div>
+
+<div class="cons">
+<p>Proponents of change argue that a wider line can make
+code more readable. The 80-column limit is an hidebound
+throwback to 1960s mainframes; modern equipment has wide screens that
+can easily show longer lines.</p>
+</div>
+
+<div class="decision">
+<p> 80 characters is the maximum.</p>
+
+<p class="exception">If a comment line contains an example
+command or a literal URL longer than 80 characters, that
+line may be longer than 80 characters for ease of cut and
+paste.</p>
+
+<p class="exception">A raw-string literal may have content
+that exceeds 80 characters. Except for test code, such literals
+should appear near top of a file.</p>
+
+<p class="exception">An <code>#include</code> statement with a
+long path may exceed 80 columns.</p>
+
+<p class="exception">You needn't be concerned about
+<a href="#The__define_Guard">header guards</a> that exceed
+the maximum length. </p>
+</div>
+
+</div>
+
+<h3 id="Non-ASCII_Characters">Non-ASCII Characters</h3>
+
+<div class="summary">
+<p>Non-ASCII characters should be rare, and must use UTF-8
+formatting.</p>
+</div>
+
+<div class="stylebody">
+
+<p>You shouldn't hard-code user-facing text in source,
+even English, so use of non-ASCII characters should be
+rare. However, in certain cases it is appropriate to
+include such words in your code. For example, if your
+code parses data files from foreign sources, it may be
+appropriate to hard-code the non-ASCII string(s) used in
+those data files as delimiters. More commonly, unittest
+code (which does not need to be localized) might
+contain non-ASCII strings. In such cases, you should use
+UTF-8, since that is an encoding
+understood by most tools able to handle more than just
+ASCII.</p>
+
+<p>Hex encoding is also OK, and encouraged where it
+enhances readability — for example,
+<code>"\xEF\xBB\xBF"</code>, or, even more simply,
+<code>u8"\uFEFF"</code>, is the Unicode zero-width
+no-break space character, which would be invisible if
+included in the source as straight UTF-8.</p>
+
+<p>Use the <code>u8</code> prefix
+to guarantee that a string literal containing
+<code>\uXXXX</code> escape sequences is encoded as UTF-8.
+Do not use it for strings containing non-ASCII characters
+encoded as UTF-8, because that will produce incorrect
+output if the compiler does not interpret the source file
+as UTF-8. </p>
+
+<p>You shouldn't use the C++11 <code>char16_t</code> and
+<code>char32_t</code> character types, since they're for
+non-UTF-8 text. For similar reasons you also shouldn't
+use <code>wchar_t</code> (unless you're writing code that
+interacts with the Windows API, which uses
+<code>wchar_t</code> extensively).</p>
+
+</div>
+
+<h3 id="Spaces_vs._Tabs">Spaces vs. Tabs</h3>
+
+<div class="summary">
+<p>Use only spaces, and indent 2 spaces at a time.</p>
+</div>
+
+<div class="stylebody">
+
+<p>We use spaces for indentation. Do not use tabs in your
+code. You should set your editor to emit spaces when you
+hit the tab key.</p>
+
+</div>
+
+<h3 id="Function_Declarations_and_Definitions">Function Declarations and Definitions</h3>
+
+<div class="summary">
+<p>Return type on the same line as function name, parameters
+on the same line if they fit. Wrap parameter lists which do
+not fit on a single line as you would wrap arguments in a
+function call.</p>
+</div>
+
+<div class="stylebody">
+
+<p>Functions look like this:</p>
+
+
+<pre>ReturnType ClassName::FunctionName(Type par_name1, Type par_name2) {
+ DoSomething();
+ ...
+}
+</pre>
+
+<p>If you have too much text to fit on one line:</p>
+
+<pre>ReturnType ClassName::ReallyLongFunctionName(Type par_name1, Type par_name2,
+ Type par_name3) {
+ DoSomething();
+ ...
+}
+</pre>
+
+<p>or if you cannot fit even the first parameter:</p>
+
+<pre>ReturnType LongClassName::ReallyReallyReallyLongFunctionName(
+ Type par_name1, // 4 space indent
+ Type par_name2,
+ Type par_name3) {
+ DoSomething(); // 2 space indent
+ ...
+}
+</pre>
+
+<p>Some points to note:</p>
+
+<ul>
+ <li>If you cannot fit the return type and the function
+ name on a single line, break between them.</li>
+
+ <li>If you break after the return type of a function
+ declaration or definition, do not indent.</li>
+
+ <li>The open parenthesis is always on the same line as
+ the function name.</li>
+
+ <li>There is never a space between the function name
+ and the open parenthesis.</li>
+
+ <li>There is never a space between the parentheses and
+ the parameters.</li>
+
+ <li>The open curly brace is always at the end of the
+ same line as the last parameter.</li>
+
+ <li>The close curly brace is either on the last line by
+ itself or (if other style rules permit) on the same
+ line as the open curly brace.</li>
+
+ <li>There should be a space between the close
+ parenthesis and the open curly brace.</li>
+
+ <li>All parameters should be named, with identical
+ names in the declaration and implementation.</li>
+
+ <li>All parameters should be aligned if possible.</li>
+
+ <li>Default indentation is 2 spaces.</li>
+
+ <li>Wrapped parameters have a 4 space indent.</li>
+</ul>
+
+<p>If some parameters are unused, comment out the
+variable name in the function definition:</p>
+
+<pre>// Always have named parameters in interfaces.
+class Shape {
+ public:
+ virtual void Rotate(double radians) = 0;
+};
+
+// Always have named parameters in the declaration.
+class Circle : public Shape {
+ public:
+ virtual void Rotate(double radians);
+};
+
+// Comment out unused named parameters in definitions.
+void Circle::Rotate(double /*radians*/) {}
+</pre>
+
+<pre class="badcode">// Bad - if someone wants to implement later, it's not clear what the
+// variable means.
+void Circle::Rotate(double) {}
+</pre>
+
+</div>
+
+<h3 id="Formatting_Lambda_Expressions">Lambda Expressions</h3>
+
+<div class="summary">
+<p>Format parameters and bodies as for any other function, and capture
+lists like other comma-separated lists.</p>
+</div>
+
+<div class="stylebody">
+<p>For by-reference captures, do not leave a space between the
+ampersand (&) and the variable name.</p>
+<pre>int x = 0;
+auto add_to_x = [&x](int n) { x += n; };
+</pre>
+<p>Short lambdas may be written inline as function arguments.</p>
+<pre>std::set<int> blacklist = {7, 8, 9};
+std::vector<int> digits = {3, 9, 1, 8, 4, 7, 1};
+digits.erase(std::remove_if(digits.begin(), digits.end(), [&blacklist](int i) {
+ return blacklist.find(i) != blacklist.end();
+ }),
+ digits.end());
+</pre>
+
+</div>
+
+<h3 id="Function_Calls">Function Calls</h3>
+
+<div class="summary">
+<p>Either write the call all on a single line, wrap the
+arguments at the parenthesis, or start the arguments on a new
+line indented by four spaces and continue at that 4 space
+indent. In the absence of other considerations, use the
+minimum number of lines, including placing multiple arguments
+on each line where appropriate.</p>
+</div>
+
+<div class="stylebody">
+
+<p>Function calls have the following format:</p>
+<pre>bool retval = DoSomething(argument1, argument2, argument3);
+</pre>
+
+<p>If the arguments do not all fit on one line, they
+should be broken up onto multiple lines, with each
+subsequent line aligned with the first argument. Do not
+add spaces after the open paren or before the close
+paren:</p>
+<pre>bool retval = DoSomething(averyveryveryverylongargument1,
+ argument2, argument3);
+</pre>
+
+<p>Arguments may optionally all be placed on subsequent
+lines with a four space indent:</p>
+<pre>if (...) {
+ ...
+ ...
+ if (...) {
+ DoSomething(
+ argument1, argument2, // 4 space indent
+ argument3, argument4);
+ }
+</pre>
+
+<p>Put multiple arguments on a single line to reduce the
+number of lines necessary for calling a function unless
+there is a specific readability problem. Some find that
+formatting with strictly one argument on each line is
+more readable and simplifies editing of the arguments.
+However, we prioritize for the reader over the ease of
+editing arguments, and most readability problems are
+better addressed with the following techniques.</p>
+
+<p>If having multiple arguments in a single line decreases
+readability due to the complexity or confusing nature of the
+expressions that make up some arguments, try creating
+variables that capture those arguments in a descriptive name:</p>
+<pre>int my_heuristic = scores[x] * y + bases[x];
+bool retval = DoSomething(my_heuristic, x, y, z);
+</pre>
+
+<p>Or put the confusing argument on its own line with
+an explanatory comment:</p>
+<pre>bool retval = DoSomething(scores[x] * y + bases[x], // Score heuristic.
+ x, y, z);
+</pre>
+
+<p>If there is still a case where one argument is
+significantly more readable on its own line, then put it on
+its own line. The decision should be specific to the argument
+which is made more readable rather than a general policy.</p>
+
+<p>Sometimes arguments form a structure that is important
+for readability. In those cases, feel free to format the
+arguments according to that structure:</p>
+<pre>// Transform the widget by a 3x3 matrix.
+my_widget.Transform(x1, x2, x3,
+ y1, y2, y3,
+ z1, z2, z3);
+</pre>
+
+</div>
+
+<h3 id="Braced_Initializer_List_Format">Braced Initializer List Format</h3>
+
+<div class="summary">
+<p>Format a <a href="#Braced_Initializer_List">braced initializer list</a>
+exactly like you would format a function call in its place.</p>
+</div>
+
+<div class="stylebody">
+
+<p>If the braced list follows a name (e.g. a type or
+variable name), format as if the <code>{}</code> were the
+parentheses of a function call with that name. If there
+is no name, assume a zero-length name.</p>
+
+<pre>// Examples of braced init list on a single line.
+return {foo, bar};
+functioncall({foo, bar});
+pair<int, int> p{foo, bar};
+
+// When you have to wrap.
+SomeFunction(
+ {"assume a zero-length name before {"},
+ some_other_function_parameter);
+SomeType variable{
+ some, other, values,
+ {"assume a zero-length name before {"},
+ SomeOtherType{
+ "Very long string requiring the surrounding breaks.",
+ some, other values},
+ SomeOtherType{"Slightly shorter string",
+ some, other, values}};
+SomeType variable{
+ "This is too long to fit all in one line"};
+MyType m = { // Here, you could also break before {.
+ superlongvariablename1,
+ superlongvariablename2,
+ {short, interior, list},
+ {interiorwrappinglist,
+ interiorwrappinglist2}};
+</pre>
+
+</div>
+
+<h3 id="Conditionals">Conditionals</h3>
+
+<div class="summary">
+<p>Prefer no spaces inside parentheses. The <code>if</code>
+and <code>else</code> keywords belong on separate lines.</p>
+</div>
+
+<div class="stylebody">
+
+<p>There are two acceptable formats for a basic
+conditional statement. One includes spaces between the
+parentheses and the condition, and one does not.</p>
+
+<p>The most common form is without spaces. Either is
+fine, but <em>be consistent</em>. If you are modifying a
+file, use the format that is already present. If you are
+writing new code, use the format that the other files in
+that directory or project use. If in doubt and you have
+no personal preference, do not add the spaces.</p>
+
+<pre>if (condition) { // no spaces inside parentheses
+ ... // 2 space indent.
+} else if (...) { // The else goes on the same line as the closing brace.
+ ...
+} else {
+ ...
+}
+</pre>
+
+<p>If you prefer you may add spaces inside the
+parentheses:</p>
+
+<pre>if ( condition ) { // spaces inside parentheses - rare
+ ... // 2 space indent.
+} else { // The else goes on the same line as the closing brace.
+ ...
+}
+</pre>
+
+<p>Note that in all cases you must have a space between
+the <code>if</code> and the open parenthesis. You must
+also have a space between the close parenthesis and the
+curly brace, if you're using one.</p>
+
+<pre class="badcode">if(condition) { // Bad - space missing after IF.
+if (condition){ // Bad - space missing before {.
+if(condition){ // Doubly bad.
+</pre>
+
+<pre>if (condition) { // Good - proper space after IF and before {.
+</pre>
+
+<p>Short conditional statements may be written on one
+line if this enhances readability. You may use this only
+when the line is brief and the statement does not use the
+<code>else</code> clause.</p>
+
+<pre>if (x == kFoo) return new Foo();
+if (x == kBar) return new Bar();
+</pre>
+
+<p>This is not allowed when the if statement has an
+<code>else</code>:</p>
+
+<pre class="badcode">// Not allowed - IF statement on one line when there is an ELSE clause
+if (x) DoThis();
+else DoThat();
+</pre>
+
+<p>In general, curly braces are not required for
+single-line statements, but they are allowed if you like
+them; conditional or loop statements with complex
+conditions or statements may be more readable with curly
+braces. Some
+projects require that an
+<code>if</code> must always always have an accompanying
+brace.</p>
+
+<pre>if (condition)
+ DoSomething(); // 2 space indent.
+
+if (condition) {
+ DoSomething(); // 2 space indent.
+}
+</pre>
+
+<p>However, if one part of an
+<code>if</code>-<code>else</code> statement uses curly
+braces, the other part must too:</p>
+
+<pre class="badcode">// Not allowed - curly on IF but not ELSE
+if (condition) {
+ foo;
+} else
+ bar;
+
+// Not allowed - curly on ELSE but not IF
+if (condition)
+ foo;
+else {
+ bar;
+}
+</pre>
+
+<pre>// Curly braces around both IF and ELSE required because
+// one of the clauses used braces.
+if (condition) {
+ foo;
+} else {
+ bar;
+}
+</pre>
+
+</div>
+
+<h3 id="Loops_and_Switch_Statements">Loops and Switch Statements</h3>
+
+<div class="summary">
+<p>Switch statements may use braces for blocks. Annotate
+non-trivial fall-through between cases.
+Braces are optional for single-statement loops.
+Empty loop bodies should use <code>{}</code> or <code>continue</code>.</p>
+</div>
+
+<div class="stylebody">
+
+<p><code>case</code> blocks in <code>switch</code>
+statements can have curly braces or not, depending on
+your preference. If you do include curly braces they
+should be placed as shown below.</p>
+
+<p>If not conditional on an enumerated value, switch
+statements should always have a <code>default</code> case
+(in the case of an enumerated value, the compiler will
+warn you if any values are not handled). If the default
+case should never execute, simply
+<code>assert</code>:</p>
+
+
+
+<div>
+<pre>switch (var) {
+ case 0: { // 2 space indent
+ ... // 4 space indent
+ break;
+ }
+ case 1: {
+ ...
+ break;
+ }
+ default: {
+ assert(false);
+ }
+}
+</pre>
+</div>
+
+
+
+
+
+<p> Braces are optional for single-statement loops.</p>
+
+<pre>for (int i = 0; i < kSomeNumber; ++i)
+ printf("I love you\n");
+
+for (int i = 0; i < kSomeNumber; ++i) {
+ printf("I take it back\n");
+}
+</pre>
+
+
+<p>Empty loop bodies should use <code>{}</code> or
+<code>continue</code>, but not a single semicolon.</p>
+
+<pre>while (condition) {
+ // Repeat test until it returns false.
+}
+for (int i = 0; i < kSomeNumber; ++i) {} // Good - empty body.
+while (condition) continue; // Good - continue indicates no logic.
+</pre>
+
+<pre class="badcode">while (condition); // Bad - looks like part of do/while loop.
+</pre>
+
+</div>
+
+<h3 id="Pointer_and_Reference_Expressions">Pointer and Reference Expressions</h3>
+
+<div class="summary">
+<p>No spaces around period or arrow. Pointer operators do not
+have trailing spaces.</p>
+</div>
+
+<div class="stylebody">
+
+<p>The following are examples of correctly-formatted
+pointer and reference expressions:</p>
+
+<pre>x = *p;
+p = &x;
+x = r.y;
+x = r->y;
+</pre>
+
+<p>Note that:</p>
+
+<ul>
+ <li>There are no spaces around the period or arrow when
+ accessing a member.</li>
+
+ <li>Pointer operators have no space after the
+ <code>*</code> or <code>&</code>.</li>
+</ul>
+
+<p>When declaring a pointer variable or argument, you may
+place the asterisk adjacent to either the type or to the
+variable name:</p>
+
+<pre>// These are fine, space preceding.
+char *c;
+const string &str;
+
+// These are fine, space following.
+char* c; // but remember to do "char* c, *d, *e, ...;"!
+const string& str;
+</pre>
+
+<pre class="badcode">char * c; // Bad - spaces on both sides of *
+const string & str; // Bad - spaces on both sides of &
+</pre>
+
+<p>You should do this consistently within a single
+file,
+so, when modifying an existing file, use the style in
+that file.</p>
+
+</div>
+
+<h3 id="Boolean_Expressions">Boolean Expressions</h3>
+
+<div class="summary">
+<p>When you have a boolean expression that is longer than the
+<a href="#Line_Length">standard line length</a>, be
+consistent in how you break up the lines.</p>
+</div>
+
+<div class="stylebody">
+
+<p>In this example, the logical AND operator is always at
+the end of the lines:</p>
+
+<pre>if (this_one_thing > this_other_thing &&
+ a_third_thing == a_fourth_thing &&
+ yet_another && last_one) {
+ ...
+}
+</pre>
+
+<p>Note that when the code wraps in this example, both of
+the <code>&&</code> logical AND operators are at
+the end of the line. This is more common in Google code,
+though wrapping all operators at the beginning of the
+line is also allowed. Feel free to insert extra
+parentheses judiciously because they can be very helpful
+in increasing readability when used
+appropriately. Also note that you should always use
+the punctuation operators, such as
+<code>&&</code> and <code>~</code>, rather than
+the word operators, such as <code>and</code> and
+<code>compl</code>.</p>
+
+</div>
+
+<h3 id="Return_Values">Return Values</h3>
+
+<div class="summary">
+<p>Do not needlessly surround the <code>return</code>
+expression with parentheses.</p>
+</div>
+
+<div class="stylebody">
+
+<p>Use parentheses in <code>return expr;</code> only
+where you would use them in <code>x = expr;</code>.</p>
+
+<pre>return result; // No parentheses in the simple case.
+// Parentheses OK to make a complex expression more readable.
+return (some_long_condition &&
+ another_condition);
+</pre>
+
+<pre class="badcode">return (value); // You wouldn't write var = (value);
+return(result); // return is not a function!
+</pre>
+
+</div>
+
+
+
+<h3 id="Variable_and_Array_Initialization">Variable and Array Initialization</h3>
+
+<div class="summary">
+<p>Your choice of <code>=</code>, <code>()</code>, or
+<code>{}</code>.</p>
+</div>
+
+<div class="stylebody">
+
+<p>You may choose between <code>=</code>,
+<code>()</code>, and <code>{}</code>; the following are
+all correct:</p>
+
+<pre>int x = 3;
+int x(3);
+int x{3};
+string name = "Some Name";
+string name("Some Name");
+string name{"Some Name"};
+</pre>
+
+<p>Be careful when using a braced initialization list <code>{...}</code>
+on a type with an <code>std::initializer_list</code> constructor.
+A nonempty <i>braced-init-list</i> prefers the
+<code>std::initializer_list</code> constructor whenever
+possible. Note that empty braces <code>{}</code> are special, and
+will call a default constructor if available. To force the
+non-<code>std::initializer_list</code> constructor, use parentheses
+instead of braces.</p>
+
+<pre>vector<int> v(100, 1); // A vector of 100 1s.
+vector<int> v{100, 1}; // A vector of 100, 1.
+</pre>
+
+<p>Also, the brace form prevents narrowing of integral
+types. This can prevent some types of programming
+errors.</p>
+
+<pre>int pi(3.14); // OK -- pi == 3.
+int pi{3.14}; // Compile error: narrowing conversion.
+</pre>
+
+</div>
+
+<h3 id="Preprocessor_Directives">Preprocessor Directives</h3>
+
+<div class="summary">
+<p>The hash mark that starts a preprocessor directive should
+always be at the beginning of the line.</p>
+</div>
+
+<div class="stylebody">
+
+<p>Even when preprocessor directives are within the body
+of indented code, the directives should start at the
+beginning of the line.</p>
+
+<pre>// Good - directives at beginning of line
+ if (lopsided_score) {
+#if DISASTER_PENDING // Correct -- Starts at beginning of line
+ DropEverything();
+# if NOTIFY // OK but not required -- Spaces after #
+ NotifyClient();
+# endif
+#endif
+ BackToNormal();
+ }
+</pre>
+
+<pre class="badcode">// Bad - indented directives
+ if (lopsided_score) {
+ #if DISASTER_PENDING // Wrong! The "#if" should be at beginning of line
+ DropEverything();
+ #endif // Wrong! Do not indent "#endif"
+ BackToNormal();
+ }
+</pre>
+
+</div>
+
+<h3 id="Class_Format">Class Format</h3>
+
+<div class="summary">
+<p>Sections in <code>public</code>, <code>protected</code> and
+<code>private</code> order, each indented one space.</p>
+</div>
+
+<div class="stylebody">
+
+<p>The basic format for a class declaration (lacking the
+comments, see <a href="#Class_Comments">Class
+Comments</a> for a discussion of what comments are
+needed) is:</p>
+
+<pre>class MyClass : public OtherClass {
+ public: // Note the 1 space indent!
+ MyClass(); // Regular 2 space indent.
+ explicit MyClass(int var);
+ ~MyClass() {}
+
+ void SomeFunction();
+ void SomeFunctionThatDoesNothing() {
+ }
+
+ void set_some_var(int var) { some_var_ = var; }
+ int some_var() const { return some_var_; }
+
+ private:
+ bool SomeInternalFunction();
+
+ int some_var_;
+ int some_other_var_;
+};
+</pre>
+
+<p>Things to note:</p>
+
+<ul>
+ <li>Any base class name should be on the same line as
+ the subclass name, subject to the 80-column limit.</li>
+
+ <li>The <code>public:</code>, <code>protected:</code>,
+ and <code>private:</code> keywords should be indented
+ one space.</li>
+
+ <li>Except for the first instance, these keywords
+ should be preceded by a blank line. This rule is
+ optional in small classes.</li>
+
+ <li>Do not leave a blank line after these
+ keywords.</li>
+
+ <li>The <code>public</code> section should be first,
+ followed by the <code>protected</code> and finally the
+ <code>private</code> section.</li>
+
+ <li>See <a href="#Declaration_Order">Declaration
+ Order</a> for rules on ordering declarations within
+ each of these sections.</li>
+</ul>
+
+</div>
+
+<h3 id="Constructor_Initializer_Lists">Constructor Initializer Lists</h3>
+
+<div class="summary">
+<p>Constructor initializer lists can be all on one line or
+with subsequent lines indented four spaces.</p>
+</div>
+
+<div class="stylebody">
+
+<p>There are two acceptable formats for initializer
+lists:</p>
+
+<pre>// When it all fits on one line:
+MyClass::MyClass(int var) : some_var_(var), some_other_var_(var + 1) {}
+</pre>
+
+<p>or</p>
+
+<pre>// When it requires multiple lines, indent 4 spaces, putting the colon on
+// the first initializer line:
+MyClass::MyClass(int var)
+ : some_var_(var), // 4 space indent
+ some_other_var_(var + 1) { // lined up
+ ...
+ DoSomething();
+ ...
+}
+</pre>
+
+</div>
+
+<h3 id="Namespace_Formatting">Namespace Formatting</h3>
+
+<div class="summary">
+<p>The contents of namespaces are not indented.</p>
+</div>
+
+<div class="stylebody">
+
+<p><a href="#Namespaces">Namespaces</a> do not add an
+extra level of indentation. For example, use:</p>
+
+<pre>namespace {
+
+void foo() { // Correct. No extra indentation within namespace.
+ ...
+}
+
+} // namespace
+</pre>
+
+<p>Do not indent within a namespace:</p>
+
+<pre class="badcode">namespace {
+
+ // Wrong. Indented when it should not be.
+ void foo() {
+ ...
+ }
+
+} // namespace
+</pre>
+
+<p>When declaring nested namespaces, put each namespace
+on its own line.</p>
+
+<pre>namespace foo {
+namespace bar {
+</pre>
+
+</div>
+
+<h3 id="Horizontal_Whitespace">Horizontal Whitespace</h3>
+
+<div class="summary">
+<p>Use of horizontal whitespace depends on location. Never put
+trailing whitespace at the end of a line.</p>
+</div>
+
+<div class="stylebody">
+
+<h4 class="stylepoint_subsection">General</h4>
+
+<pre>void f(bool b) { // Open braces should always have a space before them.
+ ...
+int i = 0; // Semicolons usually have no space before them.
+// Spaces inside braces for braced-init-list are optional. If you use them,
+// put them on both sides!
+int x[] = { 0 };
+int x[] = {0};
+
+// Spaces around the colon in inheritance and initializer lists.
+class Foo : public Bar {
+ public:
+ // For inline function implementations, put spaces between the braces
+ // and the implementation itself.
+ Foo(int b) : Bar(), baz_(b) {} // No spaces inside empty braces.
+ void Reset() { baz_ = 0; } // Spaces separating braces from implementation.
+ ...
+</pre>
+
+<p>Adding trailing whitespace can cause extra work for
+others editing the same file, when they merge, as can
+removing existing trailing whitespace. So: Don't
+introduce trailing whitespace. Remove it if you're
+already changing that line, or do it in a separate
+clean-up
+operation (preferably when no-one
+else is working on the file).</p>
+
+<h4 class="stylepoint_subsection">Loops and Conditionals</h4>
+
+<pre>if (b) { // Space after the keyword in conditions and loops.
+} else { // Spaces around else.
+}
+while (test) {} // There is usually no space inside parentheses.
+switch (i) {
+for (int i = 0; i < 5; ++i) {
+// Loops and conditions may have spaces inside parentheses, but this
+// is rare. Be consistent.
+switch ( i ) {
+if ( test ) {
+for ( int i = 0; i < 5; ++i ) {
+// For loops always have a space after the semicolon. They may have a space
+// before the semicolon, but this is rare.
+for ( ; i < 5 ; ++i) {
+ ...
+
+// Range-based for loops always have a space before and after the colon.
+for (auto x : counts) {
+ ...
+}
+switch (i) {
+ case 1: // No space before colon in a switch case.
+ ...
+ case 2: break; // Use a space after a colon if there's code after it.
+</pre>
+
+<h4 class="stylepoint_subsection">Operators</h4>
+
+<pre>// Assignment operators always have spaces around them.
+x = 0;
+
+// Other binary operators usually have spaces around them, but it's
+// OK to remove spaces around factors. Parentheses should have no
+// internal padding.
+v = w * x + y / z;
+v = w*x + y/z;
+v = w * (x + z);
+
+// No spaces separating unary operators and their arguments.
+x = -5;
+++x;
+if (x && !y)
+ ...
+</pre>
+
+<h4 class="stylepoint_subsection">Templates and Casts</h4>
+
+<pre>// No spaces inside the angle brackets (< and >), before
+// <, or between >( in a cast
+vector<string> x;
+y = static_cast<char*>(x);
+
+// Spaces between type and pointer are OK, but be consistent.
+vector<char *> x;
+set<list<string>> x; // Permitted in C++11 code.
+set<list<string> > x; // C++03 required a space in > >.
+
+// You may optionally use symmetric spacing in < <.
+set< list<string> > x;
+</pre>
+
+</div>
+
+<h3 id="Vertical_Whitespace">Vertical Whitespace</h3>
+
+<div class="summary">
+<p>Minimize use of vertical whitespace.</p>
+</div>
+
+<div class="stylebody">
+
+<p>This is more a principle than a rule: don't use blank
+lines when you don't have to. In particular, don't put
+more than one or two blank lines between functions,
+resist starting functions with a blank line, don't end
+functions with a blank line, and be discriminating with
+your use of blank lines inside functions.</p>
+
+<p>The basic principle is: The more code that fits on one
+screen, the easier it is to follow and understand the
+control flow of the program. Of course, readability can
+suffer from code being too dense as well as too spread
+out, so use your judgement. But in general, minimize use
+of vertical whitespace.</p>
+
+<p>Some rules of thumb to help when blank lines may be
+useful:</p>
+
+<ul>
+ <li>Blank lines at the beginning or end of a function
+ very rarely help readability.</li>
+
+ <li>Blank lines inside a chain of if-else blocks may
+ well help readability.</li>
+</ul>
+
+</div>
+
+<h2 id="Exceptions_to_the_Rules">Exceptions to the Rules</h2>
+
+<p>The coding conventions described above are mandatory.
+However, like all good rules, these sometimes have exceptions,
+which we discuss here.</p>
+
+
+
+<div>
+<h3 id="Existing_Non-conformant_Code">Existing Non-conformant Code</h3>
+
+<div class="summary">
+<p>You may diverge from the rules when dealing with code that
+does not conform to this style guide.</p>
+</div>
+
+<div class="stylebody">
+
+<p>If you find yourself modifying code that was written
+to specifications other than those presented by this
+guide, you may have to diverge from these rules in order
+to stay consistent with the local conventions in that
+code. If you are in doubt about how to do this, ask the
+original author or the person currently responsible for
+the code. Remember that <em>consistency</em> includes
+local consistency, too.</p>
+
+</div>
+</div>
+
+
+<h2 class="ignoreLink">Parting Words</h2>
+
+<p>Use common sense and <em>BE CONSISTENT</em>.</p>
+
+<p>If you are editing code, take a few minutes to look at the
+code around you and determine its style. If they use spaces
+around their <code>if</code> clauses, you should, too. If their
+comments have little boxes of stars around them, make your
+comments have little boxes of stars around them too.</p>
+
+<p>The point of having style guidelines is to have a common
+vocabulary of coding so people can concentrate on what you are
+saying, rather than on how you are saying it. We present global
+style rules here so people know the vocabulary. But local style
+is also important. If code you add to a file looks drastically
+different from the existing code around it, the discontinuity
+throws readers out of their rhythm when they go to read it. Try
+to avoid this.</p>
+
+
+
+<p>OK, enough writing about writing code; the code itself is much
+more interesting. Have fun!</p>
+
+<hr>
+
+<p style="text-align:right; font-style:italic;">Revision 4.45</p>
+
+</div>
+</body></html>
diff --git a/styleguide/eclipse-cpp-google-style.xml b/styleguide/eclipse-cpp-google-style.xml
new file mode 100644
index 0000000..aa05a81
--- /dev/null
+++ b/styleguide/eclipse-cpp-google-style.xml
@@ -0,0 +1,167 @@
+<?xml version="1.0" encoding="UTF-8" standalone="no"?>
+<profiles version="1">
+<profile kind="CodeFormatterProfile" name="Google C++" version="1">
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_opening_paren_in_method_declaration" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_opening_paren_in_for" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_new_line_in_empty_block" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.lineSplit" value="80"/>
+<setting id="org.eclipse.cdt.core.formatter.alignment_for_member_access" value="16"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_comma_in_base_types" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.keep_else_statement_on_same_line" value="false"/>
+<setting id="org.eclipse.cdt.core.formatter.indent_switchstatements_compare_to_switch" value="true"/>
+<setting id="org.eclipse.cdt.core.formatter.alignment_for_constructor_initializer_list" value="83"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_opening_brace_in_array_initializer" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_comma_in_method_declaration_parameters" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_opening_paren_in_if" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_closing_paren_in_parenthesized_expression" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_opening_paren_in_exception_specification" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_comma_in_base_types" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.indent_body_declarations_compare_to_access_specifier" value="true"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_closing_paren_in_exception_specification" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_comma_in_template_arguments" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_opening_brace_in_block" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_closing_paren_in_method_declaration" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.use_tabs_only_for_leading_indentations" value="false"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_colon_in_labeled_statement" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_colon_in_case" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.comment.min_distance_between_code_and_line_comment" value="2"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_comma_in_array_initializer" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_comma_in_enum_declarations" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.alignment_for_expressions_in_array_initializer" value="16"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_comma_in_declarator_list" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_opening_bracket" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_closing_paren_in_for" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_prefix_operator" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.tabulation.size" value="2"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_new_line_before_else_in_if_statement" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.alignment_for_enumerator_list" value="51"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_opening_paren_in_parenthesized_expression" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_between_empty_parens_in_method_declaration" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_closing_paren_in_switch" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.alignment_for_declarator_list" value="16"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_opening_paren_in_parenthesized_expression" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.indent_empty_lines" value="false"/>
+<setting id="org.eclipse.cdt.core.formatter.indent_switchstatements_compare_to_cases" value="true"/>
+<setting id="org.eclipse.cdt.core.formatter.keep_empty_array_initializer_on_one_line" value="false"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_opening_brace_in_method_declaration" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.put_empty_statement_on_new_line" value="true"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_opening_brace_in_switch" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_closing_paren_in_cast" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_between_empty_braces_in_array_initializer" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.brace_position_for_method_declaration" value="end_of_line"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_closing_paren_in_while" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_question_in_conditional" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_semicolon" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_closing_angle_bracket_in_template_arguments" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_colon_in_base_clause" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.indent_breaks_compare_to_cases" value="true"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_unary_operator" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.join_wrapped_lines" value="true"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_comma_in_declarator_list" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.alignment_for_arguments_in_method_invocation" value="18"/>
+<setting id="org.eclipse.cdt.core.formatter.comment.never_indent_line_comments_on_first_column" value="true"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_opening_paren_in_while" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_between_empty_brackets" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_opening_bracket" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.alignment_for_parameters_in_method_declaration" value="18"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_new_line_before_closing_brace_in_array_initializer" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.number_of_empty_lines_to_preserve" value="1"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_opening_paren_in_method_invocation" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_closing_brace_in_array_initializer" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_semicolon_in_for" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_colon_in_conditional" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.brace_position_for_block" value="end_of_line"/>
+<setting id="org.eclipse.cdt.core.formatter.comment.preserve_white_space_between_code_and_line_comments" value="true"/>
+<setting id="org.eclipse.cdt.core.formatter.brace_position_for_type_declaration" value="end_of_line"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_assignment_operator" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_opening_angle_bracket_in_template_arguments" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_comma_in_expression_list" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_opening_angle_bracket_in_template_parameters" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.continuation_indentation" value="2"/>
+<setting id="org.eclipse.cdt.core.formatter.alignment_for_expression_list" value="0"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_opening_paren_in_method_declaration" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_comma_in_template_parameters" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_colon_in_default" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_binary_operator" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.alignment_for_conditional_expression" value="34"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_between_empty_parens_in_method_invocation" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_comma_in_array_initializer" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_closing_paren_in_if" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.format_guardian_clause_on_one_line" value="false"/>
+<setting id="org.eclipse.cdt.core.formatter.indent_access_specifier_extra_spaces" value="1"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_opening_paren_in_cast" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.indent_access_specifier_compare_to_type_header" value="false"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_opening_brace_in_type_declaration" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.continuation_indentation_for_array_initializer" value="2"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_colon_in_labeled_statement" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_comma_in_method_declaration_parameters" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_semicolon_in_for" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_closing_paren_in_method_invocation" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.indent_body_declarations_compare_to_namespace_header" value="false"/>
+<setting id="org.eclipse.cdt.core.formatter.alignment_for_compact_if" value="0"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_assignment_operator" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_closing_brace_in_block" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_opening_brace_in_array_initializer" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_new_line_at_end_of_file_if_missing" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.alignment_for_assignment" value="16"/>
+<setting id="org.eclipse.cdt.core.formatter.alignment_for_conditional_expression_chain" value="18"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_comma_in_template_parameters" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_comma_in_expression_list" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_question_in_conditional" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_opening_paren_in_exception_specification" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_binary_operator" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_new_line_before_identifier_in_function_declaration" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.alignment_for_base_clause_in_type_declaration" value="16"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_comma_in_method_declaration_throws" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_between_empty_parens_in_exception_specification" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_comma_in_method_invocation_arguments" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.indent_declaration_compare_to_template_header" value="false"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_unary_operator" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_opening_paren_in_switch" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.indent_statements_compare_to_body" value="true"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_comma_in_method_declaration_throws" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.alignment_for_binary_expression" value="16"/>
+<setting id="org.eclipse.cdt.core.formatter.indent_statements_compare_to_block" value="true"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_comma_in_template_arguments" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_new_line_before_catch_in_try_statement" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.alignment_for_throws_clause_in_method_declaration" value="16"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_opening_paren_in_method_invocation" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_closing_paren_in_cast" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_closing_paren_in_catch" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_opening_angle_bracket_in_template_parameters" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.tabulation.char" value="space"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_closing_angle_bracket_in_template_parameters" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_new_line_before_colon_in_constructor_initializer_list" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_opening_paren_in_while" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_comma_in_method_invocation_arguments" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.brace_position_for_block_in_case" value="end_of_line"/>
+<setting id="org.eclipse.cdt.core.formatter.compact_else_if" value="true"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_postfix_operator" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_new_line_after_template_declaration" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_colon_in_base_clause" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_opening_paren_in_catch" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.keep_then_statement_on_same_line" value="false"/>
+<setting id="org.eclipse.cdt.core.formatter.brace_position_for_switch" value="end_of_line"/>
+<setting id="org.eclipse.cdt.core.formatter.alignment_for_overloaded_left_shift_chain" value="18"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_opening_paren_in_if" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_opening_paren_in_switch" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.keep_imple_if_on_one_line" value="false"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_new_line_after_opening_brace_in_array_initializer" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.indentation.size" value="2"/>
+<setting id="org.eclipse.cdt.core.formatter.brace_position_for_namespace_declaration" value="end_of_line"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_colon_in_conditional" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_comma_in_enum_declarations" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_prefix_operator" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_closing_angle_bracket_in_template_arguments" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.brace_position_for_array_initializer" value="end_of_line"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_colon_in_case" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_opening_paren_in_catch" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_opening_brace_in_namespace_declaration" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_postfix_operator" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_closing_bracket" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_new_line_before_while_in_do_statement" value="do not insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_before_opening_paren_in_for" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_closing_angle_bracket_in_template_parameters" value="insert"/>
+<setting id="org.eclipse.cdt.core.formatter.insert_space_after_opening_angle_bracket_in_template_arguments" value="do not insert"/>
+</profile>
+</profiles>
diff --git a/styleguide/eclipse-java-google-style.xml b/styleguide/eclipse-java-google-style.xml
new file mode 100644
index 0000000..b3177d3
--- /dev/null
+++ b/styleguide/eclipse-java-google-style.xml
@@ -0,0 +1,337 @@
+<?xml version="1.0" encoding="UTF-8" standalone="no"?>
+<profiles version="12">
+<profile kind="CodeFormatterProfile" name="GoogleStyle" version="12">
+<setting id="org.eclipse.jdt.core.formatter.comment.insert_new_line_before_root_tags" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.disabling_tag" value="@formatter:off"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_annotation" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_type_parameters" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_type_declaration" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_type_arguments" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.brace_position_for_anonymous_type_declaration" value="end_of_line"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_colon_in_case" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_brace_in_array_initializer" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.comment.new_lines_at_block_boundaries" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_cascading_method_invocation_with_arguments.count_dependent" value="16|-1|16"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_in_empty_annotation_declaration" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_annotation" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.blank_lines_before_field" value="0"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_while" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.use_on_off_tags" value="false"/>
+<setting id="org.eclipse.jdt.core.formatter.wrap_prefer_two_fragments" value="false"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_between_empty_parens_in_annotation_type_member_declaration" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_before_else_in_if_statement" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_prefix_operator" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.keep_else_statement_on_same_line" value="false"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_ellipsis" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.comment.insert_new_line_for_parameter" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.wrap_comment_inline_tags" value="false"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_annotation_type_declaration" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.indent_breaks_compare_to_cases" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_at_in_annotation" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_local_variable_declaration" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_multiple_fields" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_annotations_on_parameter" value="1040"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_expressions_in_array_initializer" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_annotations_on_type.count_dependent" value="1585|-1|1585"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_conditional_expression" value="80"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_for" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_multiple_fields.count_dependent" value="16|-1|16"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_binary_operator" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_question_in_wildcard" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.brace_position_for_array_initializer" value="end_of_line"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_between_empty_parens_in_enum_constant" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_before_finally_in_try_statement" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_after_annotation_on_local_variable" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_before_catch_in_try_statement" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_while" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.blank_lines_after_package" value="1"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_arguments_in_qualified_allocation_expression.count_dependent" value="16|4|80"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_throws_clause_in_method_declaration.count_dependent" value="16|4|48"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_type_parameters" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.continuation_indentation" value="2"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_superinterfaces_in_enum_declaration.count_dependent" value="16|4|49"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_postfix_operator" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_arguments_in_method_invocation" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_angle_bracket_in_type_arguments" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_superinterfaces" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.blank_lines_before_new_chunk" value="1"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_binary_operator" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.blank_lines_before_package" value="0"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_cascading_method_invocation_with_arguments" value="16"/>
+<setting id="org.eclipse.jdt.core.compiler.source" value="1.8"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_throws_clause_in_constructor_declaration.count_dependent" value="16|4|48"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_enum_constant_arguments" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_constructor_declaration" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.comment.format_line_comments" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_closing_angle_bracket_in_type_arguments" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_enum_declarations" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.join_wrapped_lines" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_block" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_arguments_in_explicit_constructor_call" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.wrap_non_simple_local_variable_annotation" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_method_invocation_arguments" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.align_type_members_on_columns" value="false"/>
+<setting id="org.eclipse.jdt.core.formatter.blank_lines_before_member_type" value="0"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_enum_constant" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_enum_constants.count_dependent" value="16|5|48"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_for" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_method_declaration" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_selector_in_method_invocation" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_switch" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_unary_operator" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_colon_in_case" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.comment.indent_parameter_description" value="false"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_method_declaration" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_switch" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_enum_declaration" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_angle_bracket_in_type_parameters" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.comment.clear_blank_lines_in_block_comment" value="false"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_in_empty_type_declaration" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.lineSplit" value="80"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_if" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_selector_in_method_invocation.count_dependent" value="16|4|48"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_between_brackets_in_array_type_reference" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_parenthesized_expression" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_explicitconstructorcall_arguments" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_constructor_declaration" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.blank_lines_before_first_class_body_declaration" value="0"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_after_annotation_on_method" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.indentation.size" value="4"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_between_empty_parens_in_method_declaration" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.enabling_tag" value="@formatter:on"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_enum_constant" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_annotations_on_package" value="1585"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_superclass_in_type_declaration" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_assignment" value="16"/>
+<setting id="org.eclipse.jdt.core.compiler.problem.assertIdentifier" value="error"/>
+<setting id="org.eclipse.jdt.core.formatter.tabulation.char" value="space"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_semicolon_in_try_resources" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_constructor_declaration_parameters" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_prefix_operator" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.indent_statements_compare_to_body" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.blank_lines_before_method" value="1"/>
+<setting id="org.eclipse.jdt.core.formatter.wrap_outer_expressions_when_nested" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.wrap_non_simple_type_annotation" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.format_guardian_clause_on_one_line" value="false"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_colon_in_for" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_field_declaration" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_cast" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_parameters_in_constructor_declaration" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_colon_in_labeled_statement" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.brace_position_for_annotation_type_declaration" value="end_of_line"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_in_empty_method_body" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_method_declaration" value="0"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_try" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_method_invocation" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_bracket_in_array_allocation_expression" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_enum_constant" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_annotation" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_at_in_annotation_type_declaration" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_method_declaration_throws" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_if" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.brace_position_for_switch" value="end_of_line"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_method_declaration_throws" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_parenthesized_expression_in_return" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_annotation" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_question_in_conditional" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_question_in_wildcard" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_try" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_bracket_in_array_allocation_expression" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.comment.preserve_white_space_between_code_and_line_comments" value="false"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_parenthesized_expression_in_throw" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_type_arguments" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.compiler.problem.enumIdentifier" value="error"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_generic_type_arguments" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.indent_switchstatements_compare_to_switch" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.comment_new_line_at_start_of_html_paragraph" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_ellipsis" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.brace_position_for_block" value="end_of_line"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comment_prefix" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_for_inits" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.brace_position_for_method_declaration" value="end_of_line"/>
+<setting id="org.eclipse.jdt.core.formatter.compact_else_if" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.wrap_non_simple_parameter_annotation" value="false"/>
+<setting id="org.eclipse.jdt.core.formatter.wrap_before_or_operator_multicatch" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_array_initializer" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_for_increments" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_annotations_on_method" value="1585"/>
+<setting id="org.eclipse.jdt.core.formatter.format_line_comment_starting_on_first_column" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_bracket_in_array_reference" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_after_annotation_on_field" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.comment.indent_root_tags" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.brace_position_for_enum_constant" value="end_of_line"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_enum_declarations" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_union_type_in_multicatch" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_explicitconstructorcall_arguments" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_switch" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_method_declaration_parameters" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_superinterfaces" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_allocation_expression" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.tabulation.size" value="2"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_bracket_in_array_type_reference" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_after_opening_brace_in_array_initializer" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_closing_brace_in_block" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_bracket_in_array_reference" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_in_empty_enum_constant" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_angle_bracket_in_type_arguments" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_constructor_declaration" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_constructor_declaration_throws" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_if" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_arguments_in_method_invocation.count_dependent" value="16|5|80"/>
+<setting id="org.eclipse.jdt.core.formatter.comment.clear_blank_lines_in_javadoc_comment" value="false"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_annotations_on_parameter.count_dependent" value="1040|-1|1040"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_throws_clause_in_constructor_declaration" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_assignment_operator" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_assignment_operator" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_annotations_on_package.count_dependent" value="1585|-1|1585"/>
+<setting id="org.eclipse.jdt.core.formatter.indent_empty_lines" value="false"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_synchronized" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_closing_paren_in_cast" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_method_declaration_parameters" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.force_if_else_statement_brace" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.brace_position_for_block_in_case" value="end_of_line"/>
+<setting id="org.eclipse.jdt.core.formatter.number_of_empty_lines_to_preserve" value="3"/>
+<setting id="org.eclipse.jdt.core.formatter.wrap_non_simple_package_annotation" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_method_declaration" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_catch" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_constructor_declaration" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_method_invocation" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_bracket_in_array_reference" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_arguments_in_qualified_allocation_expression" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_arguments_in_annotation.count_dependent" value="16|-1|16"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_and_in_type_parameter" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_annotations_on_type" value="1585"/>
+<setting id="org.eclipse.jdt.core.compiler.compliance" value="1.8"/>
+<setting id="org.eclipse.jdt.core.formatter.continuation_indentation_for_array_initializer" value="2"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_between_empty_brackets_in_array_allocation_expression" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_at_in_annotation_type_declaration" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_arguments_in_allocation_expression" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_cast" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_unary_operator" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_new_anonymous_class" value="20"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_angle_bracket_in_parameterized_type_reference" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_annotations_on_local_variable.count_dependent" value="1585|-1|1585"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_anonymous_type_declaration" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.keep_empty_array_initializer_on_one_line" value="false"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_in_empty_enum_declaration" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_annotations_on_field.count_dependent" value="1585|-1|1585"/>
+<setting id="org.eclipse.jdt.core.formatter.keep_imple_if_on_one_line" value="false"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_constructor_declaration_parameters" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_closing_angle_bracket_in_type_parameters" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_colon_in_labeled_statement" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_at_end_of_file_if_missing" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_colon_in_for" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_parameterized_type_reference" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_parameters_in_constructor_declaration.count_dependent" value="16|5|80"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_superinterfaces_in_type_declaration" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.brace_position_for_enum_declaration" value="end_of_line"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_binary_expression" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_while" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_after_annotation_on_type" value="insert"/>
+<setting id="org.eclipse.jdt.core.compiler.codegen.inlineJsrBytecode" value="enabled"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_try" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.put_empty_statement_on_new_line" value="false"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_after_label" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_after_annotation_on_parameter" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_angle_bracket_in_type_parameters" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_between_empty_parens_in_method_invocation" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.comment.format_javadoc_comments" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_arguments_in_enum_constant" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_before_while_in_do_statement" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_arguments_in_enum_constant.count_dependent" value="16|-1|16"/>
+<setting id="org.eclipse.jdt.core.formatter.comment.line_length" value="100"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_after_annotation_on_package" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.blank_lines_between_import_groups" value="1"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_enum_constant_arguments" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_semicolon" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.brace_position_for_constructor_declaration" value="end_of_line"/>
+<setting id="org.eclipse.jdt.core.formatter.number_of_blank_lines_at_beginning_of_method_body" value="0"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_colon_in_conditional" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.indent_body_declarations_compare_to_type_header" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_annotation_type_member_declaration" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.wrap_before_binary_operator" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.blank_lines_between_type_declarations" value="2"/>
+<setting id="org.eclipse.jdt.core.formatter.indent_body_declarations_compare_to_enum_declaration_header" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_synchronized" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_superinterfaces_in_enum_declaration" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.indent_statements_compare_to_block" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.join_lines_in_comments" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_question_in_conditional" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_multiple_field_declarations" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_compact_if" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_for_inits" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.indent_switchstatements_compare_to_cases" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_array_initializer" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_colon_in_default" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_and_in_type_parameter" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_between_empty_parens_in_constructor_declaration" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.blank_lines_before_imports" value="0"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_colon_in_assert" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_annotations_on_field" value="1585"/>
+<setting id="org.eclipse.jdt.core.formatter.comment.format_html" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_throws_clause_in_method_declaration" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_angle_bracket_in_type_parameters" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_bracket_in_array_allocation_expression" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_in_empty_anonymous_type_declaration" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_colon_in_conditional" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_angle_bracket_in_parameterized_type_reference" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_for" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_expressions_in_array_initializer.count_dependent" value="16|5|80"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_postfix_operator" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.comment.format_source_code" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_synchronized" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_allocation_expression" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_constructor_declaration_throws" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_parameters_in_method_declaration" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_brace_in_array_initializer" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.compiler.codegen.targetPlatform" value="1.8"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_resources_in_try" value="80"/>
+<setting id="org.eclipse.jdt.core.formatter.use_tabs_only_for_leading_indentations" value="false"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_arguments_in_annotation" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.comment.format_header" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.comment.format_block_comments" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_enum_constant" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_enum_constants" value="0"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_parenthesized_expression" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.indent_body_declarations_compare_to_annotation_declaration_header" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_new_line_in_empty_block" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_parenthesized_expression" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_catch" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_multiple_local_declarations" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_superinterfaces_in_type_declaration.count_dependent" value="16|4|48"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_switch" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_comma_in_for_increments" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_annotations_on_method.count_dependent" value="1585|-1|1585"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_method_invocation" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_colon_in_assert" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.brace_position_for_type_declaration" value="end_of_line"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_brace_in_array_initializer" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_between_empty_braces_in_array_initializer" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_binary_expression.count_dependent" value="16|-1|16"/>
+<setting id="org.eclipse.jdt.core.formatter.wrap_non_simple_member_annotation" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_annotations_on_local_variable" value="1585"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_opening_paren_in_method_declaration" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_semicolon_in_for" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_arguments_in_explicit_constructor_call.count_dependent" value="16|5|80"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_paren_in_catch" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_angle_bracket_in_parameterized_type_reference" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_multiple_field_declarations" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_closing_paren_in_annotation" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_generic_type_arguments.count_dependent" value="16|-1|16"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_parameterized_type_reference" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_arguments_in_allocation_expression.count_dependent" value="16|5|80"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_method_invocation_arguments" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_parameters_in_method_declaration.count_dependent" value="16|5|80"/>
+<setting id="org.eclipse.jdt.core.formatter.comment.new_lines_at_javadoc_boundaries" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.blank_lines_after_imports" value="1"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_comma_in_multiple_local_declarations" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.indent_body_declarations_compare_to_enum_constant_header" value="true"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_after_semicolon_in_for" value="insert"/>
+<setting id="org.eclipse.jdt.core.formatter.never_indent_line_comments_on_first_column" value="false"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_semicolon_in_try_resources" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.alignment_for_for_statement" value="16"/>
+<setting id="org.eclipse.jdt.core.formatter.insert_space_before_opening_angle_bracket_in_type_arguments" value="do not insert"/>
+<setting id="org.eclipse.jdt.core.formatter.never_indent_block_comments_on_first_column" value="false"/>
+<setting id="org.eclipse.jdt.core.formatter.keep_then_statement_on_same_line" value="false"/>
+</profile>
+</profiles>
diff --git a/styleguide/include/link.png b/styleguide/include/link.png
new file mode 100644
index 0000000..75d5c7b
--- /dev/null
+++ b/styleguide/include/link.png
Binary files differ
diff --git a/styleguide/include/styleguide.css b/styleguide/include/styleguide.css
new file mode 100644
index 0000000..ef62024
--- /dev/null
+++ b/styleguide/include/styleguide.css
@@ -0,0 +1,261 @@
+/* General CSS */
+
+body {
+ background-color: #fff;
+ color: #333;
+ font-family: sans-serif;
+ font-size: 10pt;
+ margin-right: 100px;
+ margin-left: 100px;
+}
+
+h1 {
+ text-align: center;
+ font-size: 18pt;
+}
+
+h1, h2, h3, h4, h5, h6 {
+ color: #06c;
+ margin-top: 2em;
+ margin-bottom: 1em;
+ padding: 25px;
+ font-weight:bold;
+}
+
+h2,
+h3,
+h4,
+h5,
+h6 {
+ margin-top:1.5em;
+ margin-bottom:.75em;
+}
+
+h1 {font-size:200%;}
+h2 {font-size:167%;}
+h3 {font-size:133%;}
+h4 {font-size:120%;}
+h5 {font-size:110%;}
+
+
+table {
+ border: 1px solid #bbb;
+ border-spacing: 0;
+ border-collapse: collapse;
+ margin: 0 0 1.5em;
+ vertical-align: middle;
+ width: 100%
+}
+
+td, th {
+ border: 1px solid #ccc;
+ padding: 2px 12px;
+ font-size: 10pt;
+}
+
+code, samp, var {
+ background-color:#FAFAFA;
+ white-space: nowrap
+}
+
+pre {
+ padding:6px 10px;
+ background-color:#FAFAFA;
+ border:1px solid #bbb;
+ overflow:auto;
+}
+
+pre.prettyprint {
+ padding:6px 10px !important;
+ border:1px solid #bbb !important;
+}
+
+code.bad, code.badcode {
+ color: magenta;
+}
+
+pre.bad, pre.badcode {
+ background-color:#ffe6d8;
+ border-top:1px inset #a03;
+ border-left:1px inset #a03;
+}
+
+hr {
+ margin-top: 3.5em;
+ border-width: 1px;
+ color: #fff;
+}
+
+/* TOC CSS */
+
+table.columns {
+ border: none;
+}
+
+td.two_columns {
+ -webkit-column-count: 2;
+ column-count: 2;
+}
+
+.toc_category {
+ font-size: 10pt;
+ padding-top: 1em;
+ padding-bottom: 1em;
+ border-left-width: 2px;
+ border-right-width: 2px;
+ border-color: grey;
+}
+
+.toc_stylepoint {
+ font-size: 10pt;
+ padding-top: 1em;
+ padding-bottom: 1em;
+}
+
+li.toc_entry {
+ padding-right: 1em;
+ display: inline;
+ list-style-type: none;
+}
+
+/*
+ * This space is required to trigger the linewrap on the links
+ * at href boundaries
+ */
+li.toc_entry::after {
+ content: " ";
+ }
+
+li.toc_entry a {
+ white-space: nowrap;
+}
+
+/* Horizontal TOC */
+.toc td, .toc th {
+ border-width: 1px 5px;
+ overflow: hidden;
+}
+
+/* Vertical TOC */
+
+.toc td.two_columns {
+ border-width: 0px;
+}
+
+/* Special Sections */
+
+address {
+ text-align: right;
+}
+
+.revision {
+ text-align: right;
+}
+
+.headerbox {
+ margin-left: 50%;
+ font-size: 75%;
+}
+
+.legend {
+ padding-top: 1em;
+ margin-left: 50%;
+ font-size: 10pt;
+}
+
+.link_button {
+ float: left;
+ display: none;
+ background-color: #f8f8ff;
+ border-color: #f0f0ff;
+ border-style: solid;
+ border-width: 1px;
+ font-size: 75%;
+ margin-top: 0;
+ margin-left: -50px;
+ padding: 24px;
+ border-radius: 3px;
+ -webkit-border-radius: 3px;
+ -moz-border-radius: 3px;
+}
+
+.ignoreLink {
+ padding: 0px;
+}
+
+.divider{
+ width:5px;
+ height:auto;
+ display:inline-block;
+}
+
+/* Style Guide semantic CSS */
+
+.summary {
+ margin-top: 1em;
+ margin-bottom: 1em;
+}
+
+.stylebody {
+ margin-top: 1em;
+ margin-bottom: 1em;
+}
+
+.stylepoint_section {
+ display: block;
+ margin-bottom: 1em;
+ font-family: sans-serif;
+ font-weight: bold;
+}
+
+.stylepoint_subsection {
+ display: block;
+ margin-bottom: 1em;
+}
+
+.stylepoint_subsubsection {
+ display: block;
+ margin-bottom: 1em;
+}
+
+.definition:before {
+ content: "Definition: ";
+ font-weight: bold;
+ display: block;
+ margin-bottom: 1em;
+}
+
+.pros:before {
+ content: "Pros: ";
+ font-weight: bold;
+ display: block;
+ margin-bottom: 1em;
+}
+
+.cons:before {
+ content: "Cons: ";
+ font-weight: bold;
+ display: block;
+ margin-bottom: 1em;
+}
+
+.decision:before {
+ content: "Decision: ";
+ font-weight: bold;
+ display: block;
+ margin-bottom: 1em;
+}
+
+.exception:before {
+ content: "Exception: ";
+ font-weight: bold;
+ display: block;
+ margin-bottom: 1em;
+}
+
+.note:before {
+ content: "Note: ";
+ font-weight: bold;
+ display: block;
+ margin-bottom: 1em;
+}
diff --git a/styleguide/include/styleguide.js b/styleguide/include/styleguide.js
new file mode 100644
index 0000000..accb779
--- /dev/null
+++ b/styleguide/include/styleguide.js
@@ -0,0 +1,289 @@
+TocTypeEnum = {
+ VERTICAL: 1,
+ HORIZONTAL: 2
+};
+
+function CreateTOC(tocElement) {
+
+ // Find the toc element DIV. We'll place our TOC there.
+ var toc = document.getElementById(tocElement);
+
+ var tocTypeClass = toc.className;
+ var tocType;
+
+ switch (tocTypeClass) {
+ case 'horizontal_toc':
+ tocType = TocTypeEnum.HORIZONTAL;
+ break;
+ case 'vertical_toc':
+ tocType = TocTypeEnum.VERTICAL;
+ break;
+ default:
+ tocType = TocTypeEnum.VERTICAL;
+ break;
+ }
+
+ // If toc_levels is defined, set headingLevels to it.
+ // Otherwise, use default value of "h2,h3"
+ var headingLevels;
+ if (typeof toc_levels === 'undefined') {
+ headingLevels = 'h2,h3';
+ } else {
+
+ }
+
+ // Collect all section heading elements in an array
+ var headings = document.querySelectorAll(headingLevels);
+
+ // Add TOC title elements
+ var tocHeadingDiv = document.createElement('div');
+ toc.appendChild(tocHeadingDiv);
+ tocHeadingDiv.className = 'toc_title';
+ var tocHeading = document.createElement('h3');
+ toc.appendChild(tocHeading);
+ tocHeading.className = 'ignoreLink';
+ tocHeading.id = 'toc';
+ var tocText = document.createTextNode('Table of Contents');
+ tocHeading.appendChild(tocText);
+
+ // Add table and tbody
+ var tocTable = document.createElement('table');
+ if (tocType == TocTypeEnum.VERTICAL) {
+ tocTable.className = 'columns';
+ }
+ toc.appendChild(tocTable);
+
+ var tbody_element = document.createElement('tbody');
+ tbody_element.setAttribute('valign', 'top');
+ tbody_element.className = 'toc';
+ tocTable.appendChild(tbody_element);
+
+ // Get the highest level heading
+ var firstHeading = headings[0];
+ var masterLevel = parseInt(headingLevels.charAt(1));
+
+ // Get the lowest heading level
+ var lowestLevel = parseInt(headingLevels.charAt(headingLevels - 1));
+
+ switch (tocType) {
+ case TocTypeEnum.HORIZONTAL:
+ CreateHorizontalTOC(headings, masterLevel, lowestLevel, tbody_element);
+ break;
+ case TocTypeEnum.VERTICAL:
+ CreateVerticalTOC(headings, masterLevel, lowestLevel, tbody_element);
+ break;
+ default:
+ }
+}
+
+function CreateHorizontalTOC(
+ headings, masterLevel, lowestLevel, tbody_element) {
+
+ // Initialize the header counter
+ var h = 0;
+ var ignoreChildren = false;
+
+ while (h < headings.length) {
+ // Get current heading
+ var heading = headings[h];
+
+ // Get the current heading level
+ var level = parseInt(heading.tagName.charAt(1));
+
+ if (isNaN(level) || level < 1 || level > lowestLevel) continue;
+
+ // If level is a masterLevel, make it a TOC parent category
+ if ((level == masterLevel) && (!hasClass(heading, 'ignoreLink'))) {
+ toc_current_row = AddTOCMaster(tbody_element, heading);
+ ignoreChildren = false;
+ }
+
+ if ((level == masterLevel) && (hasClass(heading, 'ignoreLink'))) {
+ ignoreChildren = true;
+ }
+
+ if ((level != masterLevel) && (!ignoreChildren)) {
+ AddTOCElements(toc_current_row, heading);
+ }
+
+ // Advance the header counter
+ h++;
+ }
+}
+
+// Adds a master Table of Content heading
+function AddTOCMaster(tocTable, heading) {
+
+ // Add the table row scaffolding
+ var toc_tr = document.createElement('tr');
+ tocTable.appendChild(toc_tr);
+ toc_tr.setAttribute('valign', 'top');
+ var toc_tr_td = document.createElement('td');
+ toc_tr.appendChild(toc_tr_td);
+ var toc_category = document.createElement('div');
+ toc_tr_td.appendChild(toc_category);
+ toc_category.className = 'toc_category';
+
+ // Create the link to this header
+ var link = document.createElement('a');
+ link.href = '#' + heading.id; // Create the anchor link
+ link.textContent = heading.textContent; // Link text is same as heading
+ toc_category.appendChild(link);
+
+ // Add the container table cell for its children
+ var toc_td = document.createElement('td');
+ toc_tr.appendChild(toc_td);
+ var toc_td_div = document.createElement('div');
+ toc_td_div.className = 'toc_stylepoint';
+ toc_td.appendChild(toc_td_div);
+
+ return (toc_td_div);
+}
+
+// Adds Table of Contents element to a master heading as children
+function AddTOCElements(toc_div, heading) {
+
+ if (heading.offsetParent === null) {
+ // The element is currently hidden, so don't create a TOC entry
+ } else {
+ // Create the list item element
+ var toc_list_element = document.createElement('li');
+ toc_list_element.className = 'toc_entry';
+ toc_div.appendChild(toc_list_element);
+
+ // Create the link to this header
+ var link = document.createElement('a');
+ link.href = '#' + heading.id; // Create the anchor link
+ link.textContent = heading.textContent; // Link text is same as heading
+ toc_list_element.appendChild(link);
+ }
+}
+
+function CreateVerticalTOC(headings, masterLevel, lowestLevel, tbody_element) {
+
+ // Create the Column scaffolding
+ var toc_tr = document.createElement('tr');
+ tbody_element.appendChild(toc_tr);
+ var toc_tr_td = document.createElement('td');
+ toc_tr_td.className = 'two_columns';
+ toc_tr.appendChild(toc_tr_td);
+
+
+ // Initialize the header counter and the current row
+ var h = 0;
+ var toc_current_col = null;
+ var ignoreChildren = false;
+
+ while (h < headings.length) {
+ // Get current heading
+ var heading = headings[h];
+
+ // Get the current heading level
+ var level = parseInt(heading.tagName.charAt(1));
+
+ if (isNaN(level) || level < 1 || level > lowestLevel) continue;
+
+ // If level is a masterLevel, make it a TOC parent category
+ if ((level == masterLevel) && (!hasClass(heading, 'ignoreLink'))) {
+ if (heading.offsetParent === null) {
+ // The element is currently hidden, so don't create a TOC entry
+ } else {
+ var td_dl = document.createElement('dl');
+ toc_tr_td.appendChild(td_dl);
+ var td_dt = document.createElement('dt');
+ td_dl.appendChild(td_dt);
+ toc_current_col = td_dl;
+
+ // Create the link to this header
+ var link = document.createElement('a');
+ link.href = '#' + heading.id; // Create the anchor link
+ link.textContent = heading.textContent; // Link text is same as heading
+ td_dt.appendChild(link);
+ ignoreChildren = false;
+ }
+ }
+
+ // If level is a masterLevel but it's specified to ignore links, skip it
+ // and its children.
+ if ((level == masterLevel) && (hasClass(heading, 'ignoreLink'))) {
+ ignoreChildren = true;
+ }
+
+ if ((level != masterLevel) && (!ignoreChildren)) {
+ if (heading.offsetParent === null) {
+ // The element is currently hidden, so don't create a TOC entry
+ } else {
+ var td_dd = document.createElement('dd');
+ toc_current_col.appendChild(td_dd);
+ // Create the link to this header
+ var link = document.createElement('a');
+ link.href = '#' + heading.id; // Create the anchor link
+ link.textContent = heading.textContent; // Link text is same as heading
+ td_dd.appendChild(link);
+ }
+ }
+
+ // Advance the header counter
+ h++;
+ }
+}
+
+/*
+ * Utility function for finding elements with a given
+ * class.
+ */
+function hasClass(element, cls) {
+ return (' ' + element.className + ' ').indexOf(' ' + cls + ' ') > -1;
+}
+
+/*
+ * Linkify all h2 through h4 headers, except for those marked
+ * "ignoreLink"
+ */
+
+// Add the link image to the element.
+function LinkifyHeader(header, fileName, sizePixels) {
+ var link = document.createElement('a');
+ link.href = '#' + header.id;
+ link.alt = 'link to ' + header.id;
+ link.innerHTML =
+ '<img src="include/' + fileName + '"' +
+ ' width=' + sizePixels +
+ ' height=' + sizePixels +
+ ' style="float:left;position:relative;bottom:5px;">';
+ header.appendChild(link);
+}
+
+// Find all elements of the given tag and linkify if
+// they don't have 'ignoreLink' in their class.
+function LinkifyHeadersForTag(tagName) {
+ var headers = document.getElementsByTagName(tagName);
+ var header;
+ for (var j = 0; j != headers.length; j++) {
+ header = headers[j];
+ if (!hasClass(header, 'ignoreLink') && ('id' in header)) {
+ if (header.id != '') {
+ LinkifyHeader(header, 'link.png', 21);
+ header.style.left = '-46px';
+ header.style.position = 'relative';
+ }
+ }
+ }
+}
+
+// Linkify all h2, h3, and h4s. h1s are titles.
+function LinkifyHeaders() {
+ LinkifyHeadersForTag('h2');
+ LinkifyHeadersForTag('h3');
+ LinkifyHeadersForTag('h4');
+}
+
+/*
+ * Initialize the style guide by showing all internal
+ * elements and then linkifying the headers.
+ */
+
+function initStyleGuide() {
+ LinkifyHeaders();
+ CreateTOC('tocDiv');
+}
diff --git a/styleguide/javaguide.css b/styleguide/javaguide.css
new file mode 100644
index 0000000..c42ba83
--- /dev/null
+++ b/styleguide/javaguide.css
@@ -0,0 +1,515 @@
+table {
+ border-collapse: collapse;
+}
+
+td, th {
+ border: 1px solid #ccc;
+ padding: 2px 12px;
+ font-size: 10pt;
+}
+
+code, samp, var {
+ color: #060;
+}
+
+pre {
+ font-size: 10pt;
+ display: block;
+ color: #060;
+ background-color: #e8fff6;
+ border-color: #f0fff0;
+ border-style: solid;
+ border-top-width: 1px;
+ border-bottom-width: 1px;
+ border-right-width: 1px;
+ border-left-width: 5px;
+ padding-left: 12px;
+ padding-right: 12px;
+ padding-top: 4px;
+ padding-bottom: 4px;
+}
+
+pre.badcode {
+ color: #c00;
+ background-color: #ffe6d8;
+ border-color: #fff0f0;
+}
+
+hr {
+ margin-top: 3.5em;
+ border-width: 1px;
+ color: #fff;
+}
+
+html {
+ margin-top:2em;
+ margin-left:10%;
+ margin-right:10%;
+ padding:0;
+}
+
+.bp-reset-element,
+body,
+h1,
+h2,
+h3,
+h4,
+h5,
+h6,
+article,
+aside,
+details,
+figcaption,
+figure,
+footer,
+header,
+hgroup,
+menu,
+nav,
+section,
+summary,
+blockquote,
+q,
+th,
+td,
+caption,
+table,
+div,
+span,
+object,
+iframe,
+p,
+pre,
+a,
+abbr,
+acronym,
+address,
+code,
+del,
+dfn,
+em,
+img,
+dl,
+dt,
+dd,
+ol,
+ul,
+li,
+fieldset,
+form,
+label,
+legend,
+caption,
+tbody,
+tfoot,
+thead,
+tr {
+ margin:0;
+ padding:0;
+ border:0;
+ font-weight:inherit;
+ font-style:inherit;
+ font-size:100%;
+ font-family:inherit;
+ vertical-align:baseline;
+}
+
+body {
+ font-family:'Arial', sans-serif;
+ font-size:81.25%;
+ color:#222;
+ background-color:#fff;
+ line-height:1.67;
+ overflow: auto;
+}
+
+.change {
+ text-align: right;
+ margin-bottom:1em;
+}
+
+em {
+ font-style: italic
+}
+
+h1,
+h2,
+h3,
+h4,
+h5,
+h6 {
+ font-weight:bold;
+}
+
+h1 {
+ margin-bottom:.50em;
+ text-align: center
+}
+
+h2,
+h3,
+h4,
+h5,
+h6 {
+ margin-top:1.5em;
+ margin-bottom:.75em;
+}
+
+h1 {font-size:200%;}
+h2 {font-size:167%;}
+h3 {font-size:133%;}
+h4 {font-size:120%;}
+h5 {font-size:110%;}
+
+p {
+ margin:0 0 1.5em;
+}
+
+a[href=''] {
+ cursor:default;
+}
+
+h1 img,
+h2 img,
+h3 img,
+h4 img,
+h5 img,
+h6 img {
+ margin:0;
+}
+
+a img {
+ border:none;
+}
+
+pre {
+ margin:1.5em 0;
+ white-space:pre;
+}
+
+pre,
+code,
+kbd,
+tt {
+ font:1em 'Droid Sans Mono', monospace;
+ line-height:1.5;
+}
+
+dl {
+ margin:0 0 1.5em 0;
+}
+
+dl dt {
+ font-weight:bold;
+}
+
+dd {
+ margin-left:1.5em;
+}
+
+dd.toc3 {
+ margin-left:3em;
+}
+
+hr {
+ height:0;
+ border:0;
+ border-top:1px solid #ccc;
+ background-color:#ccc;
+}
+
+table {
+ border:1px solid #bbb;
+ border-spacing:0;
+ border-collapse:collapse;
+ margin:0 0 1.5em;
+ vertical-align:middle;
+ width:100%;
+}
+
+table.unlined,
+table.unlined th,
+table.unlined tr,
+table.unlined td {
+ border:0;
+}
+
+th,
+td,
+caption {
+ float:none !important;
+ text-align:left;
+ font-weight:normal;
+ vertical-align:middle;
+ padding:4px;
+}
+
+caption {
+ padding:0;
+}
+
+td {
+ border:1px solid #bbb;
+ vertical-align:top;
+}
+
+th {
+ border:0;
+ border-bottom:1px solid black;
+ font-weight:bold;
+ background:rgb(229, 236, 249);
+}
+
+table th code {
+ background-color:inherit;
+ color:inherit;
+}
+
+table tfoot th {
+ border:1px solid #bbb;
+}
+
+tfoot {
+ font-style:italic;
+}
+
+caption {
+ background:#eee;
+}
+
+table[border='0'] {
+ border:none;
+}
+
+table[border='0']>tbody>tr>td,
+table[border='0']>tr>td {
+ border:none;
+}
+
+tr.alt td,
+td.alt {
+ background-color:#efefef;
+}
+
+table.striped tr:nth-child(even) td,
+table tr.even td {
+ background:#efefef;
+}
+
+table.columns {
+ border:none;
+}
+
+table.columns>tbody>tr>td,
+table.columns>tr>td {
+ border:none;
+ padding:0 3em 0 0;
+}
+
+table.columns>tbody>tr>td:last-child,
+table.columns>tr>td:last-child {
+ border:none;
+ padding:0;
+}
+
+ul,
+ol {
+ margin:0 1.5em 1.5em 0;
+ padding-left:2em;
+}
+
+li ul,
+li ol {
+ margin:0;
+}
+
+ul {
+ list-style-type:disc;
+}
+
+ol {
+ list-style-type:decimal;
+}
+
+ul {
+ list-style-type:disc;
+}
+
+ul ul {
+ list-style-type:circle;
+}
+
+ul ul ul {
+ list-style-type:square;
+}
+
+ul.disc {
+ list-style-type:disc;
+}
+
+ul.circle {
+ list-style-type:circle;
+}
+
+ul.square {
+ list-style-type:square;
+}
+
+ol {
+ list-style-type:decimal;
+}
+
+ol ol {
+ list-style-type:lower-alpha;
+}
+
+ol ol ol {
+ list-style-type:lower-roman;
+}
+
+ol ul {
+ list-style-type:circle;
+}
+
+ol.decimal {
+ list-style-type:decimal;
+}
+
+ol.upper-alpha {
+ list-style-type:upper-alpha;
+}
+
+ol.lower-alpha {
+ list-style-type:lower-alpha;
+}
+
+ol.upper-roman {
+ list-style-type:upper-roman;
+}
+
+ol.lower-roman {
+ list-style-type:lower-roman;
+}
+
+ol.nolist,
+ul.nolist {
+ padding-left:0;
+ list-style-image:none;
+ list-style-type:none;
+ margin-left:0;
+}
+
+.center {
+ text-align:center;
+}
+
+code,
+kbd,
+pre {
+ color:#009900;
+}
+
+kbd {
+ font-weight: bold;
+}
+
+table.striped code {
+ background-color:inherit;
+}
+
+pre {
+ padding:6px 10px;
+ background-color:#FAFAFA;
+ border:1px solid #bbb;
+ overflow:auto;
+}
+
+pre.prettyprint {
+ padding:6px 10px !important;
+ border:1px solid #bbb !important;
+}
+
+code.bad, code.badcode {
+ color: magenta;
+}
+pre.bad, pre.badcode {
+ background-color:#ffe6d8;
+ border-top:1px inset #a03;
+ border-left:1px inset #a03;
+}
+
+.tip {
+ background-color:#fffbd9;
+ padding:6px 8px 6px 10px;
+ border-left:6px solid #ffef70;
+}
+
+.note {
+ background-color:#e5ecf9;
+ padding:6px 8px 6px 10px;
+ border-left:6px solid #36c;
+}
+
+@media print {
+
+ .str {
+ color:#060;
+ }
+
+ .kwd {
+ color:#006;
+ font-weight:bold;
+ }
+
+ .com {
+ color:#600;
+ font-style:italic;
+ }
+
+ .typ {
+ color:#404;
+ font-weight:bold;
+ }
+
+ .lit {
+ color:#044;
+ }
+
+ .pun,
+ .opn,
+ .clo {
+ color:#440;
+ }
+
+ .pln {
+ color:#000;
+ }
+
+ .tag {
+ color:#006;
+ font-weight:bold;
+ }
+
+ .atn {
+ color:#404;
+ }
+
+ .atv {
+ color:#060;
+ }
+
+ h1 {
+ font-style:italic;
+ }
+}
+
+ol.linenums {
+ margin-top:0;
+ margin-bottom:0;
+}
+
+code {
+ background-color:#FAFAFA;
+ padding: 0.25em 0.5em;
+ white-space: nowrap
+}
diff --git a/styleguide/javaguide.html b/styleguide/javaguide.html
new file mode 100644
index 0000000..da34b85
--- /dev/null
+++ b/styleguide/javaguide.html
@@ -0,0 +1,806 @@
+<html lang="en">
+<head>
+ <meta http-equiv="content-type" content="text/html; charset=UTF-8" />
+ <link rel="stylesheet" type="text/css" href="javaguide.css"/>
+ <script src="https://google-code-prettify.googlecode.com/svn/loader/run_prettify.js"
+ type="text/javascript"></script>
+ <link href="http://www.google.com/favicon.ico"
+ type="image/x-icon" rel="shortcut icon" />
+ <title>WPILib Java Style</title>
+</head>
+<body>
+<h1>WPILib Java Style (Based on the
+ <a href=http://google-styleguide.googlecode.com/svn/trunk/javaguide.html>Google Java Style Guide</a>)</h1>
+ <div class="change">Last changed: June 19, 2015</div>
+<table border="0">
+<tr>
+<td>
+<dl>
+<br>
+<dt class="toc1">
+<a href="#s1-introduction">1 Introduction</a>
+</dt>
+<dd>
+<a href="#s1.1-terminology">1.1 Terminology notes</a>
+</dd>
+<dd>
+<a href="#s1.2-guide-notes">1.2 Guide notes</a>
+</dd>
+<br>
+<dt class="toc1">
+<a href="#s2-source-file-basics">2 Source file basics</a>
+</dt>
+<dd>
+<a href="#s2.1-file-name">2.1 File name</a>
+</dd>
+<dd>
+<a href="#s2.2-file-encoding">2.2 File encoding: UTF-8</a>
+</dd>
+<dd>
+<a href="#s2.3-special-characters">2.3 Special characters</a>
+</dd>
+<dd class="toc3">
+<a href="#s2.3.1-whitespace-characters">2.3.1 Whitespace characters</a>
+</dd>
+<dd class="toc3">
+<a href="#s2.3.2-special-escape-sequences">2.3.2 Special escape sequences</a>
+</dd>
+<dd class="toc3">
+<a href="#s2.3.3-non-ascii-characters">2.3.3 Non-ASCII characters</a>
+</dd>
+<br>
+<dt class="toc1">
+<a href="#s3-source-file-structure">3 Source file structure</a>
+</dt>
+<dd>
+<a href="#s3.1-copyright-statement">3.1 License or copyright information, if present</a>
+</dd>
+<dd>
+<a href="#s3.2-package-statement">3.2 Package statement</a>
+</dd>
+<dd>
+<a href="#s3.3-import-statements">3.3 Import statements</a>
+</dd>
+<dd class="toc3">
+<a href="#s3.3.1-wildcard-imports">3.3.1 No wildcard imports</a>
+</dd>
+<dd class="toc3">
+<a href="#s3.3.2-import-line-wrapping">3.3.2 No line-wrapping</a>
+</dd>
+<dd class="toc3">
+<a href="#s3.3.3-import-ordering-and-spacing">3.3.3 Ordering and spacing</a>
+</dd>
+<dd>
+<a href="#s3.4-class-declaration">3.4 Class declaration</a>
+</dd>
+<dd class="toc3">
+<a href="#s3.4.1-one-top-level-class">3.4.1 Exactly one top-level class declaration</a>
+</dd>
+<dd class="toc3">
+<a href="#s3.4.2-class-member-ordering">3.4.2 Class member ordering</a>
+</dd>
+</dl>
+</td><td>
+<dl>
+<br>
+<dt class="toc1">
+<a href="#s4-formatting">4 Formatting</a>
+</dt>
+<dd>
+<a href="#s4.1-braces">4.1 Braces</a>
+</dd>
+<dd class="toc3">
+<a href="#s4.1.1-braces-always-used">4.1.1 Braces are used where optional</a>
+</dd>
+<dd class="toc3">
+<a href="#s4.1.2-blocks-k-r-style">4.1.2 Nonempty blocks: K & R style</a>
+</dd>
+<dd class="toc3">
+<a href="#s4.1.3-braces-empty-blocks">4.1.3 Empty blocks: may be concise</a>
+</dd>
+<dd>
+<a href="#s4.2-block-indentation">4.2 Block indentation: +2 spaces</a>
+</dd>
+<dd>
+<a href="#s4.3-one-statement-per-line">4.3 One statement per line</a>
+</dd>
+<dd>
+<a href="#s4.4-column-limit">4.4 Column limit: 80 or 100</a>
+</dd>
+<dd>
+<a href="#s4.5-line-wrapping">4.5 Line-wrapping</a>
+</dd>
+<dd class="toc3">
+<a href="#s4.5.1-line-wrapping-where-to-break">4.5.1 Where to break</a>
+</dd>
+<dd class="toc3">
+<a href="#s4.5.2-line-wrapping-indent">4.5.2 Indent continuation lines at least +4 spaces</a>
+</dd>
+<dd>
+<a href="#s4.6-whitespace">4.6 Whitespace</a>
+</dd>
+<dd class="toc3">
+<a href="#s4.6.1-vertical-whitespace">4.6.1 Vertical Whitespace</a>
+</dd>
+<dd class="toc3">
+<a href="#s4.6.2-horizontal-whitespace">4.6.2 Horizontal whitespace</a>
+</dd>
+<dd class="toc3">
+<a href="#s4.6.3-horizontal-alignment">4.6.3 Horizontal alignment: never required</a>
+</dd>
+<dd>
+<a href="#s4.7-grouping-parentheses">4.7 Grouping parentheses: recommended</a>
+</dd>
+<dd>
+<a href="#s4.8-specific-constructs">4.8 Specific constructs</a>
+</dd>
+<dd class="toc3">
+<a href="#s4.8.1-enum-classes">4.8.1 Enum classes</a>
+</dd>
+<dd class="toc3">
+<a href="#s4.8.2-variable-declarations">4.8.2 Variable declarations</a>
+</dd>
+<dd class="toc3">
+<a href="#s4.8.3-arrays">4.8.3 Arrays</a>
+</dd>
+<dd class="toc3">
+<a href="#s4.8.4-switch">4.8.4 Switch statements</a>
+</dd>
+<dd class="toc3">
+<a href="#s4.8.5-annotations">4.8.5 Annotations</a>
+</dd>
+<dd class="toc3">
+<a href="#s4.8.6-comments">4.8.6 Comments</a>
+</dd>
+<dd class="toc3">
+<a href="#s4.8.7-modifiers">4.8.7 Modifiers</a>
+</dd>
+<dd class="toc3">
+<a href="#s4.8.8-numeric-literals">4.8.8 Numeric Literals</a>
+</dd>
+</dl>
+</td><td>
+<dl>
+<br>
+<dt class="toc1">
+<a href="#s5-naming">5 Naming</a>
+</dt>
+<dd>
+<a href="#s5.1-identifier-names">5.1 Rules common to all identifiers</a>
+</dd>
+<dd>
+<a href="#s5.2-specific-identifier-names">5.2 Rules by identifier type</a>
+</dd>
+<dd class="toc3">
+<a href="#s5.2.1-package-names">5.2.1 Package names</a>
+</dd>
+<dd class="toc3">
+<a href="#s5.2.2-class-names">5.2.2 Class names</a>
+</dd>
+<dd class="toc3">
+<a href="#s5.2.3-method-names">5.2.3 Method names</a>
+</dd>
+<dd class="toc3">
+<a href="#s5.2.4-constant-names">5.2.4 Constant names</a>
+</dd>
+<dd class="toc3">
+<a href="#s5.2.5-non-constant-field-names">5.2.5 Non-constant field names</a>
+</dd>
+<dd class="toc3">
+<a href="#s5.2.6-parameter-names">5.2.6 Parameter names</a>
+</dd>
+<dd class="toc3">
+<a href="#s5.2.7-local-variable-names">5.2.7 Local variable names</a>
+</dd>
+<dd class="toc3">
+<a href="#s5.2.8-type-variable-names">5.2.8 Type variable names</a>
+</dd>
+<dd>
+<a href="#s5.3-camel-case">5.3 Camel case: defined</a>
+</dd>
+<br>
+<dt class="toc1">
+<a href="#s6-programming-practices">6 Programming Practices</a>
+</dt>
+<dd>
+<a href="#s6.1-override-annotation">6.1 @Override: always used</a>
+</dd>
+<dd>
+<a href="#s6.2-caught-exceptions">6.2 Caught exceptions: not ignored</a>
+</dd>
+<dd>
+<a href="#s6.3-static-members">6.3 Static members: qualified using class</a>
+</dd>
+<dd>
+<a href="#s6.4-finalizers">6.4 Finalizers: not used</a>
+</dd>
+<br>
+<dt class="toc1">
+<a href="#s7-javadoc">7 Javadoc</a>
+</dt>
+<dd>
+<a href="#s7.1-javadoc-formatting">7.1 Formatting</a>
+</dd>
+<dd class="toc3">
+<a href="#s7.1.1-javadoc-multi-line">7.1.1 General form</a>
+</dd>
+<dd class="toc3">
+<a href="#s7.1.2-javadoc-paragraphs">7.1.2 Paragraphs</a>
+</dd>
+<dd class="toc3">
+<a href="#s7.1.3-javadoc-at-clauses">7.1.3 At-clauses</a>
+</dd>
+<dd>
+<a href="#s7.2-summary-fragment">7.2 The summary fragment</a>
+</dd>
+<dd>
+<a href="#s7.3-javadoc-where-required">7.3 Where Javadoc is used</a>
+</dd>
+<dd class="toc3">
+<a href="#s7.3.1-javadoc-exception-self-explanatory">7.3.1 Exception: self-explanatory methods</a>
+</dd>
+<dd class="toc3">
+<a href="#s7.3.2-javadoc-exception-overrides">7.3.2 Exception: overrides</a>
+</dd>
+</dl>
+</td>
+</tr>
+</table>
+<div><a name="s1-introduction"/>
+ <h2>1 Introduction <a href="#s1-introduction"><img height="21" width="21" src="javaguidelink.png"/></a></h2>
+
+<p><strong>This guide is a work in progress.</strong>
+We are currently working on getting this guide updated to
+a point where it is useful for WPILib developers to use.</p>
+
+ <p>This document serves as the style guide for WPILib. It is <em>heavily</em>
+ based on the Google Java Style Guide and copies pretty much word-for-word
+ the formatting/style components of the guide while cutting a couple
+ of the programming practices. As this guide evolves, we will likely
+ introduce more suggested/mandated programming practices specific
+ to WPILib.</p>
+ <p>It is encouraged that anyone working on the Java WPILib also
+ read the corresponding C++ guide, as we generally try to develop
+ the C++ and Java components of the library in parallel and many
+ programming practices true in one language will be true in the other
+ (although this is not universally true).</p>
+</p><a name="s1.1-terminology"/>
+ <h3>1.1 Terminology notes <a href="#s1.1-terminology"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p>In this document, unless otherwise clarified:</p><ol><li>The term <em>class</em> is used inclusively to mean an "ordinary" class, enum class,
+ interface or annotation type (<code class="prettyprint lang-java">@interface</code>).</li><li>The term <em>comment</em> always refers to <em>implementation</em> comments. We do not
+ use the phrase "documentation comments", instead using the common term "Javadoc."</li></ol><p>Other "terminology notes" will appear occasionally throughout the document.</p><a name="s1.2-guide-notes"/>
+ <h3>1.2 Guide notes <a href="#s1.2-guide-notes"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p>Example code in this document is <strong>non-normative</strong>. That is, while the examples
+are in Google Style, they may not illustrate the <em>only</em> stylish way to represent the
+code. Optional formatting choices made in examples should not be enforced as rules.</p><a name="s2-source-file-basics"/>
+ <h2>2 Source file basics <a href="#s2-source-file-basics"><img height="21" width="21" src="javaguidelink.png"/></a></h2>
+ <a name="s2.1-file-name"/>
+ <h3>2.1 File name <a href="#s2.1-file-name"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p>The source file name consists of the case-sensitive name of the top-level class it contains,
+plus the <code>.java</code> extension.</p><a name="s2.2-file-encoding"/>
+ <h3>2.2 File encoding: UTF-8 <a href="#s2.2-file-encoding"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p>Source files are encoded in <strong>UTF-8</strong>.</p><a name="s2.3-special-characters"/>
+ <h3>2.3 Special characters <a href="#s2.3-special-characters"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <a name="s2.3.1-whitespace-characters"/>
+ <h4>2.3.1 Whitespace characters <a href="#s2.3.1-whitespace-characters"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Aside from the line terminator sequence, the <strong>ASCII horizontal space
+character</strong> (<strong>0x20</strong>) is the only whitespace character that appears
+anywhere in a source file. This implies that:</p><ol><li>All other whitespace characters in string and character literals are escaped.</li><li>Tab characters are <strong>not</strong> used for indentation.</li></ol><a name="s2.3.2-special-escape-sequences"/>
+ <h4>2.3.2 Special escape sequences <a href="#s2.3.2-special-escape-sequences"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>For any character that has a special escape sequence
+(<code class="prettyprint lang-java">\b</code>,
+<code class="prettyprint lang-java">\t</code>,
+<code class="prettyprint lang-java">\n</code>,
+<code class="prettyprint lang-java">\f</code>,
+<code class="prettyprint lang-java">\r</code>,
+<code class="prettyprint lang-java">\"</code>,
+<code class="prettyprint lang-java">\'</code> and
+<code class="prettyprint lang-java">\\</code>), that sequence
+is used rather than the corresponding octal
+(e.g. <code class="badcode">\012</code>) or Unicode
+(e.g. <code class="badcode">\u000a</code>) escape.</p><a name="s2.3.3-non-ascii-characters"/>
+ <h4>2.3.3 Non-ASCII characters <a href="#s2.3.3-non-ascii-characters"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>For the remaining non-ASCII characters, either the actual Unicode character
+(e.g. <code class="prettyprint lang-java">∞</code>) or the equivalent Unicode escape
+(e.g. <code class="prettyprint lang-java">\u221e</code>) is used, depending only on which
+makes the code <strong>easier to read and understand</strong>.</p><p class="tip"><strong>Tip:</strong> In the Unicode escape case, and occasionally even when actual
+Unicode characters are used, an explanatory comment can be very helpful.</p><p>Examples:</p><table><tr><th>Example</th><th>Discussion</th></tr><tr><td><code class="prettyprint lang-java">String unitAbbrev = "μs";</code></td><td>Best: perfectly clear even without a comment.</td></tr><tr><td><code class="prettyprint lang-java">String unitAbbrev = "\u03bcs"; // "μs"</code></td><td>Allowed, but there's no reason to do this.</td></tr><tr><td><code class="prettyprint lang-java">String unitAbbrev = "\u03bcs";
+ // Greek letter mu, "s"</code></td><td>Allowed, but awkward and prone to mistakes.</td></tr><tr><td><code class="badcode">String unitAbbrev = "\u03bcs";</code></td><td>Poor: the reader has no idea what this is.</td></tr><tr><td><code class="prettyprint lang-java">return '\ufeff' + content;
+ // byte order mark</code></td><td>Good: use escapes for non-printable characters, and comment if necessary.</td></tr></table><p class="tip"><strong>Tip:</strong> Never make your code less readable simply out of fear that
+some programs might not handle non-ASCII characters properly. If that should happen, those
+programs are <strong>broken</strong> and they must be <strong>fixed</strong>.</p><a name="filestructure"/><a name="s3-source-file-structure"/>
+ <h2>3 Source file structure <a href="#s3-source-file-structure"><img height="21" width="21" src="javaguidelink.png"/></a></h2>
+ <div><p>A source file consists of, <strong>in order</strong>:</p><ol><li>License or copyright information, if present</li><li>Package statement</li><li>Import statements</li><li>Exactly one top-level class</li></ol></div><p><strong>Exactly one blank line</strong> separates each section that is present.</p><a name="s3.1-copyright-statement"/>
+ <h3>3.1 License or copyright information, if present <a href="#s3.1-copyright-statement"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p>If license or copyright information belongs in a file, it belongs here.</p><a name="s3.2-package-statement"/>
+ <h3>3.2 Package statement <a href="#s3.2-package-statement"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p>The package statement is <strong>not line-wrapped</strong>. The column limit (Section 4.4,
+<a href="#s4.4-column-limit">Column limit: 80</a>) does not apply to package statements.</p><a name="imports"/><a name="s3.3-import-statements"/>
+ <h3>3.3 Import statements <a href="#s3.3-import-statements"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <a name="s3.3.1-wildcard-imports"/>
+ <h4>3.3.1 No wildcard imports <a href="#s3.3.1-wildcard-imports"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p><strong>Wildcard imports</strong>, static or otherwise, <strong>are not used</strong>.</p><a name="s3.3.2-import-line-wrapping"/>
+ <h4>3.3.2 No line-wrapping <a href="#s3.3.2-import-line-wrapping"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Import statements are <strong>not line-wrapped</strong>. The column limit (Section 4.4,
+<a href="#s4.4-column-limit">Column limit: 80</a>) does not apply to import
+statements.</p><a name="s3.3.3-import-ordering-and-spacing"/>
+ <h4>3.3.3 Ordering and spacing <a href="#s3.3.3-import-ordering-and-spacing"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Import statements are divided into the following groups, in this order, with each group
+separated by a single blank line:</p><ol><li>All static imports in a single group</li><li><code>com.google</code> imports
+ (only if this source file is in the <code>com.google</code> package
+ space)</li><li>Third-party imports, one group per top-level package, in ASCII sort order
+ <ul><li>for example: <code>android</code>, <code>com</code>, <code>junit</code>, <code>org</code>,
+ <code>sun</code></li></ul></li><li><code>java</code> imports</li><li><code>javax</code> imports</li></ol><p>Within a group there are no blank lines, and the imported names appear in ASCII sort
+order. (<strong>Note:</strong> this is not the same as the import <em>statements</em> being in
+ASCII sort order; the presence of semicolons warps the result.)</p><a name="s3.4-class-declaration"/>
+ <h3>3.4 Class declaration <a href="#s3.4-class-declaration"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <a name="oneclassperfile"/><a name="s3.4.1-one-top-level-class"/>
+ <h4>3.4.1 Exactly one top-level class declaration <a href="#s3.4.1-one-top-level-class"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Each top-level class resides in a source file of its own.</p><a name="s3.4.2-class-member-ordering"/>
+ <h4>3.4.2 Class member ordering <a href="#s3.4.2-class-member-ordering"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>The ordering of the members of a class can have a great effect on learnability, but there is
+no single correct recipe for how to do it. Different classes may order their members
+differently.</p><p>What is important is that each class order its members in <strong><em>some</em> logical
+order</strong>, which its maintainer could explain if asked. For example, new methods are not
+just habitually added to the end of the class, as that would yield "chronological by date
+added" ordering, which is not a logical ordering.</p><a name="overloads"/><a name="s3.4.2.1-overloads-never-split"/>
+ <h5>3.4.2.1 Overloads: never split <a href="#s3.4.2.1-overloads-never-split"><img height="21" width="21" src="javaguidelink.png"/></a></h5>
+ <p>When a class has multiple constructors, or multiple methods with the same name, these appear
+sequentially, with no intervening members.</p><a name="s4-formatting"/>
+ <h2>4 Formatting <a href="#s4-formatting"><img height="21" width="21" src="javaguidelink.png"/></a></h2>
+ <p class="terminology"><strong>Terminology Note:</strong> <em>block-like construct</em> refers to
+the body of a class, method or constructor. Note that, by Section 4.8.3.1 on
+<a href="#s4.8.3.1-array-initializers">array initializers</a>, any array initializer
+<em>may</em> optionally be treated as if it were a block-like construct.</p><a name="braces"/><a name="s4.1-braces"/>
+ <h3>4.1 Braces <a href="#s4.1-braces"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <a name="s4.1.1-braces-always-used"/>
+ <h4>4.1.1 Braces are used where optional <a href="#s4.1.1-braces-always-used"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Braces are used with
+<code class="prettyprint lang-java">if</code>,
+<code class="prettyprint lang-java">else</code>,
+<code class="prettyprint lang-java">for</code>,
+<code class="prettyprint lang-java">do</code> and
+<code class="prettyprint lang-java">while</code> statements, even when the
+body is empty or contains only a single statement.</p><a name="s4.1.2-blocks-k-r-style"/>
+ <h4>4.1.2 Nonempty blocks: K & R style <a href="#s4.1.2-blocks-k-r-style"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Braces follow the Kernighan and Ritchie style
+("<a href="http://www.codinghorror.com/blog/2012/07/new-programming-jargon.html">Egyptian brackets</a>")
+for <em>nonempty</em> blocks and block-like constructs:</p><ul><li>No line break before the opening brace.</li><li>Line break after the opening brace.</li><li>Line break before the closing brace.</li><li>Line break after the closing brace <em>if</em> that brace terminates a statement or the body
+ of a method, constructor or <em>named</em> class. For example, there is <em>no</em> line break
+ after the brace if it is followed by <code class="prettyprint lang-java">else</code> or a
+ comma.</li></ul><p>Example:</p><pre class="prettyprint lang-java">
+return new MyClass() {
+ @Override public void method() {
+ if (condition()) {
+ try {
+ something();
+ } catch (ProblemException e) {
+ recover();
+ }
+ }
+ }
+};
+</pre><p>A few exceptions for enum classes are given in Section 4.8.1,
+<a href="#s4.8.1-enum-classes">Enum classes</a>.</p><a name="emptyblocks"/><a name="s4.1.3-braces-empty-blocks"/>
+ <h4>4.1.3 Empty blocks: may be concise <a href="#s4.1.3-braces-empty-blocks"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>An empty block or block-like construct <em>may</em> be closed immediately after it is
+opened, with no characters or line break in between
+(<code class="prettyprint lang-java">{}</code>), <strong>unless</strong> it is part of a
+<em>multi-block statement</em> (one that directly contains multiple blocks:
+<code class="prettyprint lang-java">if/else-if/else</code> or
+<code class="prettyprint lang-java">try/catch/finally</code>).</p><p>Example:</p><pre class="prettyprint lang-java">
+ void doNothing() {}
+</pre><a name="s4.2-block-indentation"/>
+ <h3>4.2 Block indentation: +2 spaces <a href="#s4.2-block-indentation"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p>Each time a new block or block-like construct is opened, the indent increases by two
+spaces. When the block ends, the indent returns to the previous indent level. The indent level
+applies to both code and comments throughout the block. (See the example in Section 4.1.2,
+<a href="#s4.1.2-blocks-k-r-style">Nonempty blocks: K & R Style</a>.)</p><a name="s4.3-one-statement-per-line"/>
+ <h3>4.3 One statement per line <a href="#s4.3-one-statement-per-line"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p>Each statement is followed by a line-break.</p><a name="columnlimit"/><a name="s4.4-column-limit"/>
+ <h3>4.4 Column limit: 80 <a href="#s4.4-column-limit"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p>
+ Projects should have a column limit of 80 characters.
+
+Except as noted below, any line that would exceed this limit must be line-wrapped, as explained in
+Section 4.5, <a href="#s4.5-line-wrapping">Line-wrapping</a>.
+</p><p><strong>Exceptions:</strong></p><ol><li>Lines where obeying the column limit is not possible (for example, a long URL in Javadoc,
+ or a long JSNI method reference).</li><li><code class="prettyprint lang-java">package</code> and
+ <code class="prettyprint lang-java">import</code> statements (see Sections
+ 3.2 <a href="#s3.2-package-statement">Package statement</a> and
+ 3.3 <a href="#s3.3-import-statements">Import statements</a>).</li><li>Command lines in a comment that may be cut-and-pasted into a shell.</li></ol><a name="s4.5-line-wrapping"/>
+ <h3>4.5 Line-wrapping <a href="#s4.5-line-wrapping"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p class="terminology"><strong>Terminology Note:</strong> When code that might otherwise legally
+occupy a single line is divided into multiple lines, typically to avoid overflowing the column
+limit, this activity is called
+<em>line-wrapping</em>.</p><p>There is no comprehensive, deterministic formula showing <em>exactly</em> how to line-wrap in
+every situation. Very often there are several valid ways to line-wrap the same piece of code.</p><p class="tip"><strong>Tip:</strong> Extracting a method or local variable may solve the problem
+without the need to line-wrap.</p><a name="s4.5.1-line-wrapping-where-to-break"/>
+ <h4>4.5.1 Where to break <a href="#s4.5.1-line-wrapping-where-to-break"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>The prime directive of line-wrapping is: prefer to break at a
+<strong>higher syntactic level</strong>. Also:</p><ol><li>When a line is broken at a <em>non-assignment</em> operator the break comes <em>before</em>
+ the symbol. (Note that this is not the same practice used in Google style for other languages,
+ such as C++ and JavaScript.)
+ <ul><li>This also applies to the following "operator-like" symbols: the dot separator
+ (<code class="prettyprint lang-java">.</code>), the ampersand in type bounds
+ (<code class="prettyprint lang-java"><T extends Foo & Bar></code>), and the pipe in
+ catch blocks
+ (<code class="prettyprint lang-java">catch (FooException | BarException e)</code>).</li></ul></li><li>When a line is broken at an <em>assignment</em> operator the break typically comes
+ <em>after</em> the symbol, but either way is acceptable.
+ <ul><li>This also applies to the "assignment-operator-like" colon in an enhanced
+ <code class="prettyprint lang-java">for</code> ("foreach") statement.</li></ul></li><li>A method or constructor name stays attached to the open parenthesis
+ (<code class="prettyprint lang-java">(</code>) that follows it.</li><li>A comma (<code class="prettyprint lang-java">,</code>) stays attached to the token that
+ precedes it.</li></ol><a name="indentation"/><a name="s4.5.2-line-wrapping-indent"/>
+ <h4>4.5.2 Indent continuation lines at least +4 spaces <a href="#s4.5.2-line-wrapping-indent"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>When line-wrapping, each line after the first (each <em>continuation line</em>) is indented
+at least +4 from the original line.</p><p>When there are multiple continuation lines, indentation may be varied beyond +4 as
+desired. In general, two continuation lines use the same indentation level if and only if they
+begin with syntactically parallel elements.</p><p>Section 4.6.3 on <a href="#s4.6.3-horizontal-alignment">Horizontal alignment</a> addresses
+the discouraged practice of using a variable number of spaces to align certain tokens with
+previous lines.</p><a name="s4.6-whitespace"/>
+ <h3>4.6 Whitespace <a href="#s4.6-whitespace"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <a name="s4.6.1-vertical-whitespace"/>
+ <h4>4.6.1 Vertical Whitespace <a href="#s4.6.1-vertical-whitespace"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>A single blank line appears:</p><ol><li><em>Between</em> consecutive members (or initializers) of a class: fields, constructors,
+ methods, nested classes, static initializers, instance initializers.
+ <ul><li><span class="exception"><strong>Exception:</strong> A blank line between two consecutive
+ fields (having no other code between them) is optional. Such blank lines are used as needed to
+ create <em>logical groupings</em> of fields.</span></li></ul></li><li>Within method bodies, as needed to create <em>logical groupings</em> of statements.</li><li><em>Optionally</em> before the first member or after the last member of the class (neither
+ encouraged nor discouraged).</li><li>As required by other sections of this document (such as Section 3.3,
+ <a href="#s3.3-import-statements">Import statements</a>).</li></ol><p><em>Multiple</em> consecutive blank lines are permitted, but never required (or encouraged).</p><a name="s4.6.2-horizontal-whitespace"/>
+ <h4>4.6.2 Horizontal whitespace <a href="#s4.6.2-horizontal-whitespace"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Beyond where required by the language or other style rules, and apart from literals, comments and
+Javadoc, a single ASCII space also appears in the following places <strong>only</strong>.</p><ol><li>Separating any reserved word, such as
+ <code class="prettyprint lang-java">if</code>,
+ <code class="prettyprint lang-java">for</code> or
+ <code class="prettyprint lang-java">catch</code>, from an open parenthesis
+ (<code class="prettyprint lang-java">(</code>)
+ that follows it on that line</li><li>Separating any reserved word, such as
+ <code class="prettyprint lang-java">else</code> or
+ <code class="prettyprint lang-java">catch</code>, from a closing curly brace
+ (<code class="prettyprint lang-java">}</code>) that precedes it on that line</li><li>Before any open curly brace
+ (<code class="prettyprint lang-java">{</code>), with two exceptions:
+ <ul><li><code class="prettyprint lang-java">@SomeAnnotation({a, b})</code> (no space is used)</li><li><code class="prettyprint lang-java">String[][] x = {{"foo"}};</code> (no space is required
+ between <code class="prettyprint lang-java">{{</code>, by item 8 below)</li></ul></li><li>On both sides of any binary or ternary operator. This also applies to the following
+ "operator-like" symbols:
+ <ul><li>the ampersand in a conjunctive type bound:
+ <code class="prettyprint lang-java"><T extends Foo & Bar></code></li><li>the pipe for a catch block that handles multiple exceptions:
+ <code class="prettyprint lang-java">catch (FooException | BarException e)</code></li><li>the colon (<code class="prettyprint lang-java">:</code>) in an enhanced
+ <code class="prettyprint lang-java">for</code> ("foreach") statement</li></ul></li><li>After <code class="prettyprint lang-java">,:;</code> or the closing parenthesis
+ (<code class="prettyprint lang-java">)</code>) of a cast</li><li>On both sides of the double slash (<code class="prettyprint lang-java">//</code>) that
+ begins an end-of-line comment. Here, multiple spaces are allowed, but not required.</li><li>Between the type and variable of a declaration:
+ <code class="prettyprint lang-java">List<String> list</code></li><li><em>Optional</em> just inside both braces of an array initializer
+ <ul><li><code class="prettyprint lang-java">new int[] {5, 6}</code> and
+ <code class="prettyprint lang-java">new int[] { 5, 6 }</code> are both valid</li></ul></li></ol><p class="note"><strong>Note:</strong> This rule never requires or forbids additional space at the
+start or end of a line, only <em>interior</em> space.</p><a name="s4.6.3-horizontal-alignment"/>
+ <h4>4.6.3 Horizontal alignment: never required <a href="#s4.6.3-horizontal-alignment"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p class="terminology"><strong>Terminology Note:</strong> <em>Horizontal alignment</em> is the
+practice of adding a variable number of additional spaces in your code with the goal of making
+certain tokens appear directly below certain other tokens on previous lines.</p><p>This practice is permitted, but is <strong>never required</strong> by Google Style. It is not
+even required to <em>maintain</em> horizontal alignment in places where it was already used.</p><p>Here is an example without alignment, then using alignment:</p><pre class="prettyprint lang-java">
+private int x; // this is fine
+private Color color; // this too
+
+private int x; // permitted, but future edits
+private Color color; // may leave it unaligned
+</pre><p class="tip"><strong>Tip:</strong> Alignment can aid readability, but it creates problems for
+future maintenance. Consider a future change that needs to touch just one line. This change may
+leave the formerly-pleasing formatting mangled, and that is <strong>allowed</strong>. More often
+it prompts the coder (perhaps you) to adjust whitespace on nearby lines as well, possibly
+triggering a cascading series of reformattings. That one-line change now has a "blast radius."
+This can at worst result in pointless busywork, but at best it still corrupts version history
+information, slows down reviewers and exacerbates merge conflicts.</p><a name="parentheses"/><a name="s4.7-grouping-parentheses"/>
+ <h3>4.7 Grouping parentheses: recommended <a href="#s4.7-grouping-parentheses"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p>Optional grouping parentheses are omitted only when author and reviewer agree that there is no
+reasonable chance the code will be misinterpreted without them, nor would they have made the code
+easier to read. It is <em>not</em> reasonable to assume that every reader has the entire Java
+operator precedence table memorized.</p><a name="s4.8-specific-constructs"/>
+ <h3>4.8 Specific constructs <a href="#s4.8-specific-constructs"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <a name="s4.8.1-enum-classes"/>
+ <h4>4.8.1 Enum classes <a href="#s4.8.1-enum-classes"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>After each comma that follows an enum constant, a line-break is optional.</p><p>An enum class with no methods and no documentation on its constants may optionally be formatted
+as if it were an array initializer (see Section 4.8.3.1 on
+<a href="#s4.8.3.1-array-initializers">array initializers</a>).</p><pre class="prettyprint lang-java">
+private enum Suit { CLUBS, HEARTS, SPADES, DIAMONDS }
+</pre><p>Since enum classes <em>are classes</em>, all other rules for formatting classes apply.</p><a name="localvariables"/><a name="s4.8.2-variable-declarations"/>
+ <h4>4.8.2 Variable declarations <a href="#s4.8.2-variable-declarations"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <a name="s4.8.2.1-variables-per-declaration"/>
+ <h5>4.8.2.1 One variable per declaration <a href="#s4.8.2.1-variables-per-declaration"><img height="21" width="21" src="javaguidelink.png"/></a></h5>
+ <p>Every variable declaration (field or local) declares only one variable: declarations such as
+<code class="badcode">int a, b;</code> are not used.</p><a name="s4.8.2.2-variables-limited-scope"/>
+ <h5>4.8.2.2 Declared when needed, initialized as soon as
+possible <a href="#s4.8.2.2-variables-limited-scope"><img height="21" width="21" src="javaguidelink.png"/></a></h5>
+ <p>Local variables are <strong>not</strong> habitually declared at the start of their containing
+block or block-like construct. Instead, local variables are declared close to the point they are
+first used (within reason), to minimize their scope. Local variable declarations typically have
+initializers, or are initialized immediately after declaration.</p><a name="s4.8.3-arrays"/>
+ <h4>4.8.3 Arrays <a href="#s4.8.3-arrays"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <a name="s4.8.3.1-array-initializers"/>
+ <h5>4.8.3.1 Array initializers: can be "block-like" <a href="#s4.8.3.1-array-initializers"><img height="21" width="21" src="javaguidelink.png"/></a></h5>
+ <p>Any array initializer may <em>optionally</em> be formatted as if it were a "block-like
+construct." For example, the following are all valid (<strong>not</strong> an exhaustive
+list):</p><pre class="prettyprint lang-java">
+new int[] { new int[] {
+ 0, 1, 2, 3 0,
+} 1,
+ 2,
+new int[] { 3,
+ 0, 1, }
+ 2, 3
+} new int[]
+ {0, 1, 2, 3}
+</pre><a name="s4.8.3.2-array-declarations"/>
+ <h5>4.8.3.2 No C-style array declarations <a href="#s4.8.3.2-array-declarations"><img height="21" width="21" src="javaguidelink.png"/></a></h5>
+ <p>The square brackets form a part of the <em>type</em>, not the variable:
+<code class="prettyprint lang-java">String[] args</code>, not
+<code class="badcode">String args[]</code>.</p><a name="s4.8.4-switch"/>
+ <h4>4.8.4 Switch statements <a href="#s4.8.4-switch"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p class="terminology"><strong>Terminology Note:</strong> Inside the braces of a
+<em>switch block</em> are one or more <em>statement groups</em>. Each statement group consists of
+one or more <em>switch labels</em> (either <code class="prettyprint lang-java">case FOO:</code> or
+<code class="prettyprint lang-java">default:</code>), followed by one or more statements.</p><a name="s4.8.4.1-switch-indentation"/>
+ <h5>4.8.4.1 Indentation <a href="#s4.8.4.1-switch-indentation"><img height="21" width="21" src="javaguidelink.png"/></a></h5>
+ <p>As with any other block, the contents of a switch block are indented +2.</p><p>After a switch label, a newline appears, and the indentation level is increased +2, exactly as
+if a block were being opened. The following switch label returns to the previous indentation
+level, as if a block had been closed.</p><a name="fallthrough"/><a name="s4.8.4.2-switch-fall-through"/>
+ <h5>4.8.4.2 Fall-through: commented <a href="#s4.8.4.2-switch-fall-through"><img height="21" width="21" src="javaguidelink.png"/></a></h5>
+ <p>Within a switch block, each statement group either terminates abruptly (with a
+<code class="prettyprint lang-java">break</code>,
+<code class="prettyprint lang-java">continue</code>,
+<code class="prettyprint lang-java">return</code> or thrown exception), or is marked with a comment
+to indicate that execution will or <em>might</em> continue into the next statement group. Any
+comment that communicates the idea of fall-through is sufficient (typically
+<code class="prettyprint lang-java">// fall through</code>). This special comment is not required in
+the last statement group of the switch block. Example:</p><pre class="prettyprint lang-java">
+switch (input) {
+ case 1:
+ case 2:
+ prepareOneOrTwo();
+ // fall through
+ case 3:
+ handleOneTwoOrThree();
+ break;
+ default:
+ handleLargeNumber(input);
+}
+</pre><a name="s4.8.4.3-switch-default"/>
+ <h5>4.8.4.3 The default case is present <a href="#s4.8.4.3-switch-default"><img height="21" width="21" src="javaguidelink.png"/></a></h5>
+ <p>Each switch statement includes a <code class="prettyprint lang-java">default</code> statement
+group, even if it contains no code.</p><a name="annotations"/><a name="s4.8.5-annotations"/>
+ <h4>4.8.5 Annotations <a href="#s4.8.5-annotations"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Annotations applying to a class, method or constructor appear immediately after the
+documentation block, and each annotation is listed on a line of its own (that is, one annotation
+per line). These line breaks do not constitute line-wrapping (Section
+4.5, <a href="#s4.5-line-wrapping">Line-wrapping</a>), so the indentation level is not
+increased. Example:</p><pre class="prettyprint lang-java">
+@Override
+@Nullable
+public String getNameIfPresent() { ... }
+</pre><p class="exception"><strong>Exception:</strong> A <em>single</em> parameterless annotation
+<em>may</em> instead appear together with the first line of the signature, for example:</p><pre class="prettyprint lang-java">
+@Override public int hashCode() { ... }
+</pre><p>Annotations applying to a field also appear immediately after the documentation block, but in
+this case, <em>multiple</em> annotations (possibly parameterized) may be listed on the same line;
+for example:</p><pre class="prettyprint lang-java">
+@Partial @Mock DataLoader loader;
+</pre><p>There are no specific rules for formatting parameter and local variable annotations.</p><a name="comments"/><a name="s4.8.6-comments"/>
+ <h4>4.8.6 Comments <a href="#s4.8.6-comments"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <a name="s4.8.6.1-block-comment-style"/>
+ <h5>4.8.6.1 Block comment style <a href="#s4.8.6.1-block-comment-style"><img height="21" width="21" src="javaguidelink.png"/></a></h5>
+ <p>Block comments are indented at the same level as the surrounding code. They may be in
+<code class="prettyprint lang-java">/* ... */</code> style or
+<code class="prettyprint lang-java">// ...</code> style. For multi-line
+<code class="prettyprint lang-java">/* ... */</code> comments, subsequent lines must start with
+<code>*</code> aligned with the <code>*</code> on the previous line.</p><pre class="prettyprint lang-java">
+/*
+ * This is // And so /* Or you can
+ * okay. // is this. * even do this. */
+ */
+</pre><p>Comments are not enclosed in boxes drawn with asterisks or other characters.</p><p class="tip"><strong>Tip:</strong> When writing multi-line comments, use the
+<code class="prettyprint lang-java">/* ... */</code> style if you want automatic code formatters to
+re-wrap the lines when necessary (paragraph-style). Most formatters don't re-wrap lines in
+<code class="prettyprint lang-java">// ...</code> style comment blocks.</p><a name="modifiers"/><a name="s4.8.7-modifiers"/>
+ <h4>4.8.7 Modifiers <a href="#s4.8.7-modifiers"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Class and member modifiers, when present, appear in the order
+recommended by the Java Language Specification:
+</p><pre>
+public protected private abstract static final transient volatile synchronized native strictfp
+</pre><a name="s4.8.8-numeric-literals"/>
+ <h4>4.8.8 Numeric Literals <a href="#s4.8.8-numeric-literals"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p><code>long</code>-valued integer literals use an uppercase <code>L</code> suffix, never
+lowercase (to avoid confusion with the digit <code>1</code>). For example, <code>3000000000L</code>
+rather than <code class="badcode">3000000000l</code>.</p><a name="naming"/><a name="s5-naming"/>
+ <h2>5 Naming <a href="#s5-naming"><img height="21" width="21" src="javaguidelink.png"/></a></h2>
+ <a name="s5.1-identifier-names"/>
+ <h3>5.1 Rules common to all identifiers <a href="#s5.1-identifier-names"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p>Identifiers use only ASCII letters and digits, and in two cases noted below, underscores. Thus
+each valid identifier name is matched by the regular expression <code>\w+</code> .</p><p> In Google Style special prefixes or
+suffixes, like those seen in the examples <code class="badcode">name_</code>,
+<code class="badcode">mName</code>, <code class="badcode">s_name</code> and
+<code class="badcode">kName</code>, are <strong>not</strong> used.
+For WPILib, we do make one exception to this rule for non-constant field name.</p><a name="s5.2-specific-identifier-names"/>
+ <h3>5.2 Rules by identifier type <a href="#s5.2-specific-identifier-names"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <a name="s5.2.1-package-names"/>
+ <h4>5.2.1 Package names <a href="#s5.2.1-package-names"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Package names are all lowercase, with consecutive words simply concatenated together (no
+underscores). For example, <code>com.example.deepspace</code>, not
+<code class="badcode">com.example.deepSpace</code> or
+<code class="badcode">com.example.deep_space</code>.</p><a name="s5.2.2-class-names"/>
+ <h4>5.2.2 Class names <a href="#s5.2.2-class-names"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Class names are written in <a href="#s5.3-camel-case">UpperCamelCase</a>.</p><p>Class names are typically nouns or noun phrases. For example,
+<code class="prettyprint lang-java">Character</code> or
+<code class="prettyprint lang-java">ImmutableList</code>. Interface names may also be nouns or
+noun phrases (for example, <code class="prettyprint lang-java">List</code>), but may sometimes be
+adjectives or adjective phrases instead (for example,
+<code class="prettyprint lang-java">Readable</code>).</p><p>There are no specific rules or even well-established conventions for naming annotation types.</p><p><em>Test</em> classes are named starting with the name of the class they are testing, and ending
+with <code class="prettyprint lang-java">Test</code>. For example,
+<code class="prettyprint lang-java">HashTest</code> or
+<code class="prettyprint lang-java">HashIntegrationTest</code>.</p><a name="s5.2.3-method-names"/>
+ <h4>5.2.3 Method names <a href="#s5.2.3-method-names"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Method names are written in <a href="#s5.3-camel-case">lowerCamelCase</a>.</p><p>Method names are typically verbs or verb phrases. For example,
+<code class="prettyprint lang-java">sendMessage</code> or
+<code class="prettyprint lang-java">stop</code>.</p><p>Underscores may appear in JUnit <em>test</em> method names to separate logical components of the
+name. One typical pattern is <code>test<i><MethodUnderTest></i>_<i><state></i></code>,
+for example <code class="prettyprint lang-java">testPop_emptyStack</code>. There is no One Correct
+Way to name test methods.</p><a name="constants"/><a name="s5.2.4-constant-names"/>
+ <h4>5.2.4 Constant names <a href="#s5.2.4-constant-names"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Constant names use <code class="prettyprint lang-java">CONSTANT_CASE</code>: all uppercase
+letters, with words separated by underscores. But what <em>is</em> a constant, exactly?</p><p>Every constant is a static final field, but not all static final fields are constants. Before
+choosing constant case, consider whether the field really <em>feels like</em> a constant. For
+example, if any of that instance's observable state can change, it is almost certainly not a
+constant. Merely <em>intending</em> to never mutate the object is generally not
+enough. Examples:</p><pre class="prettyprint lang-java">
+// Constants
+static final int NUMBER = 5;
+static final ImmutableList<String> NAMES = ImmutableList.of("Ed", "Ann");
+static final Joiner COMMA_JOINER = Joiner.on(','); // because Joiner is immutable
+static final SomeMutableType[] EMPTY_ARRAY = {};
+enum SomeEnum { ENUM_CONSTANT }
+
+// Not constants
+static String nonFinal = "non-final";
+final String nonStatic = "non-static";
+static final Set<String> mutableCollection = new HashSet<String>();
+static final ImmutableSet<SomeMutableType> mutableElements = ImmutableSet.of(mutable);
+static final Logger logger = Logger.getLogger(MyClass.getName());
+static final String[] nonEmptyArray = {"these", "can", "change"};
+</pre><p>These names are typically nouns or noun phrases.</p><a name="s5.2.5-non-constant-field-names"/>
+ <h4>5.2.5 Non-constant field names <a href="#s5.2.5-non-constant-field-names"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Non-constant field names (static or otherwise) are written
+in <a href="#s5.3-camel-case">lowerCamelCase</a> with a preceding <code class="prettyprint lang-java">m_</code>.</p>
+<p>These names are typically nouns or noun phrases. For example,
+<code class="prettyprint lang-java">m_computedValues</code> or
+<code class="prettyprint lang-java">m_index</code>.</p><a name="s5.2.6-parameter-names"/>
+ <h4>5.2.6 Parameter names <a href="#s5.2.6-parameter-names"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Parameter names are written in <a href="#s5.3-camel-case">lowerCamelCase</a>.</p><p>One-character parameter names should be avoided.</p><a name="s5.2.7-local-variable-names"/>
+ <h4>5.2.7 Local variable names <a href="#s5.2.7-local-variable-names"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Local variable names are written in <a href="#s5.3-camel-case">lowerCamelCase</a>, and can be
+abbreviated more liberally than other types of names.</p><p>However, one-character names should be avoided, except for temporary and looping variables.</p><p>Even when final and immutable, local variables are not considered to be constants, and should not
+be styled as constants.</p><a name="s5.2.8-type-variable-names"/>
+ <h4>5.2.8 Type variable names <a href="#s5.2.8-type-variable-names"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Each type variable is named in one of two styles:</p><ul><li>A single capital letter, optionally followed by a single numeral (such as
+ <code class="prettyprint lang-java">E</code>, <code class="prettyprint lang-java">T</code>,
+ <code class="prettyprint lang-java">X</code>, <code class="prettyprint lang-java">T2</code>)
+ </li><li>A name in the form used for classes (see Section 5.2.2,
+ <a href="#s5.2.2-class-names">Class names</a>), followed by the capital letter
+ <code class="prettyprint lang-java">T</code> (examples:
+ <code class="prettyprint lang-java">RequestT</code>,
+ <code class="prettyprint lang-java">FooBarT</code>).</li></ul><a name="acronyms"/><a name="camelcase"/><a name="s5.3-camel-case"/>
+ <h3>5.3 Camel case: defined <a href="#s5.3-camel-case"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p>Sometimes there is more than one reasonable way to convert an English phrase into camel case,
+such as when acronyms or unusual constructs like "IPv6" or "iOS" are present. To improve
+predictability, Google Style specifies the following (nearly) deterministic scheme.</p><p>Beginning with the prose form of the name:</p><ol><li>Convert the phrase to plain ASCII and remove any apostrophes. For example, "Müller's
+ algorithm" might become "Muellers algorithm".</li><li>Divide this result into words, splitting on spaces and any remaining punctuation (typically
+ hyphens).
+
+ <ul><li><em>Recommended:</em> if any word already has a conventional camel-case appearance in common
+ usage, split this into its constituent parts (e.g., "AdWords" becomes "ad words"). Note
+ that a word such as "iOS" is not really in camel case <em>per se</em>; it defies <em>any</em>
+ convention, so this recommendation does not apply.</li></ul></li><li>Now lowercase <em>everything</em> (including acronyms), then uppercase only the first
+ character of:
+ <ul><li>... each word, to yield <em>upper camel case</em>, or</li><li>... each word except the first, to yield <em>lower camel case</em></li></ul></li><li>Finally, join all the words into a single identifier.</li></ol><p>Note that the casing of the original words is almost entirely disregarded. Examples:</p><table><tr><th>Prose form</th><th>Correct</th><th>Incorrect</th></tr><tr><td>"XML HTTP request"</td><td><code class="prettyprint lang-java">XmlHttpRequest</code></td><td><code class="badcode">XMLHTTPRequest</code></td></tr><tr><td>"new customer ID"</td><td><code class="prettyprint lang-java">newCustomerId</code></td><td><code class="badcode">newCustomerID</code></td></tr><tr><td>"inner stopwatch"</td><td><code class="prettyprint lang-java">innerStopwatch</code></td><td><code class="badcode">innerStopWatch</code></td></tr><tr><td>"supports IPv6 on iOS?"</td><td><code class="prettyprint lang-java">supportsIpv6OnIos</code></td><td><code class="badcode">supportsIPv6OnIOS</code></td></tr><tr><td>"YouTube importer"</td><td><code class="prettyprint lang-java">YouTubeImporter</code><br/><code class="prettyprint lang-java">YoutubeImporter</code>*</td><td/></tr></table><p>*Acceptable, but not recommended.</p><p class="note"><strong>Note:</strong> Some words are ambiguously hyphenated in the English
+language: for example "nonempty" and "non-empty" are both correct, so the method names
+<code class="prettyprint lang-java">checkNonempty</code> and
+<code class="prettyprint lang-java">checkNonEmpty</code> are likewise both correct.</p><a name="s6-programming-practices"/>
+ <h2>6 Programming Practices <a href="#s6-programming-practices"><img height="21" width="21" src="javaguidelink.png"/></a></h2>
+ <a name="s6.1-override-annotation"/>
+ <h3>6.1 @Override: always used <a href="#s6.1-override-annotation"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p>A method is marked with the <code class="prettyprint lang-java">@Override</code> annotation
+whenever it is legal. This includes a class method overriding a superclass method, a class method
+implementing an interface method, and an interface method respecifying a superinterface
+method.</p><p class="exception"><strong>Exception:</strong><code class="prettyprint lang-java">@Override</code> may be omitted when the parent method is
+<code class="prettyprint lang-java">@Deprecated</code>.</p><a name="caughtexceptions"/><a name="s6.2-caught-exceptions"/>
+ <h3>6.2 Caught exceptions: not ignored <a href="#s6.2-caught-exceptions"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p>Except as noted below, it is very rarely correct to do nothing in response to a caught
+exception. (Typical responses are to log it, or if it is considered "impossible", rethrow it as an
+<code class="prettyprint lang-java">AssertionError</code>.)</p><p>When it truly is appropriate to take no action whatsoever in a catch block, the reason this is
+justified is explained in a comment.</p><pre class="prettyprint lang-java">
+try {
+ int i = Integer.parseInt(response);
+ return handleNumericResponse(i);
+} catch (NumberFormatException ok) {
+ // it's not numeric; that's fine, just continue
+}
+return handleTextResponse(response);
+</pre><p class="exception"><strong>Exception:</strong> In tests, a caught exception may be ignored
+without comment <em>if</em> it is named <code class="prettyprint lang-java">expected</code>. The
+following is a very common idiom for ensuring that the method under test <em>does</em> throw an
+exception of the expected type, so a comment is unnecessary here.</p><pre class="prettyprint lang-java">
+try {
+ emptyStack.pop();
+ fail();
+} catch (NoSuchElementException expected) {
+}
+</pre><a name="s6.3-static-members"/>
+ <h3>6.3 Static members: qualified using class <a href="#s6.3-static-members"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p>When a reference to a static class member must be qualified, it is qualified with that class's
+name, not with a reference or expression of that class's type.</p><pre class="prettyprint lang-java">
+Foo aFoo = ...;
+Foo.aStaticMethod(); // good
+<span class="badcode">aFoo.aStaticMethod();</span> // bad
+<span class="badcode">somethingThatYieldsAFoo().aStaticMethod();</span> // very bad
+</pre><a name="finalizers"/><a name="s6.4-finalizers"/>
+ <h3>6.4 Finalizers: not used <a href="#s6.4-finalizers"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p>It is <strong>extremely rare</strong> to override <code class="prettyprint lang-java">Object.finalize</code>.</p><p class="tip"><strong>Tip:</strong> Don't do it. If you absolutely must, first read and understand
+<a href="http://books.google.com/books?isbn=8131726592"><em>Effective Java</em></a>
+Item 7, "Avoid Finalizers," very carefully, and <em>then</em> don't do it.</p><a name="javadoc"/><a name="s7-javadoc"/>
+ <h2>7 Javadoc <a href="#s7-javadoc"><img height="21" width="21" src="javaguidelink.png"/></a></h2>
+ <a name="s7.1-javadoc-formatting"/>
+ <h3>7.1 Formatting <a href="#s7.1-javadoc-formatting"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <a name="s7.1.1-javadoc-multi-line"/>
+ <h4>7.1.1 General form <a href="#s7.1.1-javadoc-multi-line"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>The <em>basic</em> formatting of Javadoc blocks is as seen in this example:</p><pre class="prettyprint lang-java">
+/**
+ * Multiple lines of Javadoc text are written here,
+ * wrapped normally...
+ */
+public int method(String p1) { ... }
+</pre><p>... or in this single-line example:</p><pre class="prettyprint lang-java">
+/** An especially short bit of Javadoc. */
+</pre><p>The basic form is always acceptable. The single-line form may be substituted when there are no
+at-clauses present, and the entirety of the Javadoc block (including comment markers) can fit on a
+single line.</p><a name="s7.1.2-javadoc-paragraphs"/>
+ <h4>7.1.2 Paragraphs <a href="#s7.1.2-javadoc-paragraphs"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>One blank line—that is, a line containing only the aligned leading asterisk
+(<code>*</code>)—appears between paragraphs, and before the group of "at-clauses" if
+present. Each paragraph but the first has <code><p></code> immediately before the first word,
+with no space after.</p><a name="s7.1.3-javadoc-at-clauses"/>
+ <h4>7.1.3 At-clauses <a href="#s7.1.3-javadoc-at-clauses"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Any of the standard "at-clauses" that are used appear in the order <code>@param</code>,
+<code>@return</code>, <code>@throws</code>, <code>@deprecated</code>, and these four types never
+appear with an empty description. When an at-clause doesn't fit on a single line, continuation lines
+are indented four (or more) spaces from the position of the <code>@</code>.
+</p><a name="s7.2-summary-fragment"/>
+ <h3>7.2 The summary fragment <a href="#s7.2-summary-fragment"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p>The Javadoc for each class and member begins with a brief <strong>summary fragment</strong>. This
+fragment is very important: it is the only part of the text that appears in certain contexts such as
+class and method indexes.</p><p>This is a fragment—a noun phrase or verb phrase, not a complete sentence. It does
+<strong>not</strong> begin with <code class="badcode">A {@code Foo} is a...</code>, or
+<code class="badcode">This method returns...</code>, nor does it form a complete imperative sentence
+like <code class="badcode">Save the record.</code>. However, the fragment is capitalized and
+punctuated as if it were a complete sentence.</p><p class="tip"><strong>Tip:</strong> A common mistake is to write simple Javadoc in the form
+<code class="badcode">/** @return the customer ID */</code>. This is incorrect, and should be
+changed to <code class="prettyprint lang-java">/** Returns the customer ID. */</code>.</p><a name="s7.3.3-javadoc-optional"/><a name="s7.3-javadoc-where-required"/>
+ <h3>7.3 Where Javadoc is used <a href="#s7.3-javadoc-where-required"><img height="21" width="21" src="javaguidelink.png"/></a></h3>
+ <p>At the <em>minimum</em>, Javadoc is present for every
+<code class="prettyprint lang-java">public</code> class, and every
+<code class="prettyprint lang-java">public</code> or
+<code class="prettyprint lang-java">protected</code> member of such a class, with a few exceptions
+noted below.</p><p>Other classes and members still have Javadoc <em>as needed</em>. Whenever an implementation
+comment would be used to define the overall purpose or behavior of a class, method or field, that
+comment is written as Javadoc instead. (It's more uniform, and more tool-friendly.)</p><a name="s7.3.1-javadoc-exception-self-explanatory"/>
+ <h4>7.3.1 Exception: self-explanatory methods <a href="#s7.3.1-javadoc-exception-self-explanatory"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Javadoc is optional for "simple, obvious" methods like
+<code class="prettyprint lang-java">getFoo</code>, in cases where there <em>really and truly</em> is
+nothing else worthwhile to say but "Returns the foo".</p><p class="note"><strong>Important:</strong> it is not appropriate to cite this exception to justify
+omitting relevant information that a typical reader might need to know. For example, for a method
+named <code class="prettyprint lang-java">getCanonicalName</code>, don't omit its documentation
+(with the rationale that it would say only
+<code class="badcode">/** Returns the canonical name. */</code>) if a typical reader may have no idea
+what the term "canonical name" means!</p><a name="s7.3.2-javadoc-exception-overrides"/>
+ <h4>7.3.2 Exception: overrides <a href="#s7.3.2-javadoc-exception-overrides"><img height="21" width="21" src="javaguidelink.png"/></a></h4>
+ <p>Javadoc is not always present on a method that overrides a supertype method.
+</p></div> <hr/>
+ <div class="change">Last changed: March 21, 2014</div>
+</body>
+</html>
diff --git a/styleguide/javaguidelink.png b/styleguide/javaguidelink.png
new file mode 100644
index 0000000..75d5c7b
--- /dev/null
+++ b/styleguide/javaguidelink.png
Binary files differ
diff --git a/styleguide/styleguide.css b/styleguide/styleguide.css
new file mode 100644
index 0000000..adba8f3
--- /dev/null
+++ b/styleguide/styleguide.css
@@ -0,0 +1,147 @@
+body {
+ background-color: #fff;
+ color: #333;
+ font-family: sans-serif;
+ font-size: 10pt;
+ margin-right: 100px;
+ margin-left: 100px;
+}
+
+h1, h2, h3, h4, h5, h6, .toc_title {
+ color: #06c;
+ margin-top: 2em;
+ margin-bottom: 1em;
+}
+
+h1 {
+ text-align: center;
+ font-size: 18pt;
+}
+
+h2, .toc_title {
+ font-weight: bold;
+ font-size: 12pt;
+ margin-left: -40px;
+}
+
+h3, h4, h5, h6 {
+ font-size: 10pt;
+ margin-left: -20px;
+}
+
+.toc_category, .toc_stylepoint {
+ font-size: 10pt;
+ padding-top: .3em;
+ padding-bottom: .3em;
+}
+
+table {
+ border-collapse: collapse;
+}
+
+td, th {
+ border: 1px solid #ccc;
+ padding: 2px 12px;
+ font-size: 10pt;
+}
+
+.toc td, .toc th {
+ border-width: 1px 5px;
+}
+
+code, samp, var {
+ color: #060;
+}
+
+pre {
+ font-size: 10pt;
+ display: block;
+ color: #060;
+ background-color: #f8fff8;
+ border-color: #f0fff0;
+ border-style: solid;
+ border-top-width: 1px;
+ border-bottom-width: 1px;
+ border-right-width: 1px;
+ border-left-width: 5px;
+ padding-left: 12px;
+ padding-right: 12px;
+ padding-top: 4px;
+ padding-bottom: 4px;
+}
+
+pre.badcode {
+ color: #c00;
+ background-color: #fff8f8;
+ border-color: #fff0f0;
+}
+
+.showhide_button {
+ float: left;
+ cursor: pointer;
+ border-width: 1px;
+ border-style: solid;
+ border-color: #ddd #aaa #aaa #ddd;
+ padding: 0 3px 1px;
+ margin: 0 4px 8px 0;
+ border-radius: 3px;
+ -webkit-border-radius: 3px;
+ -moz-border-radius: 3px;
+}
+
+.link_button {
+ float: left;
+ display: none;
+ background-color: #f8f8ff;
+ border-color: #f0f0ff;
+ border-style: solid;
+ border-width: 1px;
+ font-size: 75%;
+ margin-top: 0;
+ margin-left: -50px;
+ padding: 4px;
+ border-radius: 3px;
+ -webkit-border-radius: 3px;
+ -moz-border-radius: 3px;
+}
+
+address {
+ text-align: right;
+}
+
+hr {
+ margin-top: 3.5em;
+ border-width: 1px;
+ color: #fff;
+}
+
+.stylepoint_section {
+ display: block;
+ margin-bottom: 1em;
+ color: #5588ff;
+ font-family: sans-serif;
+ font-size: 90%;
+ font-weight: bold;
+ margin-left: -2%;
+}
+
+.stylepoint_subsection {
+ color: #667799;
+ font-family: sans-serif;
+ font-size: 90%;
+ font-weight: bold;
+ margin-left: -1%;
+}
+
+.stylepoint_subsubsection {
+ color: #667799;
+ font-family: sans-serif;
+ font-size: 80%;
+ font-weight: bold;
+ margin-left: 0;
+}
+
+.revision {
+ text-align: right;
+}
+
diff --git a/styleguide/styleguide.xsl b/styleguide/styleguide.xsl
new file mode 100644
index 0000000..09a9b95
--- /dev/null
+++ b/styleguide/styleguide.xsl
@@ -0,0 +1,924 @@
+<xsl:stylesheet version="1.0"
+xmlns:xsl="http://www.w3.org/1999/XSL/Transform"
+xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#"
+xmlns:dc="http://purl.org/dc/elements/1.1/"
+xmlns:dcq="http://purl.org/dc/qualifiers/1.0/"
+xmlns:fo="http://www.w3.org/1999/XSL/Format"
+xmlns:fn="http://www.w3.org/2005/xpath-functions">
+ <xsl:output method="html"/>
+ <!-- Set to 1 to show explanations by default. Set to 0 to hide them -->
+ <xsl:variable name="show_explanation_default" select="0" />
+ <!-- The characters within the Webdings font that show the triangles -->
+ <xsl:variable name="show_button_text" select="'▶'" />
+ <xsl:variable name="hide_button_text" select="'▽'" />
+ <!-- The suffix for names -->
+ <xsl:variable name="button_suffix" select="'__button'"/>
+ <xsl:variable name="body_suffix" select="'__body'"/>
+ <!-- For easy reference, the name of the button -->
+ <xsl:variable name="show_hide_all_button" select="'show_hide_all_button'"/>
+
+ <!-- The top-level element -->
+ <xsl:template match="GUIDE">
+ <HTML>
+ <HEAD>
+ <TITLE><xsl:value-of select="@title"/></TITLE>
+ <META http-equiv="Content-Type" content="text/html; charset=utf-8"/>
+ <LINK HREF="http://www.google.com/favicon.ico" type="image/x-icon"
+ rel="shortcut icon"/>
+ <LINK HREF="styleguide.css"
+ type="text/css" rel="stylesheet"/>
+
+ <SCRIPT language="javascript" type="text/javascript">
+
+ function GetElementsByName(name) {
+ // Workaround a bug on old versions of opera.
+ if (document.getElementsByName) {
+ return document.getElementsByName(name);
+ } else {
+ return [document.getElementById(name)];
+ }
+ }
+
+ /**
+ * @param {string} namePrefix The prefix of the body name.
+ * @param {function(boolean): boolean} getVisibility Computes the new
+ * visibility state, given the current one.
+ */
+ function ChangeVisibility(namePrefix, getVisibility) {
+ var bodyName = namePrefix + '<xsl:value-of select="$body_suffix"/>';
+ var buttonName = namePrefix + '<xsl:value-of select="$button_suffix"/>';
+ var bodyElements = GetElementsByName(bodyName);
+ var linkElement = GetElementsByName('link-' + buttonName)[0];
+ if (bodyElements.length != 1) {
+ throw Error('ShowHideByName() got the wrong number of bodyElements: ' +
+ bodyElements.length);
+ } else {
+ var bodyElement = bodyElements[0];
+ var buttonElement = GetElementsByName(buttonName)[0];
+ var isVisible = bodyElement.style.display != "none";
+ if (getVisibility(isVisible)) {
+ bodyElement.style.display = "inline";
+ linkElement.style.display = "block";
+ buttonElement.innerHTML = '<xsl:value-of select="$hide_button_text"/>';
+ } else {
+ bodyElement.style.display = "none";
+ linkElement.style.display = "none";
+ buttonElement.innerHTML = '<xsl:value-of select="$show_button_text"/>';
+ }
+ }
+ }
+
+ function ShowHideByName(namePrefix) {
+ ChangeVisibility(namePrefix, function(old) { return !old; });
+ }
+
+ function ShowByName(namePrefix) {
+ ChangeVisibility(namePrefix, function() { return true; });
+ }
+
+ function ShowHideAll() {
+ var allButton = GetElementsByName("show_hide_all_button")[0];
+ if (allButton.innerHTML == '<xsl:value-of select="$hide_button_text"/>') {
+ allButton.innerHTML = '<xsl:value-of select="$show_button_text"/>';
+ SetHiddenState(document.getElementsByTagName("body")[0].childNodes, "none", '<xsl:value-of select="$show_button_text"/>');
+ } else {
+ allButton.innerHTML = '<xsl:value-of select="$hide_button_text"/>';
+ SetHiddenState(document.getElementsByTagName("body")[0].childNodes, "inline", '<xsl:value-of select="$hide_button_text"/>');
+ }
+ }
+
+ // Recursively sets state of all children
+ // of a particular node.
+ function SetHiddenState(root, newState, newButton) {
+ for (var i = 0; i != root.length; i++) {
+ SetHiddenState(root[i].childNodes, newState, newButton);
+ if (root[i].className == 'showhide_button') {
+ root[i].innerHTML = newButton;
+ }
+ if (root[i].className == 'stylepoint_body' ||
+ root[i].className == 'link_button') {
+ root[i].style.display = newState;
+ }
+ }
+ }
+
+
+ function EndsWith(str, suffix) {
+ var l = str.length - suffix.length;
+ return l >= 0 && str.indexOf(suffix, l) == l;
+ }
+
+ function RefreshVisibilityFromHashParam() {
+ var hashRegexp = new RegExp('#([^&#]*)$');
+ var hashMatch = hashRegexp.exec(window.location.href);
+ var anchor = hashMatch && GetElementsByName(hashMatch[1])[0];
+ var node = anchor;
+ var suffix = '<xsl:value-of select="$body_suffix"/>';
+ while (node) {
+ var id = node.id;
+ var matched = id && EndsWith(id, suffix);
+ if (matched) {
+ var len = id.length - suffix.length;
+ ShowByName(id.substring(0, len));
+ if (anchor.scrollIntoView) {
+ anchor.scrollIntoView();
+ }
+
+ return;
+ }
+ node = node.parentNode;
+ }
+ }
+
+ window.onhashchange = RefreshVisibilityFromHashParam;
+
+ window.onload = function() {
+ // if the URL contains "?showall=y", expand the details of all children
+ var showHideAllRegex = new RegExp("[\\?&](showall)=([^&#]*)");
+ var showHideAllValue = showHideAllRegex.exec(window.location.href);
+ if (showHideAllValue != null) {
+ if (showHideAllValue[2] == "y") {
+ SetHiddenState(document.getElementsByTagName("body")[0].childNodes,
+ "inline", '<xsl:value-of select="$hide_button_text"/>');
+ } else {
+ SetHiddenState(document.getElementsByTagName("body")[0].childNodes,
+ "none", '<xsl:value-of select="$show_button_text"/>');
+ }
+ }
+ var showOneRegex = new RegExp("[\\?&](showone)=([^&#]*)");
+ var showOneValue = showOneRegex.exec(window.location.href);
+ if (showOneValue) {
+ ShowHideByName(showOneValue[2]);
+ }
+
+
+ RefreshVisibilityFromHashParam();
+ }
+ </SCRIPT>
+ </HEAD>
+ <BODY>
+ <H1><xsl:value-of select="@title"/></H1>
+ <xsl:apply-templates/>
+ </BODY>
+ </HTML>
+ </xsl:template>
+
+ <xsl:template match="OVERVIEW">
+ <xsl:variable name="button_text">
+ <xsl:choose>
+ <xsl:when test="$show_explanation_default">
+ <xsl:value-of select="$hide_button_text"/>
+ </xsl:when>
+ <xsl:otherwise>
+ <xsl:value-of select="$show_button_text"/>
+ </xsl:otherwise>
+ </xsl:choose>
+ </xsl:variable>
+ <DIV style="margin-left: 50%; font-size: 75%;">
+ <P>
+ Each style point has a summary for which additional information is available
+ by toggling the accompanying arrow button that looks this way:
+ <SPAN class="showhide_button" style="margin-left: 0; float: none">
+ <xsl:value-of select="$show_button_text"/></SPAN>.
+ You may toggle all summaries with the big arrow button:
+ </P>
+ <DIV style=" font-size: larger; margin-left: +2em;">
+ <SPAN class="showhide_button" style="font-size: 180%; float: none">
+ <xsl:attribute name="onclick"><xsl:value-of select="'javascript:ShowHideAll()'"/></xsl:attribute>
+ <xsl:attribute name="name"><xsl:value-of select="$show_hide_all_button"/></xsl:attribute>
+ <xsl:attribute name="id"><xsl:value-of select="$show_hide_all_button"/></xsl:attribute>
+ <xsl:value-of select="$button_text"/>
+ </SPAN>
+ Toggle all summaries
+ </DIV>
+ </DIV>
+ <xsl:call-template name="TOC">
+ <xsl:with-param name="root" select=".."/>
+ </xsl:call-template>
+ <xsl:apply-templates/>
+ </xsl:template>
+
+ <xsl:template match="PARTING_WORDS">
+ <H2>Parting Words</H2>
+ <xsl:apply-templates/>
+ </xsl:template>
+
+ <xsl:template match="CATEGORY">
+ <DIV>
+ <xsl:attribute name="class"><xsl:value-of select="@class"/></xsl:attribute>
+ <H2>
+ <xsl:variable name="category_name">
+ <xsl:call-template name="anchorname">
+ <xsl:with-param name="sectionname" select="@title"/>
+ </xsl:call-template>
+ </xsl:variable>
+ <xsl:attribute name="name"><xsl:value-of select="$category_name"/></xsl:attribute>
+ <xsl:attribute name="id"><xsl:value-of select="$category_name"/></xsl:attribute>
+ <xsl:value-of select="@title"/>
+ </H2>
+ <xsl:apply-templates/>
+ </DIV>
+ </xsl:template>
+
+ <xsl:template match="STYLEPOINT">
+ <DIV>
+ <xsl:attribute name="class"><xsl:value-of select="@class"/></xsl:attribute>
+ <xsl:variable name="stylepoint_name">
+ <xsl:call-template name="anchorname">
+ <xsl:with-param name="sectionname" select="@title"/>
+ </xsl:call-template>
+ </xsl:variable>
+ <xsl:variable name="button_text">
+ <xsl:choose>
+ <xsl:when test="$show_explanation_default">
+ <xsl:value-of select="$hide_button_text"/>
+ </xsl:when>
+ <xsl:otherwise>
+ <xsl:value-of select="$show_button_text"/>
+ </xsl:otherwise>
+ </xsl:choose>
+ </xsl:variable>
+ <H3>
+ <A>
+ <xsl:attribute name="name"><xsl:value-of select="$stylepoint_name"/></xsl:attribute>
+ <xsl:attribute name="id"><xsl:value-of select="$stylepoint_name"/></xsl:attribute>
+ <xsl:value-of select="@title"/>
+ </A>
+ </H3>
+ <xsl:variable name="buttonName">
+ <xsl:value-of select="$stylepoint_name"/><xsl:value-of select="$button_suffix"/>
+ </xsl:variable>
+ <xsl:variable name="onclick_definition">
+ <xsl:text>javascript:ShowHideByName('</xsl:text>
+ <xsl:value-of select="$stylepoint_name"/>
+ <xsl:text>')</xsl:text>
+ </xsl:variable>
+ <SPAN class="link_button" id="link-{$buttonName}" name="link-{$buttonName}">
+ <A>
+ <xsl:attribute name="href">?showone=<xsl:value-of select="$stylepoint_name"/>#<xsl:value-of select="$stylepoint_name"/></xsl:attribute>
+ link
+ </A>
+ </SPAN>
+ <SPAN class="showhide_button">
+ <xsl:attribute name="onclick"><xsl:value-of select="$onclick_definition"/></xsl:attribute>
+ <xsl:attribute name="name"><xsl:value-of select="$buttonName"/></xsl:attribute>
+ <xsl:attribute name="id"><xsl:value-of select="$buttonName"/></xsl:attribute>
+ <xsl:value-of select="$button_text"/>
+ </SPAN>
+ <xsl:apply-templates>
+ <xsl:with-param name="anchor_prefix" select="$stylepoint_name" />
+ </xsl:apply-templates>
+ </DIV>
+ </xsl:template>
+
+ <xsl:template match="SUMMARY">
+ <xsl:param name="anchor_prefix" />
+ <DIV style="display:inline;">
+ <xsl:attribute name="class"><xsl:value-of select="@class"/></xsl:attribute>
+ <xsl:apply-templates/>
+ </DIV>
+ </xsl:template>
+
+ <xsl:template match="BODY">
+ <xsl:param name="anchor_prefix" />
+ <DIV>
+ <xsl:attribute name="class"><xsl:value-of select="@class"/></xsl:attribute>
+ <DIV class="stylepoint_body">
+ <xsl:attribute name="name"><xsl:value-of select="$anchor_prefix"/><xsl:value-of select="$body_suffix"/></xsl:attribute>
+ <xsl:attribute name="id"><xsl:value-of select="$anchor_prefix"/><xsl:value-of select="$body_suffix"/></xsl:attribute>
+ <xsl:attribute name="style">
+ <xsl:choose>
+ <xsl:when test="$show_explanation_default">display: inline</xsl:when>
+ <xsl:otherwise>display: none</xsl:otherwise>
+ </xsl:choose>
+ </xsl:attribute>
+ <xsl:apply-templates/>
+ </DIV>
+ </DIV>
+ </xsl:template>
+
+ <xsl:template match="DEFINITION">
+ <P>
+ <xsl:attribute name="class"><xsl:value-of select="@class"/></xsl:attribute>
+ <SPAN class="stylepoint_section">Definition: </SPAN>
+ <xsl:apply-templates/>
+ </P>
+ </xsl:template>
+
+ <xsl:template match="PROS">
+ <P>
+ <xsl:attribute name="class"><xsl:value-of select="@class"/></xsl:attribute>
+ <SPAN class="stylepoint_section">Pros: </SPAN>
+ <xsl:apply-templates/>
+ </P>
+ </xsl:template>
+
+ <xsl:template match="CONS">
+ <P>
+ <xsl:attribute name="class"><xsl:value-of select="@class"/></xsl:attribute>
+ <SPAN class="stylepoint_section">Cons: </SPAN>
+ <xsl:apply-templates/>
+ </P>
+ </xsl:template>
+
+ <xsl:template match="DECISION">
+ <P>
+ <xsl:attribute name="class"><xsl:value-of select="@class"/></xsl:attribute>
+ <SPAN class="stylepoint_section">Decision: </SPAN>
+ <xsl:apply-templates/>
+ </P>
+ </xsl:template>
+
+ <xsl:template match="TODO">
+ <P>
+ <xsl:attribute name="class"><xsl:value-of select="@class"/></xsl:attribute>
+ <DIV style="font-size: 150%;">TODO:
+ <xsl:apply-templates/>
+ </DIV>
+ </P>
+ </xsl:template>
+
+ <xsl:template match="SUBSECTION">
+ <P>
+ <xsl:attribute name="class"><xsl:value-of select="@class"/></xsl:attribute>
+ <SPAN class="stylepoint_subsection"><xsl:value-of select="@title"/> </SPAN>
+ <xsl:apply-templates/>
+ </P>
+ </xsl:template>
+
+ <xsl:template match="SUBSUBSECTION">
+ <P>
+ <xsl:attribute name="class"><xsl:value-of select="@class"/></xsl:attribute>
+ <SPAN class="stylepoint_subsubsection"><xsl:value-of select="@title"/> </SPAN>
+ <xsl:apply-templates/>
+ </P>
+ </xsl:template>
+
+ <xsl:template match="CODE_SNIPPET">
+ <DIV>
+ <xsl:attribute name="class"><xsl:value-of select="@class"/></xsl:attribute>
+ <PRE><xsl:call-template name="print_without_leading_chars">
+ <xsl:with-param name="text" select="."/>
+ <xsl:with-param name="strip" select="1"/>
+ <xsl:with-param name="is_firstline" select="1"/>
+ <xsl:with-param name="trim_count">
+ <xsl:call-template name="num_leading_spaces">
+ <xsl:with-param name="text" select="."/>
+ <xsl:with-param name="max_so_far" select="1000"/>
+ </xsl:call-template>
+ </xsl:with-param>
+ </xsl:call-template></PRE>
+ </DIV>
+ </xsl:template>
+
+ <xsl:template match="BAD_CODE_SNIPPET">
+ <DIV>
+ <xsl:attribute name="class"><xsl:value-of select="@class"/></xsl:attribute>
+ <PRE class="badcode"><xsl:call-template name="print_without_leading_chars">
+ <xsl:with-param name="text" select="."/>
+ <xsl:with-param name="strip" select="1"/>
+ <xsl:with-param name="is_firstline" select="1"/>
+ <xsl:with-param name="trim_count">
+ <xsl:call-template name="num_leading_spaces">
+ <xsl:with-param name="text" select="."/>
+ <xsl:with-param name="max_so_far" select="1000"/>
+ </xsl:call-template>
+ </xsl:with-param>
+ </xsl:call-template></PRE>
+ </DIV>
+ </xsl:template>
+
+ <xsl:template match="PY_CODE_SNIPPET">
+ <DIV>
+ <xsl:attribute name="class"><xsl:value-of select="@class"/></xsl:attribute>
+ <PRE><xsl:call-template name="print_python_code">
+ <xsl:with-param name="text" select="."/>
+ </xsl:call-template></PRE>
+ </DIV>
+ </xsl:template>
+
+ <xsl:template match="BAD_PY_CODE_SNIPPET">
+ <DIV>
+ <xsl:attribute name="class"><xsl:value-of select="@class"/></xsl:attribute>
+ <PRE class="badcode"><xsl:call-template name="print_python_code">
+ <xsl:with-param name="text" select="."/>
+ </xsl:call-template></PRE>
+ </DIV>
+ </xsl:template>
+
+ <xsl:template match="FUNCTION">
+ <xsl:call-template name="print_function_name">
+ <xsl:with-param name="text" select="."/>
+ </xsl:call-template>
+ </xsl:template>
+
+ <xsl:template match="SYNTAX">
+ <I>
+ <xsl:attribute name="class"><xsl:value-of select="@class"/></xsl:attribute>
+ <xsl:apply-templates/>
+ </I>
+ </xsl:template>
+
+
+ <!-- This passes through any HTML elements that the
+ XML doc uses for minor formatting -->
+ <xsl:template match="a|address|blockquote|br|center|cite|code|dd|div|dl|dt|em|hr|i|img|li|ol|p|pre|span|table|td|th|tr|ul|var|A|ADDRESS|BLOCKQUOTE|BR|CENTER|CITE|CODE|DD|DIV|DL|DT|EM|HR|I|LI|OL|P|PRE|SPAN|TABLE|TD|TH|TR|UL|VAR">
+ <xsl:element name="{local-name()}">
+ <xsl:copy-of select="@*"/>
+ <xsl:apply-templates/>
+ </xsl:element>
+ </xsl:template>
+
+ <!-- Builds the table of contents -->
+ <xsl:template name="TOC">
+ <xsl:param name="root"/>
+ <DIV class="toc">
+ <DIV class="toc_title">Table of Contents</DIV>
+ <TABLE>
+ <xsl:for-each select="$root/CATEGORY">
+ <TR valign="top">
+ <xsl:attribute name="class"><xsl:value-of select="@class"/></xsl:attribute>
+ <TD>
+ <DIV class="toc_category">
+ <A>
+ <xsl:attribute name="href">
+ <xsl:text>#</xsl:text>
+ <xsl:call-template name="anchorname">
+ <xsl:with-param name="sectionname" select="@title"/>
+ </xsl:call-template>
+ </xsl:attribute>
+ <xsl:value-of select="@title"/>
+ </A>
+ </DIV>
+ </TD><TD>
+ <DIV class="toc_stylepoint">
+ <xsl:for-each select="./STYLEPOINT">
+ <SPAN style="padding-right: 1em; white-space:nowrap;">
+ <xsl:attribute name="class"><xsl:value-of select="@class"/></xsl:attribute>
+ <A>
+ <xsl:attribute name="href">
+ <xsl:text>#</xsl:text>
+ <xsl:call-template name="anchorname">
+ <xsl:with-param name="sectionname" select="@title"/>
+ </xsl:call-template>
+ </xsl:attribute>
+ <xsl:value-of select="@title"/>
+ </A>
+ </SPAN>
+ <xsl:text> </xsl:text>
+ </xsl:for-each>
+ </DIV>
+ </TD>
+ </TR>
+ </xsl:for-each>
+ </TABLE>
+ </DIV>
+ </xsl:template>
+
+ <xsl:template name="TOC_one_stylepoint">
+ <xsl:param name="stylepoint"/>
+ </xsl:template>
+
+ <!-- Creates a standard anchor given any text.
+ Substitutes underscore for characters unsuitable for URLs -->
+ <xsl:template name="anchorname">
+ <xsl:param name="sectionname"/>
+ <!-- strange quoting necessary to strip apostrophes -->
+ <xsl:variable name="bad_characters" select="" ()#'""/>
+ <xsl:value-of select="translate($sectionname,$bad_characters,'_____')"/>
+ </xsl:template>
+
+ <!-- Given text, evaluates to the number of leading spaces. -->
+ <!-- TODO(csilvers): deal well with leading tabs (treat as 8 spaces?) -->
+ <xsl:template name="num_leading_spaces_one_line">
+ <xsl:param name="text"/>
+ <xsl:param name="current_count"/>
+ <xsl:choose>
+ <xsl:when test="starts-with($text, ' ')">
+ <xsl:call-template name="num_leading_spaces_one_line">
+ <xsl:with-param name="text" select="substring($text, 2)"/>
+ <xsl:with-param name="current_count" select="$current_count + 1"/>
+ </xsl:call-template>
+ </xsl:when>
+ <xsl:otherwise>
+ <xsl:value-of select="$current_count"/>
+ </xsl:otherwise>
+ </xsl:choose>
+ </xsl:template>
+
+ <!-- Given a block of text, each line terminated by \n, evaluates to
+ the indentation-level of that text; that is, the largest number
+ n such that every non-blank line starts with at least n spaces. -->
+ <xsl:template name="num_leading_spaces">
+ <xsl:param name="text"/>
+ <xsl:param name="max_so_far"/>
+ <!-- TODO(csilvers): deal with case text doesn't end in a newline -->
+ <xsl:variable name="line" select="substring-before($text, '
')"/>
+ <xsl:variable name="rest" select="substring-after($text, '
')"/>
+ <xsl:variable name="num_spaces_this_line">
+ <xsl:choose>
+ <xsl:when test="$line=''">
+ <xsl:value-of select="$max_so_far"/>
+ </xsl:when>
+ <xsl:otherwise>
+ <xsl:call-template name="num_leading_spaces_one_line">
+ <xsl:with-param name="text" select="$line"/>
+ <xsl:with-param name="current_count" select="0"/>
+ </xsl:call-template>
+ </xsl:otherwise>
+ </xsl:choose>
+ </xsl:variable>
+ <xsl:variable name="new_max_so_far">
+ <xsl:choose>
+ <xsl:when test="$num_spaces_this_line < $max_so_far">
+ <xsl:value-of select="$num_spaces_this_line"/>
+ </xsl:when>
+ <xsl:otherwise>
+ <xsl:value-of select="$max_so_far"/>
+ </xsl:otherwise>
+ </xsl:choose>
+ </xsl:variable>
+ <!-- now check if we're on the last line, and if not, recurse -->
+ <xsl:if test="$rest=''">
+ <xsl:value-of select="$new_max_so_far"/>
+ </xsl:if>
+ <xsl:if test="not($rest='')">
+ <xsl:call-template name="num_leading_spaces">
+ <xsl:with-param name="text" select="$rest"/>
+ <xsl:with-param name="max_so_far" select="$new_max_so_far"/>
+ </xsl:call-template>
+ </xsl:if>
+ </xsl:template>
+
+ <!-- Given text, determine the starting position of code.
+ This similar to num_leading_spaces_one_line but treats "Yes:" and "No:"
+ as spaces. Also, if there is no code on the first line, it searches
+ subsequent lines until a non-empty line is found.
+ Used to find the start of code in snippets like:
+ Yes: if(foo):
+ No : if(foo):
+ As well as:
+ Yes:
+ if (foo):
+ -->
+ <xsl:template name="code_start_index">
+ <xsl:param name="text"/>
+ <xsl:param name="current_count"/>
+ <xsl:choose>
+ <xsl:when test="starts-with($text, ' ')">
+ <xsl:call-template name="code_start_index">
+ <xsl:with-param name="text" select="substring($text, 2)"/>
+ <xsl:with-param name="current_count" select="$current_count + 1"/>
+ </xsl:call-template>
+ </xsl:when>
+ <xsl:when test="starts-with($text, 'Yes:')">
+ <xsl:call-template name="code_start_index">
+ <xsl:with-param name="text" select="substring($text, 5)"/>
+ <xsl:with-param name="current_count" select="$current_count + 4"/>
+ </xsl:call-template>
+ </xsl:when>
+ <xsl:when test="starts-with($text, 'No:')">
+ <xsl:call-template name="code_start_index">
+ <xsl:with-param name="text" select="substring($text, 4)"/>
+ <xsl:with-param name="current_count" select="$current_count + 3"/>
+ </xsl:call-template>
+ </xsl:when>
+ <!-- This is only reached if the first line is entirely whitespace or
+ contains nothing but "Yes:" or "No:"-->
+ <xsl:when test="starts-with($text, '
')">
+ <xsl:call-template name="code_start_index">
+ <xsl:with-param name="text" select="substring($text, 2)"/>
+ <xsl:with-param name="current_count" select="0"/>
+ </xsl:call-template>
+ </xsl:when>
+ <xsl:otherwise>
+ <xsl:value-of select="$current_count"/>
+ </xsl:otherwise>
+ </xsl:choose>
+ </xsl:template>
+
+ <!-- Helper for ends_with_colon. Determine whether the given line is nothing
+ but spaces and python-style comments. -->
+ <xsl:template name="is_blank_or_comment">
+ <xsl:param name="line"/>
+ <xsl:choose>
+ <xsl:when test="$line = ''">
+ <xsl:value-of select="1"/>
+ </xsl:when>
+ <xsl:when test="starts-with($line, '
')">
+ <xsl:value-of select="1"/>
+ </xsl:when>
+ <xsl:when test="starts-with($line, '#')">
+ <xsl:value-of select="1"/>
+ </xsl:when>
+ <xsl:when test="starts-with($line, ' ')">
+ <xsl:call-template name="is_blank_or_comment">
+ <xsl:with-param name="line" select="substring($line, 2)"/>
+ </xsl:call-template>
+ </xsl:when>
+ <xsl:otherwise>
+ <xsl:value-of select="0"/>
+ </xsl:otherwise>
+ </xsl:choose>
+ </xsl:template>
+
+ <!-- Determine whether the given line ends with a colon. Note that Python
+ style comments are ignored so the following lines return True:
+ - def foo():
+ - def foo(): # Bar
+ - if(foo):
+
+ But some code may confuse this function. For example the following are
+ also consider to "end_with_colon" even though they don't for Python
+ - foo(": #")
+ - foo() # No need for :
+ -->
+ <xsl:template name="ends_with_colon">
+ <xsl:param name="line"/>
+ <xsl:param name="found_colon"/>
+ <xsl:choose>
+ <xsl:when test="$line = ''">
+ <xsl:value-of select="$found_colon"/>
+ </xsl:when>
+ <xsl:when test="starts-with($line, '
')">
+ <xsl:value-of select="$found_colon"/>
+ </xsl:when>
+ <xsl:when test="starts-with($line, ' ')">
+ <xsl:call-template name="ends_with_colon">
+ <xsl:with-param name="line" select="substring($line, 2)"/>
+ <xsl:with-param name="found_colon" select="$found_colon"/>
+ </xsl:call-template>
+ </xsl:when>
+ <xsl:when test="starts-with($line, ':')">
+ <xsl:variable name="rest_is_comment">
+ <xsl:call-template name="is_blank_or_comment">
+ <xsl:with-param name="line" select="substring($line, 2)"/>
+ </xsl:call-template>
+ </xsl:variable>
+ <xsl:choose>
+ <xsl:when test="$rest_is_comment = '1'">
+ <xsl:value-of select="1"/>
+ </xsl:when>
+ <xsl:otherwise>
+ <xsl:call-template name="ends_with_colon">
+ <xsl:with-param name="line" select="substring($line, 2)"/>
+ <xsl:with-param name="found_colon" select="0"/>
+ </xsl:call-template>
+ </xsl:otherwise>
+ </xsl:choose>
+ </xsl:when>
+ <xsl:otherwise>
+ <xsl:call-template name="ends_with_colon">
+ <xsl:with-param name="line" select="substring($line, 2)"/>
+ <xsl:with-param name="found_colon" select="0"/>
+ </xsl:call-template>
+ </xsl:otherwise>
+ </xsl:choose>
+ </xsl:template>
+
+ <!-- Prints one line of python code with proper indent and calls itself
+ recursively for the rest of the text.
+ This template uses "a", "b", "c" and "d" to refer to four key column
+ numbers. They are:
+ - a: the indentation common to all lines in a code snippet. This is
+ stripped out to allow for cleaner code in the xml.
+ - b: the indentation of the most out-dented line of code. This is
+ different from "a" when code is labelled with "Yes:" or "No:"
+ - c: the indentation of the current python block, in other words, the
+ indentation of the first line of this block, which is the
+ indentation of the last line we saw that ended with a colon.
+ - d: the "total" indentation of the line, ignorng possible "Yes:" or
+ "No:" text on the line.
+
+ For example, for the last line of the following code snippet, the
+ positions of a, b, c and d are indicated below:
+ Yes: def Foo():
+ if bar():
+ a += 1
+ baz()
+ a b c d
+
+ The algorithm is:
+ 1) Split the text into first line and the rest. Note that the
+ substring-before function is supposed to handle the case where the
+ character is not present in the string but does not so we
+ automatically ignore the last line of the snippet which is always
+ empty (the closing snippet tag). This is consistent with the
+ behavior or print_without_leading_chars.
+ 2) If the current is empty (only whitespace), print newline and call
+ itself recursively on the rest of the text with the rest of the
+ parameters unchanged.
+ 3) Otherwise, measure "d"
+ 4) Measure "c" by taking:
+ - the value of "d" if the previous line ended with a colon or the
+ current line is outdented compare to the previous line
+ - the indent of the previous line otherwise
+ 5) Print line[a:c] (Note that we ignore line[0:a])
+ 6) Print line[b:c] in an external span (in order to double the block
+ indent in external code).
+ 7) Print line[c:<end>] with function names processed to produce both
+ internal and external names.
+ 8) If there are more lines, recurse.
+ -->
+ <xsl:template name="print_python_line_recursively">
+ <xsl:param name="text"/>
+ <xsl:param name="a"/>
+ <xsl:param name="b"/>
+ <xsl:param name="previous_indent"/>
+ <xsl:param name="previous_ends_with_colon"/>
+ <xsl:param name="is_first_line"/>
+ <xsl:variable name="line" select="substring-before($text, '
')"/>
+ <xsl:variable name="rest" select="substring-after($text, '
')"/>
+ <xsl:choose>
+ <xsl:when test="substring($line, $b) = '' and not($rest = '')">
+ <xsl:if test="not($is_first_line = '1')">
+ <xsl:text>
</xsl:text>
+ </xsl:if>
+ <xsl:call-template name="print_python_line_recursively">
+ <xsl:with-param name="text" select="$rest"/>
+ <xsl:with-param name="a" select="$a"/>
+ <xsl:with-param name="b" select="$b"/>
+ <xsl:with-param name="previous_indent" select="$previous_indent"/>
+ <xsl:with-param name="previous_ends_with_colon"
+ select="$previous_ends_with_colon"/>
+ <xsl:with-param name="is_first_line" select="0"/>
+ </xsl:call-template>
+ </xsl:when>
+ <xsl:otherwise>
+ <xsl:variable name="indent_after_b">
+ <xsl:call-template name="num_leading_spaces_one_line">
+ <xsl:with-param name="text" select="substring($line, $b + 1)"/>
+ <xsl:with-param name="current_count" select="0"/>
+ </xsl:call-template>
+ </xsl:variable>
+ <xsl:variable name="d" select="$b + $indent_after_b"/>
+ <xsl:variable name="c">
+ <xsl:choose>
+ <xsl:when test="$previous_ends_with_colon = '1' or
+ $previous_indent > $d">
+ <xsl:value-of select="$d"/>
+ </xsl:when>
+ <xsl:otherwise>
+ <xsl:value-of select="$previous_indent"/>
+ </xsl:otherwise>
+ </xsl:choose>
+ </xsl:variable>
+
+ <xsl:value-of select="substring($line, $a + 1, $c - $a)"/>
+ <span class="external">
+ <xsl:value-of select="substring($line, $b + 1, $c - $b)"/>
+ </span>
+ <xsl:call-template name="munge_function_names_in_text">
+ <xsl:with-param name="stripped_line"
+ select="substring($line, $c + 1)"/>
+ </xsl:call-template>
+ <xsl:if test="not(substring($rest, $a) = '')">
+ <xsl:text>
</xsl:text>
+ <xsl:call-template name="print_python_line_recursively">
+ <xsl:with-param name="text" select="$rest"/>
+ <xsl:with-param name="a" select="$a"/>
+ <xsl:with-param name="b" select="$b"/>
+ <xsl:with-param name="previous_indent" select="$c"/>
+ <xsl:with-param name="previous_ends_with_colon">
+ <xsl:call-template name="ends_with_colon">
+ <xsl:with-param name="line" select="$line"/>
+ <xsl:with-param name="found_colon" select="0"/>
+ </xsl:call-template>
+ </xsl:with-param>
+ <xsl:with-param name="is_first_line" select="0"/>
+ </xsl:call-template>
+ </xsl:if>
+ </xsl:otherwise>
+ </xsl:choose>
+ </xsl:template>
+
+ <!-- Print python code with internal and external styles.
+ In order to conform with PEP-8 externally, we identify 2-space indents
+ and an external-only 4-space indent.
+ Function names that are marked with $$FunctionName/$$ have an external
+ lower_with_underscore version added. -->
+ <xsl:template name="print_python_code">
+ <xsl:param name="text"/>
+
+ <xsl:variable name="a">
+ <xsl:call-template name="num_leading_spaces">
+ <xsl:with-param name="text" select="."/>
+ <xsl:with-param name="max_so_far" select="1000"/>
+ </xsl:call-template>
+ </xsl:variable>
+
+ <xsl:variable name="b">
+ <xsl:call-template name="code_start_index">
+ <xsl:with-param name="text" select="$text"/>
+ <xsl:with-param name="current_count" select="0"/>
+ </xsl:call-template>
+ </xsl:variable>
+
+ <xsl:call-template name="print_python_line_recursively">
+ <xsl:with-param name="text" select="$text"/>
+ <xsl:with-param name="a" select="$a"/>
+ <xsl:with-param name="b" select="$b"/>
+ <xsl:with-param name="previous_indent" select="$b"/>
+ <xsl:with-param name="previous_ends_with_colon" select="0"/>
+ <xsl:with-param name="is_first_line" select="1"/>
+ </xsl:call-template>
+ </xsl:template>
+
+ <!-- Given a block of text, each line terminated by \n, and a number n,
+ emits the text with the first n characters of each line
+ deleted. If strip==1, then we omit blank lines at the beginning
+ and end of the text (but not the middle!) -->
+ <!-- TODO(csilvers): deal well with leading tabs (treat as 8 spaces?) -->
+ <xsl:template name="print_without_leading_chars">
+ <xsl:param name="text"/>
+ <xsl:param name="trim_count"/>
+ <xsl:param name="strip"/>
+ <xsl:param name="is_firstline"/>
+ <!-- TODO(csilvers): deal with case text doesn't end in a newline -->
+ <xsl:variable name="line" select="substring-before($text, '
')"/>
+ <xsl:variable name="rest" select="substring-after($text, '
')"/>
+ <xsl:variable name="stripped_line" select="substring($line, $trim_count+1)"/>
+ <xsl:choose>
+ <!-- $line (or $rest) is considered empty if we'd trim the entire line -->
+ <xsl:when test="($strip = '1') and ($is_firstline = '1') and
+ (string-length($line) <= $trim_count)">
+ </xsl:when>
+ <xsl:when test="($strip = '1') and
+ (string-length($rest) <= $trim_count)">
+ <xsl:value-of select="$stripped_line"/>
+ </xsl:when>
+ <xsl:otherwise>
+ <xsl:value-of select="$stripped_line"/>
+ <xsl:text>
</xsl:text>
+ </xsl:otherwise>
+ </xsl:choose>
+ <xsl:if test="not($rest='')">
+ <xsl:call-template name="print_without_leading_chars">
+ <xsl:with-param name="text" select="$rest"/>
+ <xsl:with-param name="trim_count" select="$trim_count"/>
+ <xsl:with-param name="strip" select="$strip"/>
+ <xsl:with-param name="is_firstline" select="0"/>
+ </xsl:call-template>
+ </xsl:if>
+ </xsl:template>
+
+ <!-- Given a line of code, find function names that are marked with $$ /$$ and
+ print out the line with the internal and external versions of the
+ function names.-->
+ <xsl:template name="munge_function_names_in_text">
+ <xsl:param name="stripped_line"/>
+ <xsl:choose>
+ <xsl:when test="contains($stripped_line, '$$')">
+ <xsl:value-of select="substring-before($stripped_line, '$$')"/>
+ <xsl:call-template name="print_function_name">
+ <xsl:with-param name="text" select="substring-after(substring-before($stripped_line, '/$$'), '$$')"/>
+ </xsl:call-template>
+ <xsl:call-template name="munge_function_names_in_text">
+ <xsl:with-param name="stripped_line" select="substring-after($stripped_line, '/$$')"/>
+ </xsl:call-template>
+ </xsl:when>
+ <xsl:otherwise>
+ <xsl:value-of select="$stripped_line"/>
+ </xsl:otherwise>
+ </xsl:choose>
+ </xsl:template>
+
+ <!-- Given a function name, print out both the internal and external version
+ of the function name in their respective spans.-->
+ <xsl:template name="print_function_name">
+ <xsl:param name="text"/>
+ <xsl:call-template name="convert_camel_case_to_lowercase_with_under">
+ <xsl:with-param name="text" select="$text"/>
+ </xsl:call-template>
+ </xsl:template>
+
+ <!-- Given a single word of text convert it from CamelCase to
+ lower_with_under.
+ This means replacing each uppercase character with _ followed by the
+ lowercase version except for the first character which is replaced
+ without adding the _.-->
+ <xsl:template name="convert_camel_case_to_lowercase_with_under">
+ <xsl:param name="text"/>
+ <xsl:param name="is_recursive_call"/>
+ <xsl:variable name="first_char" select="substring($text, 1, 1)"/>
+ <xsl:variable name="rest" select="substring($text, 2)"/>
+ <xsl:choose>
+ <xsl:when test="contains('ABCDEFGHIJKLMNOPQRSTUVWXYZ', $first_char)">
+ <xsl:if test="$is_recursive_call='1'">
+ <xsl:text>_</xsl:text>
+ </xsl:if>
+ <xsl:value-of select="translate($first_char, 'ABCDEFGHIJKLMNOPQRSTUVWXYZ', 'abcdefghijklmnopqrstuvwxyz')"/>
+ </xsl:when>
+ <xsl:otherwise>
+ <xsl:value-of select="$first_char" />
+ </xsl:otherwise>
+ </xsl:choose>
+ <xsl:if test="not($rest='')">
+ <xsl:call-template name="convert_camel_case_to_lowercase_with_under">
+ <xsl:with-param name="text" select="$rest"/>
+ <xsl:with-param name="is_recursive_call" select="1"/>
+ </xsl:call-template>
+ </xsl:if>
+ </xsl:template>
+</xsl:stylesheet>
+
