/* Topic des codeurs [7] */

The Uploader · Le 20/04/2012, à 22:34

Aussi (Gauge) :

commit d62629e92c81726c04799f1bc30d9a6766908e1c (HEAD, refs/remotes/origin/master, refs/heads/master)
Author: Maximilien Noal <mnoal@epsi.fr>
Date:   Fri Apr 20 19:31:55 2012 +0200

    attr_reader :depth not needed
    
    Signed-off-by: Maximilien Noal <mnoal@epsi.fr>
---
 lib/menu.rb         |    1 -
 lib/mouse_sprite.rb |    1 -
 2 files changed, 2 deletions(-)

diff --git a/lib/menu.rb b/lib/menu.rb
index 3195cfa..19a1c32 100644
--- a/lib/menu.rb
+++ b/lib/menu.rb
@@ -4,7 +4,6 @@ require 'events'
 module Gauge
   class Menu
     include ListeningSprite
-        attr_reader :depth
         def initialize(game, bg_image)
             super(game, bg_image,[Gauge::Events::Update])
             @depth = 1
diff --git a/lib/mouse_sprite.rb b/lib/mouse_sprite.rb
index c607fe3..2a2f4e2 100644
--- a/lib/mouse_sprite.rb
+++ b/lib/mouse_sprite.rb
@@ -4,7 +4,6 @@ require 'events'
 module Gauge
     class MouseSprite
         include ListeningSprite
-        attr_reader :depth
         def initialize(game)
             super(game, "./data/mouse.png", [Rubygame::Events::MouseMoved])
             @depth = 0

commit dd195b907d57a411f95a0d3a2517a6710ce4c70d
Author: Maximilien Noal <mnoal@epsi.fr>
Date:   Fri Apr 20 19:24:12 2012 +0200

    Started use of UpdateDepthSortSpritesGroup
    
    Signed-off-by: Maximilien Noal <mnoal@epsi.fr>
---
 lib/menu.rb         |    9 +++++----
 lib/mouse_sprite.rb |    3 ++-
 main.rb             |    8 +++++---
 3 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/lib/menu.rb b/lib/menu.rb
index 45020d5..3195cfa 100644
--- a/lib/menu.rb
+++ b/lib/menu.rb
@@ -4,13 +4,14 @@ require 'events'
 module Gauge
   class Menu
     include ListeningSprite
-        def initialize(game)
-            super(game, "./data/start_menu.png", [Gauge::Events::Update])
-      self.depth = 1
+        attr_reader :depth
+        def initialize(game, bg_image)
+            super(game, bg_image,[Gauge::Events::Update])
+            @depth = 1
         end
 
         def update
             
         end
   end
-end
\ No newline at end of file
+end
diff --git a/lib/mouse_sprite.rb b/lib/mouse_sprite.rb
index 325e8ed..c607fe3 100644
--- a/lib/mouse_sprite.rb
+++ b/lib/mouse_sprite.rb
@@ -4,9 +4,10 @@ require 'events'
 module Gauge
     class MouseSprite
         include ListeningSprite
+        attr_reader :depth
         def initialize(game)
             super(game, "./data/mouse.png", [Rubygame::Events::MouseMoved])
-      self.depth = 0
+            @depth = 0
         end
 
         def mousemoved
diff --git a/main.rb b/main.rb
index 33b2a53..6ef2198 100755
--- a/main.rb
+++ b/main.rb
@@ -1,9 +1,11 @@
 require "rubygame"
 $LOAD_PATH << './lib'
 require 'mouse_sprite'
+require 'menu'
 require 'observer_queue'
 require 'listener'
 require 'events'
+require 'update_depth_sort_sprites_group'
 
 module Gauge
     class Game
@@ -27,9 +29,9 @@ module Gauge
             @clock.target_framerate = 20
             @clock.calibrate
             @clock.enable_tick_events
-            @sprites = Rubygame::Sprites::Group.new
-            Rubygame::Sprites::UpdateGroup.extend_object @sprites
-            @sprites << MouseSprite.new(self)
+            @sprites = Gauge::UpdateDepthSortSpritesGroup.new
+            @sprites.push(MouseSprite.new(self))
+            @sprites.push(Menu.new(self, "data/bg_start_menu.png"))
             initialize_listener(self)
             subscribe(Gauge::Events::DrawMe)
             @sprites.draw(@screen)

commit ec6212a12c60269a1c7056cc4bd60cc021fb88ab
Author: Maximilien Noal <mnoal@epsi.fr>
Date:   Fri Apr 20 18:46:20 2012 +0200

    Removed test/unit requirement
    
    Signed-off-by: Maximilien Noal <mnoal@epsi.fr>
---
 main.rb |    1 -
 1 file changed, 1 deletion(-)

diff --git a/main.rb b/main.rb
index 422e6b4..33b2a53 100755
--- a/main.rb
+++ b/main.rb
@@ -1,5 +1,4 @@
 require "rubygame"
-require "test/unit"
 $LOAD_PATH << './lib'
 require 'mouse_sprite'
 require 'observer_queue'

commit 935452294e2fa7fb39b4855bc1e58cf616c62956
Author: Maximilien Noal <mnoal@epsi.fr>
Date:   Fri Apr 20 18:44:25 2012 +0200

    Use self.send instead of eval (faster/more secure)
    
    Signed-off-by: Maximilien Noal <mnoal@epsi.fr>
---
 lib/listener.rb |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/listener.rb b/lib/listener.rb
index 1abd624..2f1bae3 100644
--- a/lib/listener.rb
+++ b/lib/listener.rb
@@ -13,7 +13,7 @@ module Gauge
 
         def push(event)
             @event = event
-            eval(event.class.name.downcase.partition(":<")[0].partition("::")[2].partition("::")[2])
+            self.send(event.class.name.downcase.partition(":<")[0].partition("::")[2].partition("::")[2])
         end
     end
 end

Codé dans le train.

Rolinh · Le 22/04/2012, à 15:59

Plop,

bon, j'ai sexifié l'output html. J'aimerais des avis critiques si possibles.
En couleur: dfc.html
Sans couleur: dfc.html

Elzen · Le 22/04/2012, à 16:19

C'est assez large (trop pour mon écran ) mais c'est chouette.
Edit : hmm, ceci dit : tu peux peut-être virer l'effet de hover sur la première et sur la dernière ligne. Il est vachement cool sur les autres, mais sur celles-là, c'est peut-être un peu de trop. Enfin, ç'n'est qu'une idée comme ça ^^

Tiens, j'viens de l'installer (par les dépôts ) et j'ai une petite demande de fonctionnalité…

Depuis quelques temps, ma partition racine, je n'sais pas pourquoi, apparaît montée par son UUID dans le /etc/mtab (enfin, /proc/mounts, vu que /etc/mtab n'est plus qu'un lien symbolique).

Du coup, le df d'origine m'affiche son UUID sur la sortie standard, et c'est assez moche (ça fait des lignes plus longues que le nombre de colonnes de mon terminal, donc retour à la ligne et tout, boaf).

Apparemment, le tien fait un peu plus gaffe au nombre de colonnes, vu que l'UUID est tronquée, mais ça reste quand même affiché par l'UUID. Si tu pouvais mettre une option qui permette de forcer l'affichage à l'adresse du périph (/dev/sdXY) plutôt que l'UUID, (et éventuellement, pour ceux qui aient l'avis inverse du mien, une option qui permette de forcer l'affichage de l'UUID plutôt que l'adresse du périph), ça pourrait être assez cool

En tout cas, adopté en ce qui me concerne

Dernière modification par ArkSeth (Le 22/04/2012, à 16:22)

tshirtman · Le 22/04/2012, à 16:41

la version en couleurs est un peu criarde ^^ ça fait jamaique aussi mais j'aime bien

Rolinh · Le 22/04/2012, à 17:12

ArkSeth a écrit :

C'est assez large (trop pour mon écran ) mais c'est chouette.
Edit : hmm, ceci dit : tu peux peut-être virer l'effet de hover sur la première et sur la dernière ligne. Il est vachement cool sur les autres, mais sur celles-là, c'est peut-être un peu de trop. Enfin, ç'n'est qu'une idée comme ça ^^

Ouais, tu as raison pour l'effet de hover. Merci.
En ce qui concerne la largeur, ben ça dépend des options choisies. Dans mon exemple, j'ai tout mis.
Je me suis cassé la tête pour que la largeur s'adapte dans le terminal mais pour la sortie HTML ben... c'est quand même vachement moins dérangeant (suffit de scroller horizontalement si l'écran est trop étroit au pire).

ArkSeth a écrit :

Tiens, j'viens de l'installer (par les dépôts )

Debian sid?

ArkSeth a écrit :

Si tu pouvais mettre une option qui permette de forcer l'affichage à l'adresse du périph (/dev/sdXY) plutôt que l'UUID, (et éventuellement, pour ceux qui aient l'avis inverse du mien, une option qui permette de forcer l'affichage de l'UUID plutôt que l'adresse du périph), ça pourrait être assez cool

C'est curieux ça. C'est vrai que c'est une option qui pourrait s'avérer utile. Pourquoi pas l'implémenter alors mais pour la prochaine version (hypothétique 3.1.0).
Enfin, plutôt qu'une option pour l'un et une option pour l'autre, je verrais plutôt une option qui prenne un paramètre (uuid, label, standard).

ArkSeth a écrit :

En tout cas, adopté en ce qui me concerne

@tshirtman: c'est vrai que c'est criard. Je suis en train de modifier le fichier de configuration afin que l'on puisse choisir des couleurs différentes pour l'output HTML que pour l'output console et du coup, je vais en mettre d'autre par défaut (utile quand il n'y a pas de fichier de configuration).

EDIT: voilà, j'ai mis à jour en tenant compte de vos remarques. Je pense que c'est mieux.

Dernière modification par Rolinh (Le 22/04/2012, à 17:39)

xapantu · Le 22/04/2012, à 23:00

C'est joli Et surtout très lisible. Pareil que ArkSeth, c'est un peu grand pour mon écran aussi, il faudrait peut-être réduire la taille des caractères, mais je suis pas sûr que tu gagnerais beaucoup en largeur, et ça deviendrait probablement illisible ^^.

Par contre, ça serait peut-être intéressant d'avoir la date quelque part.

Elzen · Le 22/04/2012, à 23:20

Rolinh a écrit :

En ce qui concerne la largeur, ben ça dépend des options choisies. Dans mon exemple, j'ai tout mis.

Yep, sans les options de montage, ça passerait ^^

Rolinh a écrit :

Debian sid?

Ouaip

Rolinh a écrit :

Enfin, plutôt qu'une option pour l'un et une option pour l'autre, je verrais plutôt une option qui prenne un paramètre (uuid, label, standard).

Yep, en effet, meilleur choix ^^

Rolinh a écrit :

EDIT: voilà, j'ai mis à jour en tenant compte de vos remarques. Je pense que c'est mieux.

Yep, par contre pour le hover, tu as modifié sur la version en couleur, mais pas sur l'autre, apparemment, c'est voulu ?

Rolinh · Le 23/04/2012, à 08:40

xapantu a écrit :

C'est joli

Merci. Je n'ai pourtant pas vraiment de talent en ce qui concerne le CSS.

xapantu a écrit :

Et surtout très lisible. Pareil que ArkSeth, c'est un peu grand pour mon écran aussi, il faudrait peut-être réduire la taille des caractères, mais je suis pas sûr que tu gagnerais beaucoup en largeur, et ça deviendrait probablement illisible ^^.

Oui, je pense aussi.

xapantu a écrit :

Par contre, ça serait peut-être intéressant d'avoir la date quelque part.

Excellente suggestion. C'est implémenté maintenant.

@ArkSeth: en fait, ça modifie évidemment les deux. Je n'avais simplement pas régénéré la version sans couleur.

Dernière modification par Rolinh (Le 23/04/2012, à 08:41)

Rolinh · Le 25/04/2012, à 01:07

Bon, le switch vers cmake me semble bon.
Ça m'aura fait apprendre des choses en tout cas.

Steap · Le 25/04/2012, à 02:53

Rolinh a écrit :

Bon, le switch vers cmake me semble bon.
Ça m'aura fait apprendre des choses en tout cas.

En l'état, est-ce que ton cmake prend en compte les variables d'environnement ? J'ai dû faire ça :

--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -74,6 +74,9 @@ add_definitions(
 set(CMAKE_BUILD_TYPE RELEASE)
 
 # CFLAGS
+SET(CMAKE_C_COMPILER $ENV{CC})
+SET(CMAKE_C_FLAGS $ENV{CFLAGS})
+SET(CMAKE_EXE_LINKER_FLAGS $ENV{LDFLAGS})
 add_definitions(-O2 -std=c89)
 
 if(CMAKE_COMPILER_IS_GNUCC

Pour que CC, CFLAGS et LDFLAGS soient pris en compte.

Rolinh · Le 25/04/2012, à 07:45

Ah? Bizarre... Je ne suis pas encore assez familier avec CMake pour te répondre mais je me pencherais sur ce problème.

Rolinh · Le 26/04/2012, à 19:50

@Steap: je n'ai pas ce problème sur mon FreeBSD 9. En revanche, gettext me faire toujours ch***... Je ne comprend pas pourquoi cmake ne s'occupe pas d'ajouter le -lintl aux LDFLAGS.

grim7reaper · Le 27/04/2012, à 01:55

@Rolinh : déjà il sort d’où ton GettextTranslate (ça n’a pas l’air d’être standard dans la mesure où il est pas sûr ma machine) ? Pourquoi tu n’utilises pas FindGettext ?
Bon par contre c’est bizarre, le FindGettext sur ma machine ne définit pas GETTEXT_LIBRARIES (par contre celui-là semble plus complet)

Dernière modification par grim7reaper (Le 27/04/2012, à 01:57)

Rolinh · Le 27/04/2012, à 09:22

@grim7reaper: d'où je ne me rappelle plus mais c'était le premier truc que j'avais trouvé en faisant des recherches (il est là).
Par la suite, j'ai vu qu'il existait officiellement FindGettext et j'ai donc voulu l'utiliser. Le problème, comme tu dis, c'est qu'il n'a pas l'air de définir correctement les bibliothèques.
J'avais mis ceci dans mon if (NLS_ENABLED):

find_package (Gettext REQUIRED)
if (GETTEXT_FOUND)
      include_directories(${GETTEXT_INCLUDE_DIRS})
else()
    MESSAGE(FATAL_ERROR "Gettext is required when you set NLS_ENABLED to true")
endif()

Et puis plus loin le target_link_librairies mais sans succès sous FreeBSD.

xapantu · Le 27/04/2012, à 09:52

Un peu hors sujet, mais à propos de gettext, j'avais fais un module pour générer les .pot si ça vous intéresse

tshirtman · Le 27/04/2012, à 14:16

irb(main):001:0> 1.0 / 0
=> Infinity
irb(main):002:0> 1 / 0
ZeroDivisionError: divided by 0
    from (irb):2:in `/'
    from (irb):2
irb(main):003:0> 1 / 0.0
=> Infinity
irb(main):004=0:0> 0 / 0.0
=> NaN

J'ai été un peu surpris par ça en ruby…

mais on m'a dit que c'était pareil en C http://users.tkk.fi/jhi/infnan.html ça m'étonnes mais j'ai un peu la flemme de regarder si c'est dans la spec ^^.

Dernière modification par tshirtman (Le 27/04/2012, à 14:17)

The Uploader · Le 27/04/2012, à 14:23

C'est clairement pas normal, j'pense.

xapantu · Le 27/04/2012, à 14:37

En C, avec tcc (c'était l'occasion d'essayer de truc, c'est super rapide ) :

int main() {
printf("%f %f %f\n", 1/0.0, 1.0/0, 1.0/0.0);
}

inf inf inf

Par contre:

int main() {
printf("%f %f %f %f\n", 1/0, 1/0.0, 1.0/0, 1.0/0.0);
}

Exception en point flottant

grim7reaper · Le 27/04/2012, à 15:26

Ha berk, le main invalide

Sinon, pour le coup de Ruby, ça pourrait s’expliquer par le fait qu’une division entre flottant renvoie un flottant donc tu peux renvoyer inf mais qu’une division entière renvoie un entier et donc tu peux pas renvoyer inf.

Sinon en C, diviser par zéro (par 0, pour 0.0 je suis pas sûr c’est un peu plus compliqué il me semble) c’est indéfini donc le compilateur fait ce qu’il veut, y compris te faire sortir des démons du pif si ça lui chante (référence), d’où la différence de comportement entre gcc et tcc.

Dernière modification par grim7reaper (Le 27/04/2012, à 15:50)

Pylades · Le 27/04/2012, à 17:47

Heu… 0.0 ça peut désigner une très petite valeur, non ? Donc renvoyer inf ne me paraît pas délirant…

xapantu · Le 27/04/2012, à 17:54

Par contre, le 1.0/0, ça l'est un peu plus

Pylades · Le 27/04/2012, à 19:08

Bah ton 0 est promu en double, donc normal.

xapantu · Le 27/04/2012, à 22:39

Ah, oui, c'est vrai

The Uploader · Le 27/04/2012, à 23:00

Truc qui m'a fait sourire:

Xfce blog a écrit :

Techie TODO
In no particular order.
Start blogging again.
Suck less at Javascript, even if it’s a shitty language.
Learn jQuery, even if it’s just a library to make a shitty language less shitty.
Learn Rails properly.
Get back into open source dev.
Find a project/idea I can potentially monetize, and build and launch it.
Throw out my website entirely and start from scratch.
Stop running MacOSX all the time on my laptop and get back to using Linux as my primary desktop OS.
tags: everything, programming, tech
posted in Software by Brian Tarricone

(c'est l'un des core developers)

D'ailleurs Xfce 4.10 sort demain. \o/

PS : Haha chez MS

Dernière modification par The Uploader (Le 27/04/2012, à 23:03)

tshirtman · Le 28/04/2012, à 01:47

grim7reaper a écrit :

Sinon en C, diviser par zéro (par 0, pour 0.0 je suis pas sûr c’est un peu plus compliqué il me semble) c’est indéfini donc le compilateur fait ce qu’il veut, y compris te faire sortir des démons du pif si ça lui chante (référence), d’où la différence de comportement entre gcc et tcc.

C'est ce que je me demandais merci

Ubuntu-fr

Navigation

Liens de recherche

Annonce

#651 Le 20/04/2012, à 22:34

Re : /* Topic des codeurs [7] */

#652 Le 22/04/2012, à 15:59

Re : /* Topic des codeurs [7] */

#653 Le 22/04/2012, à 16:19

Re : /* Topic des codeurs [7] */

#654 Le 22/04/2012, à 16:41

Re : /* Topic des codeurs [7] */

#655 Le 22/04/2012, à 17:12

Re : /* Topic des codeurs [7] */

#656 Le 22/04/2012, à 23:00

Re : /* Topic des codeurs [7] */

#657 Le 22/04/2012, à 23:20

Re : /* Topic des codeurs [7] */

#658 Le 23/04/2012, à 08:40

Re : /* Topic des codeurs [7] */

#659 Le 25/04/2012, à 01:07

Re : /* Topic des codeurs [7] */

#660 Le 25/04/2012, à 02:53

Re : /* Topic des codeurs [7] */

#661 Le 25/04/2012, à 07:45

Re : /* Topic des codeurs [7] */

#662 Le 26/04/2012, à 19:50

Re : /* Topic des codeurs [7] */

#663 Le 27/04/2012, à 01:55

Re : /* Topic des codeurs [7] */

#664 Le 27/04/2012, à 09:22

Re : /* Topic des codeurs [7] */

#665 Le 27/04/2012, à 09:52

Re : /* Topic des codeurs [7] */

#666 Le 27/04/2012, à 14:16

Re : /* Topic des codeurs [7] */

#667 Le 27/04/2012, à 14:23

Re : /* Topic des codeurs [7] */

#668 Le 27/04/2012, à 14:37

Re : /* Topic des codeurs [7] */

#669 Le 27/04/2012, à 15:26

Re : /* Topic des codeurs [7] */

#670 Le 27/04/2012, à 17:47

Re : /* Topic des codeurs [7] */

#671 Le 27/04/2012, à 17:54

Re : /* Topic des codeurs [7] */

#672 Le 27/04/2012, à 19:08

Re : /* Topic des codeurs [7] */

#673 Le 27/04/2012, à 22:39

Re : /* Topic des codeurs [7] */

#674 Le 27/04/2012, à 23:00

Re : /* Topic des codeurs [7] */

#675 Le 28/04/2012, à 01:47

Re : /* Topic des codeurs [7] */

Pied de page des forums