Skip to content

Commit

Permalink
Merge pull request flux-framework#1216 from trws/adjust-comment-blocks
Browse files Browse the repository at this point in the history 
comment blocks: stop using full-star boxes
  • Loading branch information
@mergify
mergify[bot] authored Jun 6, 2024
2 parents e90f707 + 1442bb9 commit 1e3efce
Show file tree
Hide file tree
Showing 32 changed files with 352 additions and 440 deletions.
18 changes: 8 additions & 10 deletions qmanager/modules/qmanager.cpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -32,11 +32,10 @@ using namespace Flux::opts_manager;
using namespace Flux::cplusplus_wrappers;


/******************************************************************************
* *
* Queue Manager Service Module Context *
* *
******************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Queue Manager Service Module Context
////////////////////////////////////////////////////////////////////////////////

class fluxion_resource_interface_t {
public:
Expand Down Expand Up @@ -605,11 +604,10 @@ static const struct flux_msg_handler_spec htab[] = {
};


/******************************************************************************
* *
* Module Main *
* *
******************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Module Main
////////////////////////////////////////////////////////////////////////////////

int mod_start (flux_t *h, int argc, char **argv)
{
Expand Down
27 changes: 12 additions & 15 deletions qmanager/modules/qmanager_opts.cpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,10 @@ using namespace Flux;
using namespace Flux::opts_manager;


/******************************************************************************
* *
* Private API for Queue Manager Option Class *
* *
******************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Private API for Queue Manager Option Class
////////////////////////////////////////////////////////////////////////////////

int qmanager_opts_t::parse_queues (const std::string &queues)
{
Expand Down Expand Up @@ -54,11 +53,10 @@ int qmanager_opts_t::parse_queues (const std::string &queues)
}


/******************************************************************************
* *
* Private API for Queue Property Class *
* *
******************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Private API for Queue Property Class
////////////////////////////////////////////////////////////////////////////////

bool queue_prop_t::known_queue_policy (const std::string &policy)
{
Expand All @@ -70,11 +68,10 @@ bool queue_prop_t::known_queue_policy (const std::string &policy)
}


/******************************************************************************
* *
* Public API for Queue Manager Option Class *
* *
******************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Public API for Queue Manager Option Class
////////////////////////////////////////////////////////////////////////////////

const std::string &queue_prop_t::get_queue_policy () const
{
Expand Down
18 changes: 8 additions & 10 deletions qmanager/policies/queue_policy_bf_base_impl.hpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -17,11 +17,10 @@ namespace Flux {
namespace queue_manager {
namespace detail {

/******************************************************************************
* *
* Private Methods of Queue Policy Backfill Base *
* *
******************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Private Methods of Queue Policy Backfill Base
////////////////////////////////////////////////////////////////////////////////

template<class reapi_type>
int queue_policy_bf_base_t<reapi_type>::cancel_completed_jobs (void *h)
Expand Down Expand Up @@ -147,11 +146,10 @@ int queue_policy_bf_base_t<reapi_type>::allocate_orelse_reserve_jobs (void *h,
return 0;
}

/******************************************************************************
* *
* Public API of Queue Policy Backfill Base *
* *
******************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Public API of Queue Policy Backfill Base
////////////////////////////////////////////////////////////////////////////////

template<class reapi_type>
queue_policy_bf_base_t<reapi_type>::~queue_policy_bf_base_t ()
Expand Down
18 changes: 8 additions & 10 deletions qmanager/policies/queue_policy_fcfs_impl.hpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -19,11 +19,10 @@ namespace queue_manager {
namespace detail {


/******************************************************************************
* *
* Private Methods of Queue Policy FCFS *
* *
******************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Private Methods of Queue Policy FCFS
////////////////////////////////////////////////////////////////////////////////

template<class reapi_type>
int queue_policy_fcfs_t<reapi_type>::cancel_completed_jobs (void *h)
Expand Down Expand Up @@ -157,11 +156,10 @@ int queue_policy_fcfs_t<reapi_type>::handle_match_failure (int errcode)
}


/******************************************************************************
* *
* Public API of Queue Policy FCFS *
* *
******************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Public API of Queue Policy FCFS
////////////////////////////////////////////////////////////////////////////////

template<class reapi_type>
queue_policy_fcfs_t<reapi_type>::~queue_policy_fcfs_t ()
Expand Down
9 changes: 4 additions & 5 deletions resource/evaluators/edge_eval_api.cpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,10 @@ namespace Flux {
namespace resource_model {


/****************************************************************************
* *
* Edge Evaluator Public Method Definitions *
* *
****************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Edge Evaluator Public Method Definitions
////////////////////////////////////////////////////////////////////////////////

eval_edg_t::eval_edg_t (unsigned int c, unsigned int n, unsigned int x, edg_t e)
: count (c), needs (n), exclusive (x), edge (e)
Expand Down
18 changes: 8 additions & 10 deletions resource/evaluators/expr_eval_api.cpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,10 @@ namespace Flux {
namespace resource_model {


/****************************************************************************
* *
* Expression Evaluation API Private Method Definitions *
* *
****************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Expression Evaluation API Private Method Definitions
////////////////////////////////////////////////////////////////////////////////

bool expr_eval_api_t::is_paren (const std::string &e, std::size_t at) const
{
Expand Down Expand Up @@ -259,11 +258,10 @@ int expr_eval_api_t::evaluate_pred (pred_op_t op,
}


/****************************************************************************
* *
* Expression Evaluation API Public Method Definitions *
* *
****************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Expression Evaluation API Public Method Definitions
////////////////////////////////////////////////////////////////////////////////

int expr_eval_api_t::validate (const std::string &e,
const expr_eval_target_base_t &target)
Expand Down
18 changes: 8 additions & 10 deletions resource/evaluators/scoring_api.cpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,10 @@ namespace Flux {
namespace resource_model {


/****************************************************************************
* *
* Scoring API Private Method Definitions *
* *
****************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Scoring API Private Method Definitions
////////////////////////////////////////////////////////////////////////////////

void scoring_api_t::handle_new_keys (const subsystem_t &s,
const std::string &r)
Expand All @@ -51,11 +50,10 @@ void scoring_api_t::handle_new_resrc_type (const subsystem_t &s,
}


/****************************************************************************
* *
* Scoring API Public Method Definitions *
* *
****************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Scoring API Public Method Definitions
////////////////////////////////////////////////////////////////////////////////

scoring_api_t::scoring_api_t ()
{
Expand Down
54 changes: 24 additions & 30 deletions resource/modules/resource_match.cpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -39,11 +39,10 @@ using namespace Flux::opts_manager;
// Global perf struct from schema
extern struct Flux::resource_model::match_perf_t Flux::resource_model::perf;

/******************************************************************************
* *
* Resource Matching Service Module Context *
* *
******************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Resource Matching Service Module Context
////////////////////////////////////////////////////////////////////////////////

class msg_wrap_t {
public:
Expand Down Expand Up @@ -205,11 +204,10 @@ resource_ctx_t::~resource_ctx_t ()
}


/******************************************************************************
* *
* Request Handler Prototypes *
* *
******************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Request Handler Prototypes
////////////////////////////////////////////////////////////////////////////////

static void match_request_cb (flux_t *h, flux_msg_handler_t *w,
const flux_msg_t *msg, void *arg);
Expand Down Expand Up @@ -305,11 +303,10 @@ static const struct flux_msg_handler_spec htab[] = {
FLUX_MSGHANDLER_TABLE_END
};

/******************************************************************************
* *
* Module Initialization Routines *
* *
******************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Module Initialization Routines
////////////////////////////////////////////////////////////////////////////////

static void set_default_args (std::shared_ptr<resource_ctx_t> &ctx)
{
Expand Down Expand Up @@ -474,11 +471,10 @@ static std::shared_ptr<resource_ctx_t> init_module (flux_t *h,
}


/******************************************************************************
* *
* Resource Graph and Traverser Initialization *
* *
******************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Resource Graph and Traverser Initialization
////////////////////////////////////////////////////////////////////////////////

static int create_reader (std::shared_ptr<resource_ctx_t> &ctx,
const std::string &format)
Expand Down Expand Up @@ -1487,11 +1483,10 @@ static int init_resource_graph (std::shared_ptr<resource_ctx_t> &ctx)
}


/******************************************************************************
* *
* Request Handler Routines *
* *
******************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Request Handler Routines
////////////////////////////////////////////////////////////////////////////////

static void update_match_perf (double elapsed, int64_t jobid,
bool match_success)
Expand Down Expand Up @@ -2820,11 +2815,10 @@ static void set_status_request_cb (flux_t *h, flux_msg_handler_t *w,
return;
}

/******************************************************************************
* *
* Module Main *
* *
******************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Module Main
////////////////////////////////////////////////////////////////////////////////

extern "C" int mod_main (flux_t *h, int argc, char **argv)
{
Expand Down
27 changes: 12 additions & 15 deletions resource/modules/resource_match_opts.cpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -24,11 +24,10 @@ using namespace Flux::resource_model;
using namespace Flux::opts_manager;


/******************************************************************************
* *
* Public API for Resource Match Option Class *
* *
******************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Public API for Resource Match Option Class
////////////////////////////////////////////////////////////////////////////////

const std::string &resource_prop_t::get_load_file () const
{
Expand Down Expand Up @@ -213,11 +212,10 @@ json_t *resource_prop_t::jsonify () const
}


/******************************************************************************
* *
* Private API for Resource Match Property Class *
* *
******************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Private API for Resource Match Property Class
////////////////////////////////////////////////////////////////////////////////

bool resource_opts_t::is_number (const std::string &num_str)
{
Expand All @@ -229,11 +227,10 @@ bool resource_opts_t::is_number (const std::string &num_str)
}


/******************************************************************************
* *
* Public API for Resource Match Property Class *
* *
******************************************************************************/

////////////////////////////////////////////////////////////////////////////////
// Public API for Resource Match Property Class
////////////////////////////////////////////////////////////////////////////////

resource_opts_t::resource_opts_t ()
{
Expand Down
Loading

0 comments on commit 1e3efce

Please sign in to comment.