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

AISS Proyect Group L2-5 #1205

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

AISS Proyect Group L2-5 #1205

wants to merge 3 commits into from

Conversation

Javclamar
Copy link

Added some lines of code in regards of some potential bugs and syntasix error in jgrapht-core/overview.html and jgrapht-core/GraphType.java.

-Overview.html: added a html title lable and a specification of language
-GraphType.java: made the Interface GraphType Serializable, since we use this Interface as an entry for a lot of methods that requires serializable entrys

Added a title lable in jgrapht-core\src\main\java\overview.html
Added label lang="eng" to especify the language of the html file
Make the "org.jgrapht.GraphType" class "Serializable" as the objects passed to writeObject must implement Serializable
@jsichi
Copy link
Member

jsichi commented Mar 10, 2024

The default implementation class (DefaultGraphType) is already Serializable. In general, for the main graph interfaces, we avoid declaring Serializable, and we put the Serializable marker on the default in-memory implementation classes instead. The reason for doing so is that we want people to be able to implement the graph interfaces in many different ways, and in some of these, (e.g. database-backed graphs, or remote proxies), serializability doesn't make sense.

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.

None yet

2 participants