Giter VIP home page Giter VIP logo

Comments (10)

andgineer avatar andgineer commented on August 16, 2024

Well no preferences in code formatting from my side.

We better choose some standard way for Pascal but I am not familiar with any.

from tregexpr.

Alexey-T avatar Alexey-T commented on August 16, 2024

I suggested the standard way... in Lazarus there is also internal code Formatter but I suggested manual mass replace anyway

from tregexpr.

andgineer avatar andgineer commented on August 16, 2024

Ok but I suggest to add link to some code style so we would have some rules, and not just personal opinion.

from tregexpr.

Alexey-T avatar Alexey-T commented on August 16, 2024

Lazarus code
https://github.com/graemeg/lazarus/tree/upstream/ide
I opened random .pas/.pp files, and all have

  • no spaces before ( and [,
  • no space before :,
  • but := is surrounded by 2 spaces or 0 spaces (mostly with 2 spaces),
  • var, const are in lowercase,
  • function, procedure, constructor in lowercase,
  • exit in different case (seems mostly lower).

from tregexpr.

andgineer avatar andgineer commented on August 16, 2024

I meant we need a link to some DOCUMENT with description what code style we want in the library.

You see code style often is a hot question with a lot of debates around.
So if we can say - just read this rules, that could prevent all this fruitless discussions.

from tregexpr.

andgineer avatar andgineer commented on August 16, 2024

If you do not know such a resource may be place on github wiki very short description of the rules you want to have in the code

from tregexpr.

Alexey-T avatar Alexey-T commented on August 16, 2024

I may write this doc.
Or we can use Embarcadero Formatter.exe (it's in Delphi community edition, free).
http://docwiki.embarcadero.com/RADStudio/Rio/en/Formatter.EXE,_the_Command_Line_Formatter

Ok to format with Formatter.exe?

from tregexpr.

Alexey-T avatar Alexey-T commented on August 16, 2024

I used the Formatter.exe from Delphi Comminity edition 10.3, here is the result, post this file, no need to make a pullreq.
RegExpr-fmt.zip

from tregexpr.

andgineer avatar andgineer commented on August 16, 2024

it's ok

from tregexpr.

Alexey-T avatar Alexey-T commented on August 16, 2024

So post the file to GH

from tregexpr.

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.