Coder Social home page Coder Social logo

Comments (8)

meziantou avatar meziantou commented on June 12, 2024

What would be the difference with Nullable Reference Types?

from meziantou.analyzer.

EngRajabi avatar EngRajabi commented on June 12, 2024

I did not catch. What I mean is to create a rule that warns about the possibility of nulling the variables.

from meziantou.analyzer.

meziantou avatar meziantou commented on June 12, 2024

Can you please provide a better example to describe what you want? Based on the example from your initial message, the compiler already reports the CS8629 warning:

image

from meziantou.analyzer.

EngRajabi avatar EngRajabi commented on June 12, 2024

I tested. With the example of the code I gave above. No warning has been issued.
Of course, Nullable is disabled in my project. But the above code has the possibility of null error.

from meziantou.analyzer.

meziantou avatar meziantou commented on June 12, 2024

Of course, Nullable is disabled in my project

You must enable nullables using <Nullable>enable</Nullable> or #nullable enable to get the warnings.

But this doesn't answer my initial question. Why do you want me to implement a new rule? How is your request different from what the compiler already provides with <Nullable>enable</Nullable>?

from meziantou.analyzer.

EngRajabi avatar EngRajabi commented on June 12, 2024

I don't want to use the Nullable feature both for personal reasons and because the code is old and needs a lot of changes.
There is a new rule request for projects that have Nullable disabled, but still have the possibility of a null error. be corrected
In the code above, it is correct that nullable is inactive, but there is a possibility of a null error that can be notified to the developer with a new rule.

from meziantou.analyzer.

meziantou avatar meziantou commented on June 12, 2024

So, you want something similar to Nullable Reference Types (NRT), but without using NRT.

Meziantou.Analyzer won't implement such rules. There is no need to reimplement something that already exists. If you don't want to use NRT, that your choice, but a solution exists.

from meziantou.analyzer.

EngRajabi avatar EngRajabi commented on June 12, 2024

I checked. You can use this flag if the feature is not active but the warnings are active.

<Nullable>warnings</Nullable>

from meziantou.analyzer.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.