Print

Print


I noticed we are inconsistent about <> vs "" when we
include boost, for example, in  qserv, we have 146 <>
and 36 "". I peeked at afw, and it is also not
consistent either, although in the other direction:
46 <> and 109 "".

According to our coding standard 4.15

https://dev.lsstcorp.org/trac/wiki/C%2B%2BStandard/Files#a4-15.OnlysystemincludefilepathsSHALLbedelimitedwith

we should be using "" for boost...

I'm going to clean in in qserv.

Jacek



pwd
~/work/lsst/software/qserv/core/modules

grep 'include <boost' */* |wc
     146     314    7916

grep 'include "boost' */* |wc
      36      79    2208

pwd
~/work/lsst/software/afw/src

grep 'include <boost' */* |wc
      46      92    3256

grep 'include "boost' */* |wc
     109     218    6053

########################################################################
Use REPLY-ALL to reply to list

To unsubscribe from the QSERV-L list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=QSERV-L&A=1