BitShares Forum

Main => 中文 (Chinese) => Topic started by: cdryan on June 10, 2014, 02:35:44 pm

Title: emfrias是兼职人员吗.按他的代码提交进度10天半月WIN32 的编译能通过不..
Post by: cdryan on June 10, 2014, 02:35:44 pm
emfrias是兼职人员吗.按他的代码提交进度10天半月WIN32 的编译能通过不..
 void create_asset_operation::evaluate( transaction_evaluation_state& eval_state )
     { try {
 -      if( not is_power_of_ten( this->precision ) )
 +      if( !is_power_of_ten( this->precision ) )
           FC_CAPTURE_AND_THROW( invalid_precision, (precision) );
 
 -      if( not eval_state._current_state->is_valid_symbol_name( this->symbol ) )
 +      if( !eval_state._current_state->is_valid_symbol_name( this->symbol ) )
           FC_CAPTURE_AND_THROW( invalid_asset_symbol, (symbol) );
 
        if( this->maximum_share_supply <= 0 )
 @@ -27,10 +27,10 @@ namespace bts { namespace blockchain {
        if( issuer_account_id != asset_record::market_issued_asset )
        {
           auto issuer_account_record = eval_state._current_state->get_account_record( this->issuer_account_id );
 -         if( not issuer_account_record )
 +         if( !issuer_account_record )
              FC_CAPTURE_AND_THROW( unknown_account_id, (issuer_account_id) );
 
 -         if( not eval_state.check_signature( issuer_account_record->active_address() ) )
 +         if( !eval_state.check_signature( issuer_account_record->active_address() ) )
              FC_CAPTURE_AND_THROW( missing_signature, (issuer_account_record) );
        }
 
 @@ -60,25 +60,25 @@ namespace bts { namespace blockchain {
     void update_asset_operation::evaluate( transaction_evaluation_state& eval_state )
     { try {
        auto current_asset_record = eval_state._current_state->get_asset_record( this->asset_id );
 -      if( not current_asset_record )
 +      if( !current_asset_record )
           FC_CAPTURE_AND_THROW( unknown_asset_id, (asset_id) );
 
        auto issuer_account_record = eval_state._current_state->get_account_record( current_asset_record->issuer_account_id );
 
 -      if( not issuer_account_record )
 +      if( !issuer_account_record )
           FC_CAPTURE_AND_THROW( unknown_account_id, (current_asset_record->issuer_account_id) );
 
 -      if( not eval_state.check_signature(issuer_account_record->active_address()) )
 +      if( !eval_state.check_signature(issuer_account_record->active_address()) )
            FC_CAPTURE_AND_THROW( missing_signature, (issuer_account_record->active_key()) );
 
        if( this->issuer_account_id != current_asset_record->issuer_account_id )
        {
            auto new_issuer_account_record = eval_state._current_state->get_account_record( this->issuer_account_id );
 
 -          if( not new_issuer_account_record )
 +          if( !new_issuer_account_record )
                FC_CAPTURE_AND_THROW( unknown_account_id, (issuer_account_id) );
 
 -          if( not eval_state.check_signature(new_issuer_account_record->active_address()) )
 +          if( !eval_state.check_signature(new_issuer_account_record->active_address()) )
                FC_CAPTURE_AND_THROW( missing_signature, (new_issuer_account_record->active_key()) );
        }
 
 @@ -100,19 +100,19 @@ namespace bts { namespace blockchain {
           FC_CAPTURE_AND_THROW( negative_issue, (amount) );
 
        auto current_asset_record = eval_state._current_state->get_asset_record( this->amount.asset_id );
 -      if( not current_asset_record )
 +      if( !current_asset_record )
           FC_CAPTURE_AND_THROW( unknown_asset_id, (amount.asset_id) );
 
        auto issuer_account_record = eval_state._current_state->get_account_record( current_asset_record->issuer_account_id );
 -      if( not issuer_account_record )
 +      if( !issuer_account_record )
           FC_CAPTURE_AND_THROW( unknown_account_id, (current_asset_record->issuer_account_id) );
 
 -      if( not eval_state.check_signature( issuer_account_record->active_address() ) )
 +      if( !eval_state.check_signature( issuer_account_record->active_address() ) )
        {
           FC_CAPTURE_AND_THROW( missing_signature, (issuer_account_record->active_key()) );
        }
 
 -      if( not current_asset_record->can_issue( this->amount ) )
 +      if( !current_asset_record->can_issue( this->amount ) )
        {
           FC_CAPTURE_AND_THROW( over_issue, (amount)(current_asset_record) );
        }

今天修改就是,not  改称! :'(..看来主要矛盾还是人手...
Title: Re: emfrias是兼职人员吗.按他的代码提交进度10天半月WIN32 的编译能通过不..
Post by: crazybit on June 10, 2014, 02:48:37 pm
win32一直都能编译通过。
Title: Re: emfrias是兼职人员吗.按他的代码提交进度10天半月WIN32 的编译能通过不..
Post by: Amazon on June 10, 2014, 02:49:04 pm
他是全职阿,很猛很猛的,不能就靠一段代码判断
Title: Re: emfrias是兼职人员吗.按他的代码提交进度10天半月WIN32 的编译能通过不..
Post by: sudo on June 10, 2014, 03:14:57 pm
The C keyword 'not' is not friendly on VC++ but we still want to use
 + *  it for readability, so we will have the pre-processor convert it to the
 + *  more traditional form.  The goal here is to make the understanding of
 + *  the validation logic as english-like as possible.

bm说的

https://github.com/BitShares/bitshares_toolkit/commit/c426d21d44e5fe54ac6cdbf4682776cacd7fd6b1
Title: Re: emfrias是兼职人员吗.按他的代码提交进度10天半月WIN32 的编译能通过不..
Post by: cdryan on June 10, 2014, 03:49:13 pm
他是全职阿,很猛很猛的,不能就靠一段代码判断

 ::) 真没看出来,因为BUG还那么多.WIN 的版本钱包测试也没释放...应该有很多活.
Title: Re: emfrias是兼职人员吗.按他的代码提交进度10天半月WIN32 的编译能通过不..
Post by: muse-umum on June 10, 2014, 04:09:46 pm
他就是Eric吧?

我记得某次月报里说过他是网络编程方面的专家,Dan Notestein把他挖来的,当时给予了他非常高的评价。toolkit的p2p网络架构就是他设计的。toast之前说过他加入bitshares开发团队之后才知道他离BM和Eric的差距是有多么的大。

其他的就不清楚了,他貌似很低调的,之前bitshares官网里团队介绍的页面上,就他是没有头像的,现在干脆不出现在上面了。
Title: Re: emfrias是兼职人员吗.按他的代码提交进度10天半月WIN32 的编译能通过不..
Post by: cdryan on June 10, 2014, 04:22:35 pm
他就是Eric吧?

我记得某次月报里说过他是网络编程方面的专家,Dan Notestein把他挖来的,当时给予了他非常高的评价。toolkit的p2p网络架构就是他设计的。toast之前说过他加入bitshares开发团队之后才知道他离BM和Eric的差距是有多么的大。

其他的就不清楚了,他貌似很低调的,之前bitshares官网里团队介绍的页面上,就他是没有头像的,现在干脆不出现在上面了。

希望如此,很久没看代码提交了,就看了最近几天.感觉BM是顶梁柱.
Title: Re: emfrias是兼职人员吗.按他的代码提交进度10天半月WIN32 的编译能通过不..
Post by: gyhy on June 11, 2014, 01:43:26 am
牛人 +5%