Boost logo

Boost-Commit :

From: jurko.gospodnetic_at_[hidden]
Date: 2008-06-08 18:36:09


Author: jurko
Date: 2008-06-08 18:36:08 EDT (Sun, 08 Jun 2008)
New Revision: 46263
URL: http://svn.boost.org/trac/boost/changeset/46263

Log:
Minor stylistic Boost Build script changes.
Text files modified:
   trunk/tools/build/v2/build/type.jam | 22 ++++++++++++++++++++--
   trunk/tools/build/v2/build/version.jam | 2 +-
   trunk/tools/build/v2/build/virtual-target.jam | 2 +-
   trunk/tools/build/v2/kernel/errors.jam | 2 +-
   trunk/tools/build/v2/tools/msvc.jam | 4 ++--
   5 files changed, 25 insertions(+), 7 deletions(-)

Modified: trunk/tools/build/v2/build/type.jam
==============================================================================
--- trunk/tools/build/v2/build/type.jam (original)
+++ trunk/tools/build/v2/build/type.jam 2008-06-08 18:36:08 EDT (Sun, 08 Jun 2008)
@@ -18,8 +18,8 @@
 # project -> project-root -> builtin -> type -> targets -> project
 # import targets ;
 
-# The feature is optional so it would never get added implicitly. It's used only
-# for internal purposes and in all cases we want to use it explicitly.
+# The feature is optional so it would never get added implicitly. It is used
+# only for internal purposes and in all cases we want to use it explicitly.
 feature.feature target-type : : composite optional ;
 
 feature.feature main-target-type : : optional incidental ;
@@ -30,6 +30,7 @@
 # of 'suffixes' here is a shortcut for separately calling the register-suffixes
 # rule with the given suffixes and the set-generated-target-suffix rule with the
 # first given suffix.
+#
 rule register ( type : suffixes * : base-type ? )
 {
     # Type names cannot contain hyphens, because when used as feature-values
@@ -81,6 +82,7 @@
 
 # Given a type, returns the name of the main target rule which creates targets
 # of that type.
+#
 rule type-to-rule-name ( type )
 {
     # Lowercase everything. Convert underscores to dashes.
@@ -91,6 +93,7 @@
 
 
 # Given a main target rule name, returns the type for which it creates targets.
+#
 rule type-from-rule-name ( rule-name )
 {
     return $(.main-target-type.$(rule-name)) ;
@@ -100,6 +103,7 @@
 # Specifies that files with suffix from 'suffixes' be recognized as targets of
 # type 'type'. Issues an error if a different type is already specified for any
 # of the suffixes.
+#
 rule register-suffixes ( suffixes + : type )
 {
     for local s in $(suffixes)
@@ -118,6 +122,7 @@
 
 
 # Returns true iff type has been registered.
+#
 rule registered ( type )
 {
     if $(type) in $(.types)
@@ -128,6 +133,7 @@
 
 
 # Issues an error if 'type' is unknown.
+#
 rule validate ( type )
 {
     if ! $(type) in $(.types)
@@ -138,6 +144,7 @@
 
 
 # Sets a scanner class that will be used for this 'type'.
+#
 rule set-scanner ( type : scanner )
 {
     if ! $(type) in $(.types)
@@ -149,6 +156,7 @@
 
 
 # Returns a scanner instance appropriate to 'type' and 'properties'.
+#
 rule get-scanner ( type : property-set )
 {
     if $(.scanner.$(type))
@@ -160,6 +168,7 @@
 
 # Returns the given type and all of its base types in order of their distance
 # from type.
+#
 rule all-bases ( type )
 {
     local result = $(type) ;
@@ -184,6 +193,7 @@
 
 
 # Returns true if 'type' has 'base' as its direct or indirect base.
+#
 rule is-derived ( type base )
 {
     if $(base) in [ all-bases $(type) ]
@@ -194,6 +204,7 @@
 
 
 # Returns true if 'type' is either derived from or is equal to 'base'.
+#
 rule is-subtype ( type base )
 {
     if $(type) = $(base)
@@ -233,6 +244,7 @@
 
 # Change the suffix previously registered for this type/properties combination.
 # If suffix is not yet specified, sets it.
+#
 rule change-generated-target-suffix ( type : properties * : suffix )
 {
     change-generated-target-ps suffix : $(type) : $(properties) : $(suffix) ;
@@ -241,6 +253,7 @@
 
 # Returns the suffix used when generating a file of 'type' with the given
 # properties.
+#
 rule generated-target-suffix ( type : property-set )
 {
     return [ generated-target-ps suffix : $(type) : $(property-set) ] ;
@@ -255,6 +268,7 @@
 # should be used.
 #
 # Usage example: library names use the "lib" prefix on unix.
+#
 rule set-generated-target-prefix ( type : properties * : prefix )
 {
     set-generated-target-ps prefix : $(type) : $(properties) : $(prefix) ;
@@ -263,6 +277,7 @@
 
 # Change the prefix previously registered for this type/properties combination.
 # If prefix is not yet specified, sets it.
+#
 rule change-generated-target-prefix ( type : properties * : prefix )
 {
     change-generated-target-ps prefix : $(type) : $(properties) : $(prefix) ;
@@ -299,6 +314,7 @@
 # when generating a target of 'type' with the specified properties. Parameter
 # 'ps' can be either "prefix" or "suffix". If no prefix/suffix is specified for
 # 'type', returns prefix/suffix for base type, if any.
+#
 local rule generated-target-ps-real ( ps : type : properties * )
 {
     local result ;
@@ -348,6 +364,7 @@
 # Returns file type given it's name. If there are several dots in filename,
 # tries each suffix. E.g. for name of "file.so.1.2" suffixes "2", "1", and "so"
 # will be tried.
+#
 rule type ( filename )
 {
     local type ;
@@ -365,6 +382,7 @@
 # into the global namespace under different alias names and exactly what type of
 # target it is supposed to construct is read from the name of the alias rule
 # actually used to invoke it.
+#
 rule main-target-rule ( name : sources * : requirements * : default-build *
     : usage-requirements * )
 {

Modified: trunk/tools/build/v2/build/version.jam
==============================================================================
--- trunk/tools/build/v2/build/version.jam (original)
+++ trunk/tools/build/v2/build/version.jam 2008-06-08 18:36:08 EDT (Sun, 08 Jun 2008)
@@ -90,7 +90,7 @@
         errors.error Invalid version specifier: : $(version:E="(undefined)") ;
     }
 
- if ! $(.jam-version-check.$(version-tag))-is-defined
+ if ! $(.jam-version-check.$(version-tag))-is-not-empty
     {
         local jam-version = [ modules.peek : JAM_VERSION ] ;
         if ! $(jam-version)

Modified: trunk/tools/build/v2/build/virtual-target.jam
==============================================================================
--- trunk/tools/build/v2/build/virtual-target.jam (original)
+++ trunk/tools/build/v2/build/virtual-target.jam 2008-06-08 18:36:08 EDT (Sun, 08 Jun 2008)
@@ -1216,7 +1216,7 @@
     rule implicit-includes ( feature : target-type ? )
     {
         local key = ii$(feature)-$(target-type:E="") ;
- if ! $($(key))-is-nonempty
+ if ! $($(key))-is-not-empty
         {
             local target-paths = [ all-target-directories $(target-type) ] ;
             target-paths = [ sequence.unique $(target-paths) ] ;

Modified: trunk/tools/build/v2/kernel/errors.jam
==============================================================================
--- trunk/tools/build/v2/kernel/errors.jam (original)
+++ trunk/tools/build/v2/kernel/errors.jam 2008-06-08 18:36:08 EDT (Sun, 08 Jun 2008)
@@ -89,7 +89,7 @@
 
     import sequence ;
 
- if ! $(.last-error-$(.args))-is-nonempty
+ if ! $(.last-error-$(.args))-is-not-empty
     {
         error-skip-frames 3 expected an error, but none occurred ;
     }

Modified: trunk/tools/build/v2/tools/msvc.jam
==============================================================================
--- trunk/tools/build/v2/tools/msvc.jam (original)
+++ trunk/tools/build/v2/tools/msvc.jam 2008-06-08 18:36:08 EDT (Sun, 08 Jun 2008)
@@ -393,9 +393,9 @@
 
                 setup-$(c) = [ feature.get-values <setup-$(c)> : $(options) ] ;
 
- if ! $(setup-$(c))-is-defined
+ if ! $(setup-$(c))-is-not-empty
                 {
- if $(global-setup)-is-defined
+ if $(global-setup)-is-not-empty
                     {
                         setup-$(c) = $(global-setup) ;
                         


Boost-Commit list run by bdawes at acm.org, david.abrahams at rcn.com, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk