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

BaseJob and Super Nono #324

Closed
Atemo opened this issue Mar 4, 2015 · 2 comments
Closed

BaseJob and Super Nono #324

Atemo opened this issue Mar 4, 2015 · 2 comments
Labels
component:core A fault that lies within the main framework of rAthena

Comments

@Atemo
Copy link
Contributor

Atemo commented Mar 4, 2015

BaseJob NPC script command return the default value -1 for Super novice, super baby, super novice ex super baby ex class. I think in map.h

MAPID_SUPER_NOVICE = JOBL_SUPER_NOVICE|JOBL_2_1|0x0,
MAPID_SUPER_BABY = JOBL_SUPER_NOVICE|JOBL_BABY|JOBL_2_1|0x0,
MAPID_SUPER_NOVICE_E = JOBL_SUPER_NOVICE|JOBL_THIRD|JOBL_2_1|0x0,
MAPID_SUPER_BABY_E = JOBL_SUPER_NOVICE|JOBL_THIRD|JOBL_BABY|JOBL_2_1|0x0,

should be

MAPID_SUPER_NOVICE = JOBL_2_1|0x0,
MAPID_SUPER_BABY = JOBL_BABY|JOBL_2_1|0x0,
MAPID_SUPER_NOVICE_E = JOBL_THIRD|JOBL_2_1|0x0,
MAPID_SUPER_BABY_E = JOBL_THIRD|JOBL_BABY|JOBL_2_1|0x0,

@aleos89 aleos89 added the component:core A fault that lies within the main framework of rAthena label Mar 4, 2015
@cydh
Copy link
Contributor

cydh commented Mar 5, 2015

the idea was we can do fast lookup for Sunov using bitwise &JOBL_SUPER_NOVICE
even it's MAPID_SUPER_NOVICE = JOBL_2_1|0x0
see on pc_readparam for SP_BASEJOB,

        case SP_BASEJOB:         val = pc_mapid2jobid(sd->class_&MAPID_UPPERMASK, sd->status.sex); break; //Base job, extracting upper type.

it masks every Sunov to 0x100. list not found on pc_mapid2jobid, tell me it's been like that since long time ago.

then what's BasJob of Super Novices? Novice? like this
https://gist.github.com/cydh/541bf46ba9d855e6835e

@Atemo
Copy link
Contributor Author

Atemo commented Mar 5, 2015

Ha I get it

then what's BasJob of Super Novices? Novice? like this

The BaseJob should be Super nono for the 4 Super nono
JOB_SUPER_NOVICE

https://github.com/rathena/rathena/blob/master/src/map/map.h#L131

//Super Novices are considered the 2-1 version of the novice! Novices are considered a first class type, too...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:core A fault that lies within the main framework of rAthena
Projects
None yet
Development

No branches or pull requests

3 participants