Page MenuHomeDevCentral
Differential D885

Fix long incoherence for user levels
ClosedPublic
Actions

Authored by dereckson on Jan 29 2017, 23:04.
Tags
None
Referenced Files
F24642687: D885.id2249.diff
Tue, Mar 3, 01:18
F24642024: D885.id2248.diff
Tue, Mar 3, 00:47
F24638194: D885.diff
Mon, Mar 2, 21:12
F24635552: D885.id2247.diff
Mon, Mar 2, 18:47
F24635551: D885.id2248.diff
Mon, Mar 2, 18:47
F24631725: D885.id2249.diff
Mon, Mar 2, 14:56
F24631723: D885.id2247.diff
Mon, Mar 2, 14:56
F24631721: D885.id2248.diff
Mon, Mar 2, 14:56
View All Files
Subscribers
None

Details

Reviewers
dereckson
Commits
rDARKBOTcf2534fee6e3: Fix long incoherence for user levels
Summary

As the user level is declared as a long, we should compare it with
other long values. As such, check_access now returns a long.

Test Plan

Recompile, test level commands

Diff Detail

Repository
rDARKBOT Darkbot
Lint
No Lint Coverage
Unit
No Test Coverage
Branch
long (branched from master)
Build Status
Buildable 1364
Build 1612: arc lint + arc unit

Event Timeline

dereckson created this revision.Jan 29 2017, 23:04
Harbormaster completed remote builds in B1364: Diff 2247.Jan 29 2017, 23:04
dereckson planned changes to this revision.Jan 29 2017, 23:06
dereckson added inline comments.
source/chansrv.c
612

Declare at top instead.

dereckson updated this revision to Diff 2248.Jan 29 2017, 23:51
dereckson marked an inline comment as done.
dereckson edited edge metadata.

Initialize at chanserv_help's top

dereckson accepted this revision.Jan 29 2017, 23:56
This revision is now accepted and ready to land.Jan 29 2017, 23:56
Closed by commit rDARKBOTcf2534fee6e3: Fix long incoherence for user levels (authored by dereckson, committed by dereckson). · Explain WhyJan 29 2017, 23:57
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
source/
23 lines
2 lines
4 lines

Diff 2247

View Options

source/chansrv.c

Loading...
View Options

source/helpers.c

Loading...
View Options

source/prototypes.h

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator