Skip to content
Snippets Groups Projects
  1. Dec 07, 2009
  2. Dec 04, 2009
  3. Dec 03, 2009
  4. Dec 02, 2009
  5. Dec 03, 2009
    • Mark Olesen's avatar
      PackedBoolList specializaton for operator= · 063d8ede
      Mark Olesen authored
      - now that I re-examined the code, the note in commit 51fd6327
        can be mostly ignored
      
        PackedList isMaster(nPoints, 1u);
        is not really inefficient at all, since the '1u' is packed into
        32/64-bits before the subsequent assignment and doesn't involve
        shifts/masking for each index
      
        The same misinformation applies to the PackedList(size, 0u) form.
        It isn't much slower at all.
      
        Nonetheless, add bool specialization so that it is a simple assign.
      063d8ede
    • Mark Olesen's avatar
      58740164
    • Mark Olesen's avatar
      commit existing sizeof test · 67b79d92
      Mark Olesen authored
      67b79d92
    • Mark Olesen's avatar
      pedantic changes: 'forAll (' -> 'forAll(' in applications/ · c091d856
      Mark Olesen authored
      - to match coding guidelines
      c091d856
    • Mark Olesen's avatar
      Use argList::addOption, argList::addBoolOption (almost) everywhere · 58b7e641
      Mark Olesen authored
      - ensure that the standard options (eg, from timeSelector) also have
        some usage information
      58b7e641
    • Mark Olesen's avatar
      argList fixes/enhancements · ca7acea5
      Mark Olesen authored
      - bugfix: noParallel() didn't remove 'parallel' from validParOptions
        allowing it to sneak through to the Pstream layer.
        noParallel() now clears the entire validParOptions as well
      
      - new convenience methods
        * addOption()
        * removeOption()
        * addBoolOption() - as per addOption(), but for bool options (no param)
      
      - printUsage() output format
        * options sorted alphabetically
        * options listed on separate lines for better readability
      
      - new optionUsage static member for short usage information about
        an option
        * corresponding addUsage() method or provide usage information
          in addOption() / addBoolOption()
      ca7acea5
  6. Dec 02, 2009
    • henry's avatar
      Cosmetic changes. · bd8ff136
      henry authored
      bd8ff136
    • Mark Olesen's avatar
      use PackedBoolList typedef instead of PackedList<1> · 51fd6327
      Mark Olesen authored
      Note:
      
      PackedList constructor initializes to zero, faster not to do it
      ourselves.
      ie,
          PackedList foo(nPoints);
      vs.
          PackedList foo(nPoints, 0);
      
      saves an extra nPoints operations with shifts/masks etc.
      
      If speed is important, change this type of code
      
          PackedList isMaster(nPoints, 1u);
          for (loop)
          {
              if (condition)
              {
                  isMaster.set(i, 0u);   // unset bit
              }
          }
          return isMaster;
      
      into this:
      
          PackedList notMaster(nPoints);
          for (loop)
          {
              if (!condition)
              {
                  notMaster.set(i, 1u);
              }
          }
          notMaster.flip();
          return notMaster;
      
      or this:
      
          PackedList isMaster(nPoints);
          isMaster.flip();
      
          for (loop)
          {
              if (condition)
              {
                  isMaster.set(i, 0u);
              }
          }
          return isMaster;
      51fd6327