Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Servo's code should be beautiful #130

Closed
brson opened this issue Oct 18, 2012 · 2 comments
Closed

Servo's code should be beautiful #130

brson opened this issue Oct 18, 2012 · 2 comments
Labels

Comments

@brson
Copy link
Contributor

@brson brson commented Oct 18, 2012

Servo is Rust's showpiece. The maintainability of Servo reflects very strongly on Rust's own design, and its claims about 'programming in the large'.

Additionally, browser engines are notoriously complex and difficult to contribute to for various reasons. Surely many developers could be tempted to contribute to a browser engine with a modern design, and in a potentially much more pleasant language.

When people look at Servo's code they should like what they see and want to participate.

@brson
Copy link
Contributor Author

@brson brson commented Oct 18, 2012

Some general goals:

  • Settle on conventions for library naming, binding organization, naming (all kinds), actor and object patterns; document and use them consistently
  • Arrange modules consistently
  • Identify patterns and abstract them out, into std where applicable
  • Order module contents top down so that they can be easily read (I think we mostly do this in servo)
  • Document modules

The code should be clean enough that, reading from main, one can easily learn the general structure and behavior of the entire system and the major actors.

ChrisParis pushed a commit to ChrisParis/servo that referenced this issue Sep 7, 2014
shadow-dom: Reorganize the test structure
@nox
Copy link
Member

@nox nox commented Oct 7, 2015

Old meta-issue with no real book-keeping, we can probably close this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.