Go Back   vb.org Archive > Community Central > vBulletin.org Site Feedback
FAQ Community Calendar Today's Posts Search

Closed Thread
 
Thread Tools Display Modes
  #121  
Old 08-02-2005, 12:32 PM
Boofo's Avatar
Boofo Boofo is offline
 
Join Date: Mar 2002
Location: Des Moines, IA (USA)
Posts: 15,776
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

Quote:
Originally Posted by KirbyDE
No they don't, and I never said they do.
Just wanted to point out that it is possible, although I guess nobody would ever even think of proofing their Code.
This is only being done for really important software like some parts or Aircraft Firmware.
Kirby, relax. I was just trying to get your goat since you got as late start in here.

Read a few posts down.
  #122  
Old 08-02-2005, 12:33 PM
The Geek's Avatar
The Geek The Geek is offline
 
Join Date: Sep 2003
Location: Behind you
Posts: 2,779
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

The point is that only mod authors look at the code to modify.

jelsoft encourages people to modify and create macks - therefore the importance of a standard for them. Mod authors (free ones especially) have the right to not give a toss if others want to modify their modifications thereby making the coding style argument moot.

As for enforcing a code style so its easy to debug your own code... who cares as by the time it hits the proverbial shelf, it shouldnt need debugging anymore anyway.

Im not overly bothered to spend an extra day respacing, tabbing and tweaking my code so someone can give me a 'good boy' sticker. I would rather move on to something else that would benefit my users or the vb community.

If it works as intended then who cares where the freaking curley braces are?
  #123  
Old 08-02-2005, 12:34 PM
akanevsky akanevsky is offline
 
Join Date: Apr 2005
Posts: 3,972
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

Quote:
I was only kidding Kirby in that remark. Kirby does excellent work and his stuff always works.
I know Esspecially the HowTo's

Quote:
But to say that every hack/mod you do ALWAYS works the first time, is not only arrogant, but false. If it hasn't happened yet, it will, you can be sure.
I did not say that it ALWAYS works. Show me where I said "ALWAYS"... Thanks.
  #124  
Old 08-02-2005, 12:35 PM
The Geek's Avatar
The Geek The Geek is offline
 
Join Date: Sep 2003
Location: Behind you
Posts: 2,779
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

ooo better idea!!!!

Have the 'QA' team simply reformat the code to the correct style!

That accomplishes everyones needs:
1- The QA people have macks that are standardized
2- The authors can move on to far more important issues like making more macks.

Whoot!
  #125  
Old 08-02-2005, 12:36 PM
yoyoyoyo's Avatar
yoyoyoyo yoyoyoyo is offline
 
Join Date: Dec 2004
Location: USA
Posts: 1,612
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

Quote:
Originally Posted by Dark Visor
Come on, what kind of language is that... You are a moderator. Set a good example...
Quote:
Originally Posted by Dark Visor
Shut the f*ck up and stop trying to say that I said what I didn't say. Thanks.
Pot <--> kettle
  #126  
Old 08-02-2005, 12:36 PM
akanevsky akanevsky is offline
 
Join Date: Apr 2005
Posts: 3,972
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

Quote:
f it works as intended then who cares where the freaking curley braces are?
I do... When the "freaking curley braces" are where they shouldn't be, I cannot read the code and therefore I cannot be sure that you did not put any spyware or viruses into it.

Quote:
Pot <--> kettle
lol yea. If a moderator may, then so may I ...

Quote:
ooo better idea!!!!

Have the 'QA' team simply reformat the code to the correct style!

That accomplishes everyones needs:
1- The QA people have macks that are standardized
2- The authors can move on to far more important issues like making more macks.

Whoot!
That'd be a no, because QA does not have that much time..
  #127  
Old 08-02-2005, 12:37 PM
Andreas's Avatar
Andreas Andreas is offline
 
Join Date: Jan 2004
Location: Germany
Posts: 6,863
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

Quote:
Originally Posted by Dark Visor
The coders will. I'd like you to recall the coding style of vBulletin 2 and compare it with vBulletin 3. Which one is easier to modify? This:

PHP Code:
        if ($a) {
            while (
$b $c)
                {
            echo 
"!!!!!";    }
    }
                else { print 
"yo";
    } 
or this

PHP Code:
if ($a
 {
     while (
$b $c)
     {
         echo 
"!!!!!";
     }
 }
 else
 {
     print 
"yo";
 } 
Obviously the 2nd one, as it is easier to read.
However, as already said, you could just use a prettyprinter to transform Code #1.
  #128  
Old 08-02-2005, 12:38 PM
calorie calorie is offline
 
Join Date: May 2003
Posts: 2,804
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

Quote:
Originally Posted by KirbyDE
The 2nd one complies with vBulletin coding standards.
Yep, it does, but if error reporting is E_ALL only...

The point being that X's standards may not be Y's standards may not be Z's standards. If a hack doesn't pose a security risk, let style be as it may. If a hack does pose a security risk, see this.
  #129  
Old 08-02-2005, 12:38 PM
yoyoyoyo's Avatar
yoyoyoyo yoyoyoyo is offline
 
Join Date: Dec 2004
Location: USA
Posts: 1,612
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

Quote:
Originally Posted by Dark Visor
I do... When the "freaking curley braces" are where they shouldn't be, I cannot read the code and therefore I cannot be sure that you did not put any spyware or viruses into it.
better notify symantec about the curley brace virus!
  #130  
Old 08-02-2005, 12:38 PM
sabret00the's Avatar
sabret00the sabret00the is offline
 
Join Date: Jan 2003
Location: London
Posts: 5,268
Благодарил(а): 0 раз(а)
Поблагодарили: 0 раз(а) в 0 сообщениях
Default

Quote:
Originally Posted by yoyoyoyo
Pot <--> kettle
echo'd thoughts.
Closed Thread


Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

BB code is On
Smilies are On
[IMG] code is On
HTML code is Off

Forum Jump


All times are GMT. The time now is 08:00 PM.


Powered by vBulletin® Version 3.8.12 by vBS
Copyright ©2000 - 2024, vBulletin Solutions Inc.
X vBulletin 3.8.12 by vBS Debug Information
  • Page Generation 0.04591 seconds
  • Memory Usage 2,268KB
  • Queries Executed 11 (?)
More Information
Template Usage:
  • (1)SHOWTHREAD
  • (1)ad_footer_end
  • (1)ad_footer_start
  • (1)ad_header_end
  • (1)ad_header_logo
  • (1)ad_navbar_below
  • (1)ad_showthread_beforeqr
  • (1)ad_showthread_firstpost
  • (1)ad_showthread_firstpost_sig
  • (1)ad_showthread_firstpost_start
  • (2)bbcode_php
  • (12)bbcode_quote
  • (1)footer
  • (1)forumjump
  • (1)forumrules
  • (1)gobutton
  • (1)header
  • (1)headinclude
  • (1)navbar
  • (3)navbar_link
  • (120)option
  • (1)pagenav
  • (1)pagenav_curpage
  • (4)pagenav_pagelink
  • (1)pagenav_pagelinkrel
  • (10)post_thanks_box
  • (10)post_thanks_button
  • (1)post_thanks_javascript
  • (1)post_thanks_navbar_search
  • (10)post_thanks_postbit_info
  • (10)postbit
  • (10)postbit_onlinestatus
  • (10)postbit_wrapper
  • (1)spacer_close
  • (1)spacer_open
  • (1)tagbit_wrapper 

Phrase Groups Available:
  • global
  • inlinemod
  • postbit
  • posting
  • reputationlevel
  • showthread
Included Files:
  • ./showthread.php
  • ./global.php
  • ./includes/init.php
  • ./includes/class_core.php
  • ./includes/config.php
  • ./includes/functions.php
  • ./includes/class_hook.php
  • ./includes/modsystem_functions.php
  • ./includes/functions_bigthree.php
  • ./includes/class_postbit.php
  • ./includes/class_bbcode.php
  • ./includes/functions_reputation.php
  • ./includes/functions_post_thanks.php 

Hooks Called:
  • init_startup
  • init_startup_session_setup_start
  • init_startup_session_setup_complete
  • cache_permissions
  • fetch_threadinfo_query
  • fetch_threadinfo
  • fetch_foruminfo
  • style_fetch
  • cache_templates
  • global_start
  • parse_templates
  • global_setup_complete
  • showthread_start
  • showthread_getinfo
  • forumjump
  • showthread_post_start
  • showthread_query_postids
  • showthread_query
  • bbcode_fetch_tags
  • bbcode_create
  • showthread_postbit_create
  • postbit_factory
  • postbit_display_start
  • post_thanks_function_post_thanks_off_start
  • post_thanks_function_post_thanks_off_end
  • post_thanks_function_fetch_thanks_start
  • post_thanks_function_fetch_thanks_end
  • post_thanks_function_thanked_already_start
  • post_thanks_function_thanked_already_end
  • fetch_musername
  • postbit_imicons
  • bbcode_parse_start
  • bbcode_parse_complete_precache
  • bbcode_parse_complete
  • postbit_display_complete
  • post_thanks_function_can_thank_this_post_start
  • pagenav_page
  • pagenav_complete
  • tag_fetchbit_complete
  • forumrules
  • navbits
  • navbits_complete
  • showthread_complete