Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Totally steals trams from Paradise ♥ #229

Merged
merged 3 commits into from Jun 4, 2015
Merged

Totally steals trams from Paradise ♥ #229

merged 3 commits into from Jun 4, 2015

Conversation

ghost
Copy link

@ghost ghost commented May 30, 2015

@thomasstoeckert
Copy link
Contributor

Have you tested these things yet?

@scrdest
Copy link
Contributor

scrdest commented May 30, 2015

Neat stuff. I've been worried it'd use shuttle controller (which is getting modified in the Bay merge if memory serves) but nope.avi, so just test it if you haven't.

@ghost
Copy link
Author

ghost commented May 30, 2015

I tested it last weekend and it worked.

@scrdest
Copy link
Contributor

scrdest commented May 30, 2015

Seems fine. Glloyd should be able to review it himself today, considering he mentioned doing events so I presume he'll be on.

@ghost
Copy link
Author

ghost commented May 30, 2015

wzxgpsz

@Glloyd
Copy link
Member

Glloyd commented May 31, 2015

Events and scom are next weekend btw. I'll have Internet by midweek

@scrdest
Copy link
Contributor

scrdest commented May 31, 2015

Yeah, I misread the post, but I assumed everyone here realized I'm a derp.

@scrdest
Copy link
Contributor

scrdest commented May 31, 2015

👍, Glloyd?

@ghost
Copy link
Author

ghost commented Jun 4, 2015

@Glloyd That might help @scrdest


/obj/tram/blob_act()
if(prob(50))
del(src)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

del()

Change to qdel()

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So remove the src?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

qdel(src)

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GLLOYD UPDATE YOUR SHIT ITS 2015 ONLY CASUALS DON'T USE QDEL

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Iamgoofball Still haven't seen it on Bay http://i.imgur.com/EDu2oZb.png :I Also Glloyd only pulls from the normal Bay not dev.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Iamgoofball people have this thing called 'life' and 'being busy' and if you followed the community you'd know both of those mythical, rare events occurred for Glloyd. Update is scheduled this week, IIRC, as a matter of fact.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What are the benefits of qdel over just del?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we don't have qdel yet.

On Thu, Jun 4, 2015 at 7:49 PM, Thomas Stoeckert notifications@github.com
wrote:

In code/modules/tram/tram.dm
#229 (comment)
:

  • switch(severity)
  •   if(1.0)
    
  •       del(src)
    
  •       return
    
  •   if(2.0)
    
  •       if(prob(50))
    
  •           del(src)
    
  •       return
    
  •   if(3.0)
    
  •       if(prob(25))
    
  •           del(src)
    
  •       return
    
    +/obj/tram/blob_act()
  • if(prob(50))
  •   del(src)
    

What are the benefits of qdel over just del?


Reply to this email directly or view it on GitHub
https://github.com/UristMcStation/UristMcStation/pull/229/files#r31777509
.

Graham Lloyd

@Glloyd
Copy link
Member

Glloyd commented Jun 4, 2015

also still no internet, i hate canadians isps

Glloyd added a commit that referenced this pull request Jun 4, 2015
Totally steals trams from Paradise ♥
@Glloyd Glloyd merged commit 9026c4c into UristMcStation:master Jun 4, 2015
@ghost
Copy link
Author

ghost commented Jun 5, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants