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

Quest log in var db/quest-log #511

Closed
wants to merge 8 commits into from

Conversation

omattornot
Copy link

I add the var for new quest-log of letter, orum and fur quest.

@wushin wushin added the test label May 17, 2016
@Helianthella
Copy link
Member

the bitmasking seems wrong..

@wushin
Copy link
Member

wushin commented May 17, 2016

Aye, @omattornot you need to replace the VAR name in the quest with the new var. Look at the other quests or chat with us in IRC.

@@ -28,3 +28,10 @@
20,QL_TERRANITE_ARMOR,QUEST_NorthTulimshar,6,4
// 21 => assassin quest
22,QL_CELESTIA,QUEST_Hurnscald,2,8
23,QL_Caretaker,QUEST_Graveyard_Caretaker,0,4
Copy link
Member

Choose a reason for hiding this comment

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

All QL_ variables should be in ALL CAPS


QUEST_Graveyard_Caretaker should be moved to an existing bitmasked var, and converted in clear_vars. It only needs 3 bits so we can fit it easily somewhere


if (@state >= 6) goto L_Lord_Hideout_Open;
if (@QL_Bandit >= 6) goto L_Lord_Hideout_Open;
Copy link
Member

Choose a reason for hiding this comment

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

remove the @

…on in clear_vars; update and fix some things about comments of last commit
@wushin wushin added this to the v2016.6.6 milestone May 31, 2016
@wushin
Copy link
Member

wushin commented May 31, 2016

@omattornot you need to rebase. Let me know when you fix the conflict so I can restart test-server.

@omattornot
Copy link
Author

@wushin I fixed the conflict. But what rebase i need to do ? Squash all commit in one ?

@wushin
Copy link
Member

wushin commented May 31, 2016

@omattornot the conflict was the rebasing.

@Helianthella
Copy link
Member

@omattornot is this done? if yes I'll review again. @wushin feel free to check too

@pazkero
Copy link
Member

pazkero commented Jan 29, 2022

I'm not going to merge this, it needs a bit more than just a rebase at this point 😰

I'm closing but if you want to fix and try again, please, be my guest. This is by no means a rejection, just spring cleaning.

@pazkero pazkero closed this Jan 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants