Skip to content

Is fuzz directory needed in brotli? #26435

@ghost

Description

@ghost

I am sweeping C++ sources in the repo and found two unused .cc files in fuzz subdirectory under brotli:

https://github.com/dotnet/corefx/tree/b1a9ddd2f64f81b37eb0c737de601e6d7a524cba/src/Native/AnyOS/brotli/fuzz

@ianhays, the script in that directory requires a CMakeLists.txt to be present two directories above (such as https://github.com/xingyun86/ppshuai_brotli/blob/9898e334a9a749fc387048433cdef9643657852a/src/brotli/brotli/CMakeLists.txt), but in CoreFX port that CMakeLists.txt file was removed from brotli's root. Currently there is nothing in corefx that touches this directory. Should it be removed?

Metadata

Metadata

Assignees

No one assigned

    Labels

    area-System.IO.CompressionenhancementProduct code improvement that does NOT require public API changes/additions

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions