Skip to content

wip: Split fuzz.rs into individual fuzz targets with cargo-fuzz - #38

Open
mozfreddyb wants to merge 2 commits into
servo:mainfrom
mozfreddyb:individual_fuzz_targets
Open

wip: Split fuzz.rs into individual fuzz targets with cargo-fuzz#38
mozfreddyb wants to merge 2 commits into
servo:mainfrom
mozfreddyb:individual_fuzz_targets

Conversation

@mozfreddyb

@mozfreddyb mozfreddyb commented Aug 30, 2019

Copy link
Copy Markdown

I've ommitted the test for tendril.clear() as I fail to understand the risk associated with its implementation.

I'm also ommitting this block, as I'm failing to understand its utility:

            32...47 => {
                let lenstr = format!("[length = {}]", buf_tendril.len());
                buf_string.push_str(&lenstr);
                buf_tendril.push_slice(&lenstr);
                assert_eq!(&*buf_string, &*buf_tendril);
            }

The "[length = ..]" thing does not seem to cause any different behavior. There is no syntax within tendrils that I could find.

NB: As a follow-up I would like to format the whole crate with rustfmt, as it would make it tremendously easier to develop code while rustfmt is running on the whole repo.


This change is Reviewable

@Manishearth

Copy link
Copy Markdown
Member

Note: this repo has been moved to https://github.com/servo/html5ever

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.

2 participants