@solnic I traced the spec failures for sqlite on jruby and found I could fix them locally with this change: https://github.com/datamapper/do/issues/81


@glasser is it the new process? Historically we have been making up history.md before each release. I can add a record, no problem.


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.94](http://teamcity.monogame.net/viewLog.html?buildId=5902&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 462, ignored: 6 Build time: 00:14:13


xargsの`-i`は非推奨(The ‘-i’ option is deprecated)らしいので、`-I{}` に置き換えました。
https://www.gnu.org/software/findutils/manual/html_node/find_html/xargs-options.html

なお、osx yosemite付属のxargsでは`-i`オプション自体がなくなってました…


This is app is not intended for commercial.


Also improved and clarified the custom column name explanation from:

    You can override the default column name if you need to:

to

    You can override the default column name by specifying a custom column name in
    the `counter_cache` declaration instead of `true`. For example, to use
    `count_of_orders` instead of `orders_count`:"


This is a useless commit now, could you revert it?


#1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'CREATE TABLE `users` ( `id` int(11) NOT NULL AUTO_INCREMENT, `companyid` i' at line 9


Is this real? :D


Can you use $(CLOSURE_COMPILER_VERSION) to generate the exact URL? Then it won't be necessary to update the string in two places.


I think I'd rather pass around IKernel than do that funky little cast. We started to do that to begin with, but we really didn't want folks having access to the entire IKernel interface. The Wrapper class ServiceLocator with its IServiceLocator interface gave us only the pieces of the interface we wanted, but adds abstraction. IKernel eliminates abstraction but exposes whole interface. Your M.O., James, tends to favor the latter. Thoughts?


where is this file for delete??


I saif am sorry i do not feel the same way about you,you need to grow up,you are a good looking girl many guys   will date you,where there is no trust there is no relation period,I have to stick to my principle or else i will have no principles to live by..I really am sorry i am not asking you to love me you just keep coming around when i have no slight interest.I can help you tho here is what you do.
- Stop lying to yourself
-  Exceed expectations in every aspect of life
- dehydrate your body so the blood is flowing  steady to avoid negative thoughts...
- Have principles and learn to live by them and you will have no mood swings

Only If you see yourself from eyes you will understand what i am saying,it just wasn't ment to be,now i am asking you i do not wanna interact with you anymore,this will help you move on to a better future.


Regresa pr: $('.preguntasRespuestas') a pr: $('#preguntasRespuestas') este cambio no es de esta rama.
La idea es tener seccionado tu codigo y tus cambios, para que en un futuro tu puedas ver que fue lo que cambiaste especificamente en cada cosa, ahora, en este commit tu rama dice agregando preguntas y el cambio que te estoy señalando es para la rama 6.

Arregla esto, haz checkout [tuBranch], push y esperas a que autorice los cambios


inline comment test


LGTM


wrong title
should be "[FIX bug] models path"


No bashisms


k


Thank you @kcwu I actually updated the code here https://github.com/odeke-em/drive/commit/a659f363f37f44aa7015319fe9c82a4431819a99


LGTM

I ran into the same problem, but didn't look further into it.


Thank you for supporting this. its very appreciated. Im looking for ward to ur windows build. Don't make us wait to long :)


@tmhammer your change got us part of the way, but it didn't cover the case where the date was in the correct format but wasn't an actual date (e.g. 2012-99-99). If we defer the heavy lifting required for this check to `Date.parse`, we can cover the case without too many lines of code.

I'll merge this PR once I've got specs passing (it seems like this check is causing some other specs to fail :-) ).


Nice


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.95](http://teamcity.monogame.net/viewLog.html?buildId=5904&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


ブランチテスト1


Path is fixed in master.


Still need to add customizable directories


this will be quite in-efficient. Try to move this logic to single python function, that accepts a `user` and `comment_id`


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.95](http://teamcity.monogame.net/viewLog.html?buildId=5904&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 462, ignored: 6 Build time: 00:14:06


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.96](http://teamcity.monogame.net/viewLog.html?buildId=5906&buildTypeId=MonoGame_DevelopWin&guest=1) is now running


I did it on devel


Will try it, thanks!


send files to me and i  will add your translation

2015-03-28 5:25 GMT+01:00 Yerdivad <notifications@github.com>:

> I have a spanish translation.
>
> how can i add?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MIUIADRIA/Xiaomi-Apps-Localization/commit/e182bbc84deb1d10c2a745026e9b732e659cc6d3#commitcomment-10448704>
> .
>


I know, but it fails on our internal bamboo and we need to figure out why
before we re-enable.
On 28 Mar 2015 01:15, "Anders Olsen Sandvik" <notifications@github.com>
wrote:

> Why turn it off? I just ran master and everything is passing.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/finn-no/unleash/commit/629535552c07bf09faa0992ab3855413287a52aa#commitcomment-10447837>
> .
>


TeamCity MonoGame :: Develop (Win) [Build 3.4.0.96](http://teamcity.monogame.net/viewLog.html?buildId=5906&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 462, ignored: 6 Build time: 00:14:41


how can i send you? sorry, i cant see your email


See http://vim.wikia.com/wiki/Alternative_tab_navigation for more details.


masva10@gmail.com

2015-03-28 7:51 GMT+01:00 Yerdivad <notifications@github.com>:

> how can i send you? sorry, i cant see your email
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/MIUIADRIA/Xiaomi-Apps-Localization/commit/e182bbc84deb1d10c2a745026e9b732e659cc6d3#commitcomment-10449045>
> .
>


Con: Quality != the best...

Pro: Doesn't destroy your ears!


Yes, this gets rid of that undefined error for kernel_thread_helper.

You also need this:
https://github.com/StefanescuCristian/hammerhead/commit/6e68f6514ea6f17e58180981f368190821bee1ad
On Mar 28, 2015 7:38 AM, "Chang Long" <notifications@github.com> wrote:

> hi @StefanescuCristian <https://github.com/StefanescuCristian>
>
> this fix is for LTO pass ? I can build with disable LTO for process.c ,
> but I guess enable LTO for process.c will make the kernel much better .
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/StefanescuCristian/hammerhead/commit/6a25d2fb6eee607f8fb6ead5c82b8c25f79a6b13#commitcomment-10448908>
> .
>


Because the type does not play any role role for the injectionpoint here. The container will omit further deployment due to the `@Vetoed` annotation. I will replace it with `Object`.


Por que se eliminó una versión?


Shouldnt this rather be "command input" instead of "arguments"


90946736_WSHP_#4567 (Nigusse) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1167 of 1169 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.83%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-69.8%25-green.svg)


` also add `CONFIG_GENERIC_KERNEL_THREAD=y`, but get that error...


Since it is already sorted in ARSound we dont need to do that here.


I have added unit test for 'setMaxRows()' and 'getMaxRows'.
Thanks.


0 is a reserved port, I'm not sure I support additional complexity to allow support for a port that you would never use.


I am gettting an exception here, saying "use of unitialized variable".

What's the overall status? is this working? It would be nice to include some sample tasks with example of command line.


Se necesita iniciar ret en - 1?
Data index debe inicializares con 0 o con - 1?
Estoy en el celular asique solo veo algunas líneas :(


As you said, there is really no advantage using this *method object*. It is a leftover from my previous *failed* PR. I will inline it into the surrounding `InjectionHelper` utility. Thx for the hint :+1:


this works on python3? I didnt manage to install matplotlib on python3, didnt debug it. does it work for you?


Well, Packages-Kde creates also followed warnings:

warning: file-5.22-1 is up to date -- reinstalling
warning: findutils-4.4.2-6 is up to date -- reinstalling
warning: gawk-4.1.1-1 is up to date -- reinstalling
warning: gettext-0.19.4-1 is up to date -- reinstalling
warning: grep-2.21-2 is up to date -- reinstalling
warning: groff-1.22.3-2 is up to date -- reinstalling
warning: gzip-1.6-1 is up to date -- reinstalling
warning: pacman-4.2.1-1 is up to date -- reinstalling
warning: pkg-config-0.28-2 is up to date -- reinstalling
warning: sed-4.2.2-3 is up to date -- reinstalling
warning: sudo-1.8.12-1 is up to date -- reinstalling
warning: texinfo-5.2-3 is up to date -- reinstalling
warning: util-linux-2.25.2-1 is up to date -- reinstalling
warning: which-2.20-7 is up to date -- reinstalling
resolving dependencies...
:: There are 2 providers available for libx264.so=144-64:
:: Repository extra
   1) libx264  2) libx264-10bit

... maybe we should check them also out ...


Yes, you're right. I'll correct this error and tests.


原因を分析してください。


There seems to be a behaviour by our ifup/netconfig scripts, that when you got your IP+DNS from DHCP and then switch to static IP-config, it also wants static DNS-servers, dropping the earlier ones from DHCP.


Exception message can be better.


Add annotation.


Is this really needed?


Denim i find the car more attractive than you l,the car turns me on,I am jealous :+1:


OH, i am so stupid!    I made the same mistake !!!!!!!!!!!!! This dump action was fixed by the next commit.


Add this
https://github.com/StefanescuCristian/hammerhead/commit/c67877a50570df059ebe59d6064f2dcb2a94dbff

Or you didn't cherry-picked 6e6 correctly.
Can you also enable CONFIG_UNUSED_SYMBOLS?

I'll give your sources a look in a hour or so.

also addCONFIG_GENERIC_KERNEL_THREAD=y`, but get that error...

—
Reply to this email directly or view it on GitHub
<https://github.com/StefanescuCristian/hammerhead/commit/6a25d2fb6eee607f8fb6ead5c82b8c25f79a6b13#commitcomment-10449067>
.


It would be nice to document it on https://nixos.org/nixpkgs/manual/#idm140737318982464, assuming the meaning is considered stable.


There are a lot of bugs if try to edit a patient demographics without setting anything up. If try to save, Get alert box that need to place a postal code. If try to save with a postal code get a blank alert box.


It's not working for me now. Which part will trigger "docker run quay.io/coreos/flannel:${FLANNEL_VER}"?


use xls() function here


わたしはどこ


test


use xls() function here. Also need to use echo:
echo xls()


Uninitialized variables (like run_after and run_before) were print out on console even when there was no content of them.


In the instructions, what is the sqlpatch?


thanks, I already add c67877a50570df059ebe59d6064f2dcb2a94dbff.

I do this change it start work:
```diff
 git diff
diff --git a/include/linux/sched.h b/include/linux/sched.h
index 3b2e9b3..cd82f67 100644
--- a/include/linux/sched.h
+++ b/include/linux/sched.h
@@ -2447,9 +2447,7 @@ extern int do_execve(const char *,
                     const char __user * const __user *, struct pt_regs *);
 extern long do_fork(unsigned long, unsigned long, struct pt_regs *, unsigned long, int __user *, int __user *);
 struct task_struct *fork_idle(int);
-#ifdef CONFIG_GENERIC_KERNEL_THREAD
 extern pid_t kernel_thread(int (*fn)(void *), void *arg, unsigned long flags);
-#endif

 extern void set_task_comm(struct task_struct *tsk, char *from);
 extern char *get_task_comm(char *to, struct task_struct *tsk);
```


Is it possible to conditionally set the home_dir based on the target config chosen in some way? I'm guessing the .cerbero/cerbero.cbc is loaded first so it won't work...


That test was something of a marathon! Congrats, although I think some of the points made in the fishcooking discussion do need to be considered (does this simplification only work because our treatment of isolated/backward pawns is missing something?) before merge.

If it is merged I suspect a tuning run on the new combined array is in order.


I agree that this is not a nice place to deal with that problem, but dealing with it in the main script means changing each place where `args['<id>']` is used, so about 8 changes rather than 1.

Otherwise, I think the cleanest way of dealing with this seems to be changing the docopt line for gist delete to something like `gist delete <ids>`, so that the other commands don't need to be changed at all. What do you think, which one would be the best?


I've been trying to do it as I go along lately (at least as a rough draft), as have a few others.  A lot easier than git diving later for random tiny things!


@VishnuDuttSharma
>  If you want, take out the publisher from the loop
> and put it inside the callback.

What do you mean by this?

Note that the subscriber callback has been removed, as you said that it is not required inside these two files. (`testThrusters.py` and `stopKraken.py`)


Interesting!

This means that original projects on Travis got this grandfathered in, whereas new ones are always executed under the slower container infrastructure by default ;)

The reason your latest CI run failed is because of the infrastructure to keep authorship current and correct - you need to add yourself to [the AUTHORS list](https://github.com/dbsrgits/dbix-class/blob/current/blead/AUTHORS), with the name/email whathave you you would like to be seen in [the credits](https://metacpan.org/pod/DBIx::Class#AUTHORS). You can verify everything is kosher by running `prove -l xt/dist/authors.t`

Thanks for playing with this!


Surely it's a sign that someone has Perl'd too much.


90946736_WSHP_#4567 (Nigusse) - i386-Windows-7-VisualStudio-12: OK (1671 of 1677 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.64%25-yellow.svg)


thats handled in the exception


Sorry. copy paste overlook.


0 no one can wear, really ??


What problem are you trying to solve with that?


Jenkins build jobs. Currently I create a new cerbero.cbc for every build job run which means one machine can only do one job at a time. Maybe that's wise but it's a restriction.


I'm not sure how to get this into a format that will play nicely with OpenEMR. Need to think about this some more.


呃我真是太粗心了


I wouldn't do this, since users in other countries may want to use this.


Programming Assignment 1
For this assignment, students will be setting up their BitBucket accounts and pushing the Rails application to a repository on BitBucket. (For instructions on how to use Git to accomplish this, please refer to the posted instructions) For help and more in depth instructions on this assignment, please refer to the Module 2 Lecture 3 Video.
This corresponds to iteration 1 of the blog app in the lecture videos. Please make sure to use the same names used in the videos for all Rails variables, classes, etc.. The grading scheme depends on you following the exact instructions provided in the videos.
Create a BitBucket account You will need to create a BitBucket account at bitbucket.org.
Create the application using Ruby on Rails Create a rails project, using the command. This project should be named . You will need to scaffold tables for posts and comments. The posts table should have the following two columns: of type string and of type text. The comments table will also have two columns as follows: of type integer and  of type text.
Push the application to BitBucket Create a repository on BitBucket (be sure this repository is public). Then push your Rails project to this repository. Refer to the git instructions posted for help with this step. Finally, follow the instructions on the assignment page to submit your Bitbucket URL to the automated grader queue.


what exactly do you mean by a suborder?


Wigs of giants? Is this a typo?


Great, love it! Any chance for a reverse proxy link on Plex as well?

Thnx!


Mathiaaaas :smile: The width and height are relative (54c7b9b5911913528cc38c7431c236ed03cf8034).


1. Would make a list for cities.
2. Then when editing the postal codes list, would have a columns for city, state, and country which would each be a selector from the respective list.
3. When save the postal_codes list, would place all three elements in the 'notes' sql_column with a double || as delimiter


see https://github.com/jbenet/ipfs/pull/14


Would use the globals switch to override the layout settings in order for the city, state, zipcode, to all be coupled to each other.


Have you exposed that? If you want to I can do it, but wouldn't you just use plex.tv.
The entire thing was because you requested it :P


This changes makes IrScrutinizer-jar-with-dependencies be full of "junk" (5.7 MB -> 9.0 MB) like maven classes etc. The log is full of new messages like the ones below. Comment?

Adding org.apache.maven.toolchain.ToolchainManager
Adding org.apache.maven.toolchain.ToolchainManagerPrivate
Adding org.apache.maven.toolchain.ToolchainFactoryjdk
Adding org.apache.maven.toolchain.ToolchainFactoryjavac
Adding org.apache.maven.plugin.PluginManager
Adding org.apache.maven.extension.ExtensionManager
Adding org.apache.maven.plugin.PluginMappingManager
Adding org.apache.maven.project.path.PathTranslator
Adding org.apache.maven.Maven


1.程序能够正确的运行
2.询问用户是否需要游戏时有纠错功能
3.在用户输入数字的时候有纠错功能
4.程序总体上看起来条理清晰，简洁明了


update MySQL DDL statements

mysql> create table product(productID DECIMAL(20) UNSIGNED ZEROFILL PRIMARY KEY, modelNo VARCHAR(20) REFERENCES productInfo(modelNo) ON DELETE CASCADE ON UPDATE CASCADE, inflowDate DATE NOT NULL, outflowDate DATE);

mysql> create table productInfo(modelNo VARCHAR(20) PRIMARY KEY, productName VARCHAR(30) NOT NULL, specsRef VARCHAR(30), descFileLoc VARCHAR(30));

mysql> create table [modelNo]_specs(specskey VARCHAR(50) NOT NULL UNIQUE, specsvalue VARCHAR(100));


Testing


komentarz, nie potrzebne


test comment


Please add *.py[cod]

It is python bite code.  This files can be present in repo folder if you run any python script inside.

I check https://github.com/github/gitignore/blob/master/Python.gitignore but I don't expect that we need something more from it right now.

PS if you this request will be missed I will make pull request after release :)


just some demo code


Why was this removed?


Remove comments.


This may as well also be removed


Ouch, sorry.


Remove


1.定义了多个函数，看起来更清晰
2.有输入纠错功能
3.输出部分语法有问题（small question = =）


1.定义了多个函数，看起来更清晰简洁
2.输入部分有纠错功能
3.输出部分英语语法有问题（small question = =）


da kha tasweer dey,duey mung ta ghalat khaberay key,ta ignore ka, genay ma ta na pre dey so auk kum,ma pesey chaway dey dua kala oshu ta sa uka pleasee babe..


I think i.e. is correct because developer tools means C compilers here.


This is going to take a while, and entry 23 gave me a headache


Good idea :-)
In a second step it would be nice to rename the material to RunelordDwarf too, but that would involve changing it in the mesh too (and maybe some changes in the code). But it would be clearer than Dwarf3 IMO :)


Yeah, maybe just hardcode the three settings in distinct keys for now, eg:

slow-sql: 200, very-slow-sql: 400, very-very-slow-sql: 800 ... it
completely clarifies the level of flexibility here (clearly this should all
be optional. )

Long term we need to think about css flexibility and such, but that is way
too much to bite off for now.

On Sat, Mar 28, 2015 at 10:18 AM, Thomas McDonald <notifications@github.com>
wrote:

> Ah I was thinking those 3 keys (s,vs,vvs) would be the only valid keys.
>
> You could pass the colour values in a similar way and inject thru JS but
> that's pushing a lot of stuff away from convention and adding complexity.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/thomas-mcdonald/ui/commit/b50c682fb4ff678f6748506258c84b4a34e74131#commitcomment-10442687>
> .
>


<!DOCTYPE html>

<title>Marcelo Trovato</title>

My CV

       Marcelo

       Trovato

       marcelo@background.com.ar

Programas

 Adobe Ilustartor

 Adobe Photoshop

Información de contacto

FORMACIÓN ACADEMICA

PRIMARIA: Ntra. Sra. de las Nieves

SECUNDARIO: Inst. San Cosme y San Dami&aacuten

TERCIARIO: La Fundaci&oacuten

UNIVERSITARIO: Universidad Abierta Interamericana

 p

            marcelo@background.com.ar


Well, the T is easy enough. I don't like the '-' in filenames but that should not stop them being used (we discussed this on IRC briefly) so %FT%F is ok with me. The ms or serial is more problematic that that...


I don't agree and the proof is on how you've used it. Check it and you see you're getting a Syntax and then calling toIdString() on it. We should pass a syntax, I'm pretty sure about that. Do you have a use case where you don't have a Syntax?

Also whether we pass a Syntax or a String doesn't change the fact that the tool is not placed correctly in Commons. It must have knowledge of the Rendering and thus it should go at least in Rendering or in Platform.

Thanks!


Uhm, no :) I'll move it to my fork and let's delete this one.


rebar do not need meta information


The G_TYPE_POINTER here should probably be a GST_TYPE_PLAYER_MEDIA_INFO


You're confusing things.... :) The notion of URL is not a notion brought by xwiki-platform-url! Same for quotedPrintable.

Regarding the options, we already have a RenderingScriptService with parse(), render() and resolveSyntax() method. I'd much prefer to add an escape method in there. What you propose sound too complex and not needed IMO. It's much more clear to use ```$services.rendering.escape(...)```.


@MirceaKitsune but why?


No need for the ifndef-define-endif header guard in C++. Just use `#pragma once` at the top of your header files!


"Is it only me or does this feel like we are over-engineering a simple solution?"

If you had put it directly in RenderingScriptService from the beginning we would be done with it :) It sounds complex because we've made two mistakes to far: forgetting to pass the Syntax and putting the tool in the wrong place.

I really don't feel than adding a method in an existing ScriptService that is dedicated to what you're trying to do can be called over-engineering :)


The type is not required here, you can get it via G_OBJECT_TYPE() and can check the types also with GST_IS_PLAYER_VIDEO_MEDIA_INFO(), etc.


You need to unref the pad after usage


Ahhh I'm so sorry, I wasn't aware they were independent of each other. Will revise so that we only launch croutonwheen in the xorg case.


Works for me™


I tried to compile the libusb-sys v1.2.6.0 project using Visual Studio community 2013 and WinDDK v7600.16385.1. and got some error messages as below:

1>  Unable to configure WinDDK.
1>C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V120\Microsoft.MakeFile.Targets(38,5): error MSB3073: The command "CD "D:\LibUSB1260\libusb-win32-src-1.2.6.0\projects\..\ddk_make"
1>C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\V120\Microsoft.MakeFile.Targets(38,5): error MSB3073: CMD /C make.cmd "arch=x86" "app=driver" "noclean=true" "outdir=D:\LibUSB1260\libusb-win32-src-1.2.6.0\projects\Win32\Debug\libusb-sys\" "DIR_INTERMEDIATE=D:\LibUSB1260\libusb-win32-src-1.2.6.0\projects\Win32\Debug\libusb-sys\" "debugmode=true"" exited with code 1.

By the way, I had modified the make.cfg and NMake Include Search Path setting based on my own environment.


good


ok, it turned out it wasn't as simple as this. You can see the amendments I made to this commit on master. But I still used `Date.parse`.


@Hellowlol I know - great to see that you have picked it up so fast, thnx!

The only thing plex.tv do is a redirect to the local/public IP, essentially the same thing I'm trying to do. It is somewhat difficult to explain to my end-users that they can click on the 'front-end title' to go to the underlying application, but that for plex they have to do something differently.

No big deal here - I can add an external website link easily. But for the sake of consistancy it would be a nice addition.


If I were you I would not delete this yet, I still see some people using pre 2.0 versions


Also note that AFAIK there's no way to escape wiki syntax if you're in Groovy or other scripting language which means that putting the feature in a Velocity tool is not the right place. RenderingScriptService sounds much better for all those reasons.


You mean the button map on the X input devices we're modifying? That makes sense at a glance and wouldn't be hard to add to this script, though if we want to monitor changes it would have to happen in croutonwheel?


As far as I understand runcmd is supposed to run on every boot? Isn't that the case?


I'm pretty sure it's spelled "JavaScript", not "JavasScript". :smile:


Исправил в 32261db658887f3ed26bbc82f65119cffa03a120


1.程序能够正确运行
2.第9行略显多余，可以在定义的时候赋初值
3.srand (time (NULL))应该放到循环外吧
4.注释很多很详细，有助于reader阅读程序~~~


@agudian , looks good to me. Please, go ahead, rebase and merge


Ah, to be honest I completely forgot about it and have not looked into it yet.


Sure @sampsyo, shouldn't have missed that in the first place!


New function with misleding name. Too general name for a too specific problem.

If the column contained dot at the and, it would be the solution:
```sql
column_name like '%.$value.%'
```
Without dot at the end, here is a solution without regexp:
```sql
concat(column_name, '.') like '%.$value.%'
```

And the example should be before annotations. Then PHPStorm would not think it is a return value.


I think that @gunnarmorling has been most involved in this one. Perhaps he can take a look.  For me its ok.


Zip contains an exe file that can run without Python


it should be
if(item.warehouse && item.item_code && item.batch_no) {


What's going here?


tracked configurations ini filenya belum ter-push


strange, maybe better to remove optimize option for Symfony 2.6 ?


Nanti setelah diperbaiki Pak Mody bagian ini harus muncul lagi hash code nya.


ここ、basicのとことnotesのところで、結構名前の間違いがあったんでしょうか？（もう修正していると思いますが）


Yeah, some are al little "hidden", we need to start making a list for our next release ... :-)

Regarding this A-GPS download feature: I am not convinced it works yet as desired, at first sight the  GPS Status app does not seem to indicate that A-GPS was updated via OsmAnd after 16h. I will add a display of getSettings().AGPS_DATA_LAST_TIME_DOWNLOADED to our Debugging settings (below the memory displays) to facilitate debugging, ok?


90946736_WSHP_#4567 (Nigusse) - Win64-Windows-7-VisualStudio-12: OK (1671 of 1677 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.64%25-yellow.svg)


go go go


I think a foreach loop would be more readable here.


Could we done guard clauses?

```java
if (!enabled) {
    return;
}
```
It removes some of the indentation and it's still really obvious.


Please run some kind of auto format before you send a PR. Lots of functions are using the Allman style and single line if statements are not surrounded by `{}`.


Shouldn't this release be two lines down? (Right before the foreach end)


On 03/28/2015 09:43 AM, Robert Nagy wrote:
> As far as I understand runcmd is supposed to run on every boot? Isn't
> that the case?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/felfert/cargos-buildroot/commit/da1aa716b114235ec87e94ec0bdac97f3cb375d2#commitcomment-10449250>.
>
Nope. The default is only on first boot. There is a feature where one
can switch to run all the time but I never used that.


@Rikk: Because you keep reporting bugs in the commits and ignoring the fact that we have a bug tracker just for this purpose. It's impossible for the people involved with the project to follow 100 places.


Yes it works.

At least in debian, it is as simple as ``apt-get install python3-matplotlib``.


@Gustorn Thanks I will. :)


Last commit introduced a bug (since no tests yet).

Do you mean a sampe task in some test-case? In the readme??


This looks like it's either incomplete or the variable declaration should just be an assignment.


Can the configFile become null?


I think Lucas makes a good argument (as is usually the case!) I myself have been seduced by an STC SPRT result and ended up wasting time as a result. Moreover, in the case of a false negative, compelling ideas always end up getting tested again some time later when someone has the same idea (or re-visits it). So imho moving to a slightly tighter STC condition makes sense. But it should ideally be consistent if for no other reason than to avoid wrangling over test parameters.


why you should use it and you should be all set up to do so.
感覺這句翻譯的怪怪的


die idee ist,das (wenn die variablen gesetzt sind) internet erstmal pauschal blockiert wird (beim booten), solange bis vpn geht (das heisst die <tt>erste</tt> verbindung aufgebaut wird). falls die <tt>letzte</tt> verbindung abgebaut wird, geht vpn nicht und es wird wieder geblockt. der funktionsname ist nicht gut gewaehlt und das ganz ist recht generisch aufgebaut. im prinzip gibt es ja nur eine verbindung...ich benenne es mal um: aba2c9c44e9406e0d134f5bbfc4c959919b8c2c5


Hm... first fact is that exec-maven-plugin obviously *is* a dependency. What seems to happen here is that a complete maven  toolchain is pulled in, which is as it should be if the jar-with-dependencies should be complete.

The thing is that this is a build dependency, not a runtime one. Have we sorted out if the jar-with-dependencies reflects build-time or runtime deps? I havn't,  for sure. Also seems that this should/could be handled by some scope in the dep, but I'm not sure what's suitable...


Ok, I'll add it.


This is selecting the max value using `String` comparison instead of `Slice`.  Since the min/max range can be slightly larger in our setup, I think you will need to verify that the min of `chunk` is greater than or equal to `columnStatistics.getStringStatistics().getMin()`.


i'll add a warning on the release notes saying they should use 2.0.x before updating to 2.1 but this code is causing more issues then anything else sadly


You should have a basic understanding of what Git is and how it's different from the centralized version control system you may have previously been using.
這一句翻譯也怪怪的。


that's set up with your personal identity.
這個沒有翻到


:D like that change


updated


Исправлено в b4d7f9449ee0b1656ce77da2d8f25137a9471707


Thanks, fixed here: https://github.com/jgranick/haxelib-shim/commit/6f6a3a205c433e494b5aec1d4537c14f4b7d483a


Do you mean 'pfx' instead of 'pdx' here and L53?


This commit can be merged into master without conflicts.


Why shp extension?


(一半一半已认领)


This looks better, thanks for the fix. But one suggestion: maybe you should parse the number until the first dash occurs, because this would allow to catch nzbget development version below 12 e.g. 11.0-dev, which passes the check right now.


30414/Huruiala V. Vasile Alexandru/assignments/codeblocks/assignment3extra/functions.c

"the strlen condition is necessary since a negative number would be seen as an operator"

not if you use ```strcmp``` and compare the whole content


Yes, I did that intentionally, if someone is using a dev branch it's their responsibility.


looks good as well


j


Fair enough :-)


（一半一半已认领）


gstprovider/gstprovider.cpp:36:#include <QWidget>


Fixed


I think it's referenced only once by the settings ["nw..."] it's _curCharName or a char name from a forEach loop.
Yes, it meant for info gathering only, not for use. But there are other objects/maps, like the settings one that will be for use and referenced by the char name, doing a compact/sparse array is not reliable.
The index of the characters is not guaranteed when pulling from the model. And it's actually guaranteed to change if someone deletes any character (except the last)

Most of the settings as it is is planned to go away as soon as there is a working way to make a form out of a JSON object and vice versa, for now the only thing i can think of is make the <input> names parsable like for example
settings__char__char name__vendor__vendor_kits
or/and use data attributes
<input type="text"  data-char="char name" data-group="vendor" data-setting="vendor_kits">

2. I suppose it's trivial to initialize to zeroes for example: "Leadership": { used: [0,0,0,0,0,0], unused: [0,0,0,0,0,0] }, (there are always 6 worker types and 4 tool types scanned for now)
but nulls are easily converted to empty string with $.trim(null) so i don't know what is better.


Why can't we upload the www folder (python files) with arduino folder sketch?
And is there any solution to this warning message by arduino:
Warning: problem with accessing board folder /www/sd


Well, it seems like we are breaking our own convention. And it may be problematic when we have more than to `SyncableTanks`.

But now consider this: why I would merge it, and remove contents from every tank in existence, without any explanation what it currently breaks or why it's necessary?

Your idea is good, but it's not good PR.


there is no init.d in the install. and I use systemd.


I think you mean .pfx here


I think you mean .publishsettings here


These warnings are generated during build and I believe they are of the dangerous type:
```
src/Core/OOOXZManager.m: In function '-[OOOXZManager showInstallOptions]':
src/Core/OOOXZManager.m:1718:5: warning: 'NSDateFormatter' may not respond to '-setDateFormat:' [enabled by default]
src/Core/OOOXZManager.m:1718:5: warning: (Messages without a matching method signature [enabled by default]
src/Core/OOOXZManager.m:1718:5: warning: will be assumed to return 'id' and accept [enabled by default]
src/Core/OOOXZManager.m:1718:5: warning: '...' as arguments.) [enabled by default]
src/Core/OOOXZManager.m:1719:5: warning: 'NSDateFormatter' may not respond to '-stringFromDate:' [enabled by default]
```


Hi Audun

do you intend to publish your code?

I have an intern whom I'd like to have work on it

-D.


We should hide only private comments (Reviewers comments) right.
I did send pull request with this change (https://github.com/pythonindia/junction/pull/192)


Instead of **~** do we need **ENV['home']** here and all occurrences in the README.md to support Windows too?


`with open(self.filename) as file:` could technically never execute the inner scope (in this case line 23). (Thats a default python contextmanager.) So line 24 would still be executed but it doesn't make sense if line 23 doesn't get executed. So I'd change that to:

```python
        with open(self.filename) as file:
            lines = file.readlines()
            self.assertEqual(lines, [])
```


I don't understand your concern. This patch intends to change the logic in a sense that if `RAPIDJSON_HAS_STDSTRING` is defined to `1`, then `std::string` support is included, if it is defined to `0`, then it is excluded. If undefined, it is set to `0` (unlesss we're currently building the Doxygen documentation).

What problem do you see?


as above


What's the reason behind those changes? I wonder if I should merge them in. :)


変えて見舞いs他


Is there any specific reason (eg. performance) to limit max length? If not, I suggest you bump it to 125 which is the max of core john.


Исправлено в efbf73f02f5376ca73e819bb10117c91fbaa6f45


If you claim a binary align of 4, the binary() function needs to return a pointer with that alignment but it currently returns a char pointer. In this case I think you can just set BINARY_ALIGN to 1.


Ясно, учту.


求解这样的递归算法，一定要画出递归树，这道题，分治意思就是先分解再治理，先把大问题分解成一个很小的问题，最后，再求值。


removed setwebdriver


You'd have to declare ```p``` as a var somewhere before using it in directly, which is why assigning the key as a char mad sense ```self.uut.write_tags(p: "test", t:  "another test")```  and so probably ```self.uut.write_tags('p': "test", t:  "another test")``` instead?


Ok, so it seams only the first example works:

```scala
def f(a: (Int @Implicit) ⇒ String) = println(a(5))

def g(implicit x: Int) = x.toString

f("Hi, " + g)  // works
```


hm, What is this file?


@rascarlo ok...thanks for the info...
I ll try this week-end...seems that the mVolumeFlag doesn't block the KEY_EVENT...
a deep investigation is needed


[Semantic Markup with Foundation 5 and Sass](http://www.sitepoint.com/semantic-markup-foundation-5-sass/) by [@George Martsoukos](http://www.sitepoint.com/author/gmartsoukos/)（一半一半已认领）


This is very interesting, but you're just breaking a lot of getting-started guides like [Atomic](https://github.com/projectatomic/atomic/issues/30), which don't use DNS yet.


OMP_SCALE is for hiding OpenMP overhead for very fast formats. In this case (slow hash) I suspect you can leave it at 1. Or you could start at one and benchmark, double it and benchmark again. Stop when there's no significant performance increase.


`bool(options.cover_erase)` is probably just as effective here.


Sorry to be nit-picky, but can the comment be above the line rather than after it?


Will this fix https://github.com/strands-project/fremen/issues/10 ?


You are right. Those two files do not require any subscriber. I was talking
about seabotix.py because you were asking me what happens when nothing is
being published and I agree that when nothing is being published, thrusters
should not use use previous data; instead they should not move.
Regarding the testThruster.py, I think that we should keep a subscriber in
it, so that we can check specific movements (like forward) if we need.

Vishnu Dutt Sharma
3rd Year Undergraduate Student
Department of Electronics & Electrical Communication Engineering
IIT kharagpur

On Sat, Mar 28, 2015 at 1:01 PM, Siddharth Kannan <notifications@github.com>
wrote:

> @VishnuDuttSharma <https://github.com/VishnuDuttSharma>
>
> If you want, take out the publisher from the loop
> and put it inside the callback.
>
> What do you mean by this?
>
> Note that the subscriber callback has been removed, as you said that it is
> not required inside these two files. (testThrusters.py and stopKraken.py)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/icyflame/kraken_3.0/commit/9ca6eeeb3264b9087a3a336757ba85a6e3069523#commitcomment-10449126>
> .
>


Very nice!


It seems that the client sending the empty string is disconnected. The client seems to do this randonly.
Later we'll stop it from doing this, but for now your connection must survive this error.


This is awesome.


TBH, I think the snapshot is a bit OTT and trying to be a bit too clever. If we can guarantee the README.txt structure then that would much easier to implement and would provide a consistent list of cards for each new release.


This works however I need to make it more clear and specific for adding multiple ingredients, and also have another one for adding booze items.


te


Wow, thank you for quick update


I can to knock something up before 1.60.


Just return at the start of the function if this is zsh.


Yeah, but using appveyor, we can publish nuget version in our CI, without having to do this manualy.


@andresgutierrez
Do I read this correctly that you only can set boolean values? (since zend_is_true is used?)


Oops! Committed before finishing my comment! Comment should read:

Watch tells time in binary. Currently showing time in 5 digits. Will update to tell time in 8 digits.


r+


Test


91069532-FixUnitTestFailures-Part2 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (1675 of 1676 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.94%25-yellow.svg)


saw approval from jdm
at https://github.com/Adenilson/servo/commit/41e02035f996513d9d67f72487e282c905c41434


merging Adenilson/servo/loadPlaceholder01 = 41e02035 into auto


Adenilson/servo/loadPlaceholder01 = 41e02035 merged ok, testing candidate = b999c0b6


It's `std::vector`, not `vecter`

Also, it's a good idea to name arguments in the interfaces too.


Please use 4 spaces instead of tabs for indentation.


How do we test the converter then?
On Mar 28, 2015 7:24 PM, "Bart van Merriënboer" <notifications@github.com>
wrote:

> Could you put all the commands inside if [ ! -f binarized_mnist.hdf5 ]
> and do rm binarized_mnist_train.amat after fuel-convert? The caching
> mechanism on Travis isn't very fast, so it'd be nice not to have to
> retrieve binarized_mnist_train.amat from the cache each test if we don't
> really need it.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/bartvm/fuel/commit/3384d328ab50c4c81d7820cce75341b9763fee3a#commitcomment-10455180>
> .
>


The goal is to follow packaging guidelines more closely. We want to move in the direction of distro-proper packaging, which means the prefix for most software should be '/usr'.


    for (auto const& arg : args) {
        if (arg == "-h" ...

would be cleaner, safer, and more idiomatic.


i think we need to emit some warning here, saying that parsing the header failed, instead of silently ignore the previous error.


Functionality in this commit:
- On initialisation of node, default pwm of 0.015 sec on both pwm1 && pwm2
- Default Functionality::
    - LEFT STICK   -> PWM1 >> Full Forward (0.02) || Full Reverse (0.009) || Default (0.015)
    - RIGHT STICK -> PWM2 >> Full Forward (0.02) || Full Reverse (0.009) || Default (0.015)
- KILL SWITCH -> One press PWM1 >> 0 && PWM2 >> 0
- START SWITCH -> Fall back to default functionality


Test comment


I'm sorry. I didn't remember that


meeeehhhh


what


Oh, I think I know, You have disabled clang for ART.. maybe that's it


prime number should be different for each value


Hive dev here - why this change? I'd think it would be a better idea to use the official appcast which contains only the versions that the author actually wants to publish as updates to existing users. By including everything that's on the Releases page you will also get occasional beta and test versions which are only intended for a small group of people or even for single users who have some kind of problem (scroll through the https://github.com/hivewallet/hive-mac/releases page to see some of them) - are you sure that's what you want?

I'd recommend to use the official appcast URL from Info.plist, which right now is https://mac.hivewallet.com/feed/hive-mac; the appcast is actually stored in the repo now at https://github.com/hivewallet/hive-mac/blob/master/appcast.xml, the official link simply lets me gather Sparkle system profile info (though that's irrelevant in case of Cask), and also potentially change the appcast location without breaking it for existing users.

BTW, you can also change the homepage address to https://mac.hivewallet.com :)


jjj


kkk


This will not pass. I think one has to use DELETE


A riesgo de no cometer (en mi opinión) el mismo error que @BogotaJS, mencionaría que adaptar el código de conducta implica que mencionemos que es @IbagueJS y no `colombia.dev`, pues esto último ya se aclara en la parte inicial.


Proposed fix in Pull Request https://github.com/ansible/ansible-modules-core/pull/1024


Yes, that is my testing. haha


This is the essential part of the program.


Looking good, @JakeWengroff!

![](http://www.reactiongifs.com/r/m-i-r.gif)


Oh no, not my lovely toast! :bread:


why?


some tests failed:
failure: http://build.servo.org/builders/gonk/builds/374


To avoid using message(WARNING "..."):
    message("")
    message(" *** WITH_SOURCE_TREE - WARNING!")
    message(" *** This functionality is ONLY supported on CMake 2.8.12 or higher.")
    message(" *** You are running ${CMAKE_VERSION}, which does not have the functions needed")
    message(" *** to create a sourcetree - this option is thus forced to disabled!")
    message("")


:+1: nice


:bow: :D


saw approval from jdm
at https://github.com/frewsxcv/servo/commit/46f14449d0ddaf7fc2db60a40364d968c2283be4


frewsxcv/servo/spec-links = 46f14449 merged ok, testing candidate = 4cf76d65


merging frewsxcv/servo/spec-links = 46f14449 into auto


@cmaloney — Thanks for the clarification and makes total sense.


:thumbsup: :thumbsup: :thumbsup: :tada: :tada: :tada: :shipit:


The `skip` flag has to be considered here!


THAT'S BREAD


Is there a significant difference between this and Bricked?


OMG, this is my FIRST comment on GitHub!

And with an embedded animated .GIF to boot!

You never cease to amaze me, @Brit200313!


you should use HomeTabletPage homeTabletPage = new HomeTabletPage();, please correct that in all of your test cases


Yea, the code is WAY more simplified but written by the same person. I though I would at least try it out to see how I liked it before making a judgement


`libmodplug` still needs to go in here


I feel that reading from README.txt and assuming a particular structure is more error prone. Restored the the call to updateNewCardsLog in https://github.com/magarena/magarena/commit/3ef1969759581695edc2b4673b1da6fcb93a0e00


Block Devices


LGTM ;)


91069532-FixUnitTestFailures-Part2 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1675 of 1676 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.94%25-yellow.svg)


`module.exports` is [reassigned a few lines later](https://github.com/fwg/s-expression/blob/7065ab3aa5be26ed2e1f74187a14efd5778d49e9/index.js#L32).

What's the use-case for having this in code, rather than just documented?


91069532-FixUnitTestFailures-Part2 (mjwitte) - x86_64-MacOS-10.9-clang: OK (1669 of 1670 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.94%25-yellow.svg)


readme-edits


91069532-FixUnitTestFailures-Part2 (mjwitte) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


pull


wwhat theee


haha @JakeWengroff. Just wait -- Github is going to be your new #1 favorite social network!


It'll still get included in the final product when `MC-183` is merged into `master`.


I already like it more than LinkedIn.  LinkedIn totally s--- s---.

Hey, wait, aren't you supposed to be sleeping?


I figured :P


LGTM


finally


Just woke up :)


Can you explain this line?


why did you name this commit yeah@@?


what does actual title mean?


Cool, sounds good :+1:
In that context, it does indeed make sense to really drive the idea home.


This should probably be `apt-mark hold xserver-xorg-video-dummy$ltspackages` now...


All those may be my fault for copy/pasting files without worrying too much about unneeded includes


use `--depth=1` so we don't download all the history, just to delete it in the next step (there may be a better way with `git archive` see http://stackoverflow.com/questions/1209999/using-git-to-get-just-the-latest-revision/1210140#1210140).


We should really be using execFile so we don't have to worry about escaping.


:laughing:


NIce catch


Hash size must be 32. Everywhere that Bitmessage uses SHA512, it either uses the first half or the second half of the hash.


Pretty much my results MRI - 2.8 here (2.5 on my setup) and JRuby - 1.9 here (2.0 on my setup).

Method invocation speed (class with method calling method returning instance variable) seems in between the two, so just in general:

would it be ok to refactor out the global variables + put the logging/debugging elsewhere? Or is speed really critical in this regard? Just asking to get a sense of what needs people have out there. (At this point optimizing JRuby internals would probably bring more benefit).

I believe refactoring out would actually help the JVM with inlining optimizations (based on headius's blog article) if classes like Task don't have the debugging to begin with.


Why didn't you also moved the ODPacket ? We should keep both (ODPacket and streams) together...


Please don't use the first thing you find, that is actually completely unrelated to your problem, to open a comment.

1. Your issue is with libusb-win32, which is a __different__ project. This is libusb. Yes they both have 'libusb' in their names. No, that doesn't mean that they are the same project.
2. This commit has NOTHING to do with your issue. Especially, it has nothing to do with WinDDK.

The less effort you put into trying to open an issue at the right location, the less you will receive help.


Just newHash is fine. No need for newHash[:]. My bad, sorry.


Lot of code cleaning here ^^


Additions unnecessary since ShaSize must be 32 not 64.


Same issue as previous.


@ypling 少年，你没有上传CBTINavBar.js 文件……


Nevermind, I see now.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1195
success: http://build.servo.org/builders/linux2/builds/1193
success: http://build.servo.org/builders/android/builds/1229
success: http://build.servo.org/builders/mac1/builds/1408
success: http://build.servo.org/builders/mac2/builds/1411
success: http://build.servo.org/builders/gonk/builds/375


91069532-FixUnitTestFailures-Part2 (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (657 of 658 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.85%25-yellow.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-4.87%25-green.svg)


@rm -f libevent-2.0.21/staged => @rm -rf libevent-2.0.21/staged


Indent's a bit weird for the first imports here.


fast-forwarding master to auto = 4cf76d65


Looks good.

When I had done the PR to update language files, I mostly just looked at directly replicating the individual small commits.  There was one really big commit involved with lower-casing the optional and/or unnecessary capitalization of a bunch of things, and that was the commit that I got frustrated with replicating and just re-did the change manually with my eyes to the language files.  I'm glad to see that this didn't leave many differences for you to tend to.


Same issue.


Nice change mate


... and this could be a `"$getBaseName() $id"`


You the man


@jkr, thank you for your kind words.

This discussion is spawned by #2032 -- it didn't break anything really, just a minor inconvenience, which is already fixed. That said, commit message seems a bit puzzling, since 6848f642e82322c0894c62d3215e98325ab7fd8c and up just ignore reference settings.xml.


Consider `@TypeChecked` or `@CompileStatic`, to make it more robust and understandable for others :)


Для того, чтобы не расставлять символ конца строки `\`, стоит использовать дополнительную пару скобок. Более того, использовать символ `\` не рекомендуется, потому что от этого затудняется изменение кода.
```
if (".fasta" in sys.argv[1] or ".FASTA" in sys.argv[1] or
    ".fas"   in sys.argv[1] or ".fsa" in sys.argv[1] or ".fa"  in sys.argv[1]):
    in_path = sys.argv[1]
else:
    sys.exit("Wrong input file format!")
```


Hola,

Ya estoy en Madrid con lo cual no podré ir al taller de mañana. Como
hablamos me gustaría seguir involucrado desde la distancia. Mi pequeña
contribución esta semana:

- Cambiado los colores de los favicons según branding the las distintas
redes sociales
- Incluido algo the layouting básico (más que nada jugué con él)
- Intentado añadir animación a los favicons utilizando el componente
core-animations. Al ejecutar dice que el generador del animaciones que
ofrece core-elements (CoreAnimation) no está definido. Se agradece alguna
pista.

No sabía exactamente como trabajar con github por lo que me cree una rama.
https://github.com/dfleston/polymer-study-group-app/ y empujé mis cambios.
Los ficheros cambiados son:
- index.html
- elements.html
- main.csss

Agradezco si alguien sugiere un mejor approach para trabjar con Git Hub.

Para mañana: Una idea genérica sobre una de las pantallas claves: los
listados (conversaciones, contactos, etc...)
Estaría bien si usáramos algún mecanismo/componente para poder hacer select
de elementos en la lista de tal manera que al tener algun elemento
seleccionado aparezca un paper-fav (abajo derecha) que sirva para abrir un
menú de opciones.
Un mecanismo interesante podría ser un swipe sobre el elemento de la lista,
pero si no valdría un click sobre el icono del elemento. De esta manera se
pueden hacer acciones masivas sobre selecciones.  Ej: seleccionar contactos
en un grupo y crear un subgrupo.

Simplemente una idea.

P.D.: Por otra parte, al clonar el proyecto, este no fue desplegable. Daba
errores de algún componente grunt que no dejaba generar un servidor web.
Los instalé pero no solventó el problema con lo cual lancé un servidor http
de npm. Esto es un tema mínimo pero "just for completness".

A ver que tal avanzamos.

Saludos y gracias por cualquier comment

Dos pequeñas aportaciones:
· Cambio el sizing a cover para que la imagen se ajuste al tamaño de la
ventana.
· Añado los iconos sociales. (Tras obtener teneis que volver a ejecutar
bower install para obtener la dependencia social-post-icons)

—
Reply to this email directly or view it on GitHub
<https://github.com/vilasmaciel/polymer-study-group-app/commit/7136551172ce4868d830049a2d344ded89c47e7f#commitcomment-10371416>
.


This is the master function that is used to set node properties for specified nodes directly.


Good job! :)


ignoreeeeeeeeeeeeeeeeeeee!


30411/Alb V. Corina Bianca/tree-list-tree/main.c

You can simply return NULL here to avoid the warning


为什么选择6。。


Interesting choice of location! Joke aside, I think it would be a better idea to take the map of a small country, like Andorra for instance. This way, the test map can then be bundled with the code.


There is a converter for Syntax so you don't need a special method taking a String.


The reason for this change is that eclipse seems unable to recognize .o as a Presto type, it always defaults to the binary editor.
So we move to 3 letter extensions: ped, pod and ppd
Overall, this is not a bad idea since those extensions are free for languages (only ppd is in use for Postscript Printer Definition), and makes it easier to add dialects in the future, such as pfd (french), pgd (german) etc...


And I thought I knew you!


r+


r=jdm


Are you going to make a pull request for these changes? I would happily merge them.


Why is this 100? Won't this severely hurt perf?


@brki I was first surprised to see this parameter being added on the next_version and previous_version functions. But after reading the documentation below I understood what it meant.
I think it would be good to rename this parameter to 'relations_as_of' to mark the difference between this 'as_of' and the other 'as_of' parameters we already have on some functions.

What do you think?


Комментарий выше относился к правильности написания длинных условий и других выражений.

Если написать это правильнее, то примерно так:
```python
in_path = sys.argv[1]
fasta_extensions = ['.fasta', '.fas', '.fsa', '.fa']
if not any(in_path.lower().endswith(extension) for extension in fasta_extensions):
    sys.exit("Wrong input file format!")
```
Таким образом, добавить новое расширение будет очень просто, а также не учитывается то, в каком регистре написано расширение самого файла (`.FaS` также будет приниматься)


wow, easy there.
```curl http://localhost:8080/t/../../../../../etc/passwd```


Like the functionality if fixed.

The Invert string should have ?__( ) macro
and most important it loses previous hard edges that are not selected.


Ok, fine! However, I can't see the updated version. There is just one commit on this PR.


Ok


增加或移除一个class可以通过classList对象的相关方法进行，比较方便，而且也比较安全


Good functionnality.
Can you modify your PR for the following:
Line 2006 should log with LogDebug instead of Log. This would prevent having a logged line every MAX_WAIT seconds.
Also line 2008 should be used with LogError so an alert email gets triggered.


hi guys, it looks like the behaviour you wanted at RouletteV1YibnlRaphaelRacineTest.java@L41 was:
assertFalse(client.isConnected()) or assertTrue(!client.isConnected()), not assertFalse(!client.isConnected());
cheers, Val.


Ack


This one is kind of tricky. For number formatting, we want to use CurrentCulture. But for abbreviations, we want CurrentUICulture, since that is related to localization of an application. In this context, you should pass in CurrentCulture. That would format the numbers correctly and will not have any effect on the abbreviation strings. The abbreviation strings are produced with CurrentUICulture if cultureInfo is null.


2 semester/3 homework/Task 3.2/Task 3.2/HashTable.cs

лучше явно писать модификаторы видимости


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1196
success: http://build.servo.org/builders/linux2/builds/1194
success: http://build.servo.org/builders/android/builds/1230
success: http://build.servo.org/builders/mac1/builds/1409
success: http://build.servo.org/builders/mac2/builds/1412
success: http://build.servo.org/builders/gonk/builds/376


Для таких штук стоит использовать raw literals из специализированных пакетов, например, [raw-strings-qq](https://hackage.haskell.org/package/raw-strings-qq-1.0) ([примеры](https://hackage.haskell.org/package/raw-strings-qq-1.0/src/examples/RawRegex.hs))


Same here.


Referring to: http://jsfiddle.net/doronhorwitz/022nnszd/
If you shift+click in the overlapping area it selects both features even though ```multi: false``` is set explcitly on the ```ol.interaction.Select```
Also, if you select both features and then click on the overlapping area it deselects both features.
The way I believe to fix this would be as I mentioned above to put the ```multi_``` logic in the ```else``` of the ```if (set)```


а зачем она public? ведь никто, кроме хеш-таблицы, не сможет правильно воспользоваться хеш-значением всё равно


I must investigate this as i don't see the logic behind your PR, it looks just the same to me.


r+


This is great, thanks for that. Do you think it would be very difficult to add the option to start at a random vertex, sampled according to some predefined distribution? Interesting and simple distributions include *uniform* and *stationary* (the latter is interesting / makes sense only if the random walk is ergodic).


We might want to add a simple test for ToString() as well, one with default culture and one with specified culture. There are similar tests for ToString you can look at.


fast-forwarding master to auto = 7d0d8514


Now when you use bulk edit, the **attribute name** is a selected field. In this field you can **only select attributes who are common between all of objects selected**.

I add a list : **PYNAG_ATTRIBUTES**. She contain all attributes related only with Pynag objects. I use it to exclude more easily these attributes in BulkEditForm and AdvancedEditForm.


@Tourorist вроде норм, спасибо


вот тут не очень хорошо... так как на самое деле это никакое не ДВ, а УКВ


In old version:
In parent run (with --on-changes param) LockDirectories is not called at all. This function is called from Main.
So in old version, parent doesn't lock, and child doesn't lock too.

In my new version:
In parent lock LockDiorectories is not called at all too, but lock is done in child. Change on line 2089 is for UnlockDirectories in parent, which is run when SIGTERM is received.


И не КВ, а тоже УКВ.


Можешь вернуть эти штуки, как были?


credits to tpruvot.


Done, also for Mass


@RReverser Please don't introduce merge commit when it isn't necessary. (I.e. don't use Github's merge button, but rather cherry-pick or rebase the contributed changes on top of the existing master branch.)


saw approval from Ms2ger
at https://github.com/michaelwu/servo/commit/3af3f350adb64e9004261c8a6319bbbfdab2b931


merging michaelwu/servo/fix-more-gonk-warnings = 3af3f350 into auto


merging michaelwu/servo/fix-more-gonk-warnings = 3af3f350 into auto failed


I wonder if we should remove the last subtract, that can be done in user code, and
change the name of the functions to add_neighbor_faces().

Also do not like the strict name do not understand it.

Remove wings_face external calls to local functions.

Optimize both functions by using outer_edges to avoid creating large gb_sets.


It would be nice if the library could be expanded to support the esp8266 as well; this is probably dependent on the PWM code being implemented


How do you think about the applicability of [the parameter "VERBATIM"](http://www.cmake.org/cmake/help/v3.2/command/add_custom_command.html "Reuse of VERBATIM?") at more places?


excellent, thanks!


erm what?


2006: after timeout, sync is run so in log it would be many other messages. Is it good idea to not have information why sync started (without debug enabled)?
2008: ok, fixed

thanks


nice catch @PareshMayani


r+


saw approval from Ms2ger, jdm
at https://github.com/servo/servo/commit/e8a1e9eabb121aff2aab3eb82395880b733d0090


merging servo/servo/rooted-vec = e8a1e9ea into auto


servo/servo/rooted-vec = e8a1e9ea merged ok, testing candidate = 08e8ab89


stable 29/03


@Melvar @edwinb I do not remember a discussion that brought consensus but the related issue is #379, and i think consensus was not reached.


The dead connection detection should rely on UTC instead of local time as humans of the stupid habit to time travel for some daylight saving. This leads to the bug that SmartIrc4net thinks the connection is dead for an hour.


yeah it's totally unnecessary, was just the beginning of an idea to export parser features to check against..
originally this module started as an exercise, and I wanted to re-use the tests for different iterations of that exercise.


I can think of the following extra TODOs:

* **Lazy random walk** which stays put with probability 1/2, 1/n or some other probability (however, this may be emulated by adding edges to the underlying graph, if weights are allowed)
* **Self-interacting random walks**, such as
 - *Self-avoiding random walk* (a walk which won't visit the same vertex twice)
 - *Loop-erased random walk*
 - *reinforced random walk*
 - *non-backtracking random walk*
* **Interacting random walks**: if you allow multiple random walks, it would be nice to let them interact. A few common examples include:
 - *Talkative particles* (For k independent random walks, which communicate on meeting, estimate the expected time to broadcast a message)
 - *Annihilating particles* (Walks destroy each other when they meet, estimate the expected time for extinction)
 - *Coalescing particles* (Walks coalesce when meet, estimate the expected time for a single walker to remain)


Yes this is what I've explained to Edy already in some other comment :)


lol, "shit the lines" I mean, "shift the line" :)


每个都有这个问题：点击气泡后有红色小圈圈出现，但是一直没有显示数字，其它的气泡没有被禁用。
（如果是server.js的启动方式不太一样的话请快回复！）
S2的会直接跳到E的气泡那里去
S3、S4因为第一个就显示不了所以后面的气泡都不会出现小红圈


(y)


@myabc Thanks for the heads up! I was asking Austin about that very thing but he probably didn't have time to answer yet. I think I'll steal / port the .hidden-for-sighted then. :)


Not sure if all languages have only 2 letters. Maybe a better idea is to `explode` the string with `-`?


@bascht if you want to test out a screenreader, try the [ChromeVox extension](http://www.chromevox.com/) or enabling Voiceover with Safari (Mac).


Nice finding the `getAcceptableLanguages` method :)


Можно заменить на `os.makedirs(out_path, exist_ok=True)`


@myabc ChromeVox looks nice! I mostly sticking to the hints that http://wave.webaim.org gave.


shouldn't the exit call be removed ? because atm the unit tests are stopped in travis.


显示数据的顺序和实际点击的顺序当然是不一样的，因为每个都是在1到3秒内随机的。所以五指金龙也是不一样的顺序的。firefox我还真的没有试过。囧


Fixed in the later merge commit..


感觉可以用上x&x-1....，但是是不是只能对正数有用


同意修改


@ksooo, ca you test with the demo addon from here https://github.com/Glenn-1990/pvr.demo/commit/c78a9d275ae8a7b5ce1022cf2ee7bcc9d0ba7b2a
Try to add a recording from timers and from epg, I think all will get clear what the difference is then

I updated most of your comments


I remember I cherry-picked your commit for flto, and I commented the CONFIG parts because I was already doing graphite.
I really don't know if it works without the CONFIG part (I've always done it with CONFIG), but I'd say that I don't like the FFAST_MATH part. I'd personally go with += insead of := just to be sure.

https://www.gnu.org/software/make/manual/make.html#Flavors
https://www.gnu.org/software/make/manual/make.html#Appending


This will in many cases give a C = {0.0,0.0,0.0} since Y and Z will be parallell


Nya~


Why does class Deck have method TakeTopCard AND player TakeCard? Difference? Both needed? Why does board containg the players' expeditions? Why not player? How can player's TakeCard take a card as a card as a parameter? how does that work? who calls it?


J'ai matté votre fichier .cor pour la partie header et c'est du propre, gg les gars :dancer:


@bors: retry
* #4041


:-) Kein Stress ... wenn du soweit bist, dann lass uns mal mit einem Brainstürming @ backup anfangen.

LG
Micha


some tests failed:
exception: http://build.servo.org/builders/linux1/builds/1197
exception: http://build.servo.org/builders/linux2/builds/1195
exception: http://build.servo.org/builders/mac1/builds/1410


GCC 5.0, that's nice


Yeah, I have my kernel compiling with UBERTC 5.0:)


调试信息记得删掉呀= =


saw approval from Ms2ger, jdm
at https://github.com/servo/servo/commit/e8a1e9eabb121aff2aab3eb82395880b733d0090


merging servo/servo/rooted-vec = e8a1e9ea into auto


servo/servo/rooted-vec = e8a1e9ea merged ok, testing candidate = 1282850b


@jdittrich I don't see your changes :/ Which were these?


No it doesn't. I changed it from pango markup to regular text so it can be styled with CSS and text wrapping will work properly. :smile:


Hey Sebastian,

Entity Extends sf::rect which has a top and a left variable. You should use those to check your current position (don't set them move does that for you)


no,you need to pass the string id from the addon's strings.po, not from kodi
check https://github.com/Glenn-1990/xbmc/commit/3c38bcbb4ac8c6258020fa15153f910fbed7767b#diff-6691ab676bd7f3ee8f62528621e85fc6R297

Better to pass a string maybe?


Geez... sorry. I'll change it back.


Det här görs av cancan automatiskt, du kan ta bort denna.


Thanks @polluks for spotting this. Fixed locally in my repository, but I will not create a separate pull request for this fix. Will be part of an upcoming future pull request.


use xxx is a format to import a module
once we import strict with "use strict", we have to announce the variable with "local", "our" or "my" before itself, and the common way to do this it add "my", saying:
my $x =123


Hi Farbdose, it seems that grepping for active:\"true breaks pushing to specified devices. At least it does so on my branch. Can somebody else confirm this?


should be qRound or you will not reach min value


i know... :dizzy_face:  hehe


About AdaptiveLINQ


@ksooo I don't mind if you take this over, I don't have much time this week and I'll get this in ;-)


-Fixed some id and Class on index.html
-Fixed CSS
-Added Media Queries


TODO check the possibility of a try catch fallback to avoid script halt on failing of some packages


I'll  try to make some russian translations for you, shortly.


Oh. I suggest removing the function remove_item() from the class diagram as we can just say you pickup a new item and it "drops" the other item.

Also remaning picup_item (which I assume is pickup_item) to get_item to be consistant with get_totem.


I kinda see what you're attempting to do here, it feels like the wrong approach. Seems it would be easier to handle a touch-device's touch/blur events, then it would be to retool every devices hover feature.


Aha! Found them!


Let's try!

2015-03-26 21:37 GMT+01:00 Jo Van Bulck <notifications@github.com>:

> As soon as we start replacing app elements by native kotnetcli code, we
> should be very vigilant not to break stuff. That comes down to changing
> only one part every time, so we can easily roll back.
>
> I agree indeed. As far as I understood, for now we only have a conflict
> with mechanize and BeautifulSoup4? I would suggest to drop BeautifulSoup4
> then and use regular expressions as you indicated above.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/GijsTimmers/kotnetcli/commit/caaa6a0e701dc06dc8c7ff881f42ab668ab7efe6#commitcomment-10420852>
> .
>


BTW Thomas, would there be any security issue in making available the ConverterManager through a PropertiesScriptService?


Done. I've only done tests for English speaking cultures, because no non-English speaking cultures use Feet/Inches and Stone/Pounds.


ba12dddf51914d680374994a78fca89c6030bb8a same as


same as 94666e57b286798897d1df96a942c48ac6529e02


They add dependencies on odpacket, and means having to link to SFML for every test that were to use them.


Initialized the repo


S1：
1. 如果正在获取数字的过程中重复点击该按钮会出错


S4：
1. 没有首先把顺序显示在大气泡中
2. 如果正在获取数字的过程中重复点击该按钮会出错
3. 全部获取完成后再点击@+按钮控制台会报错


hmm there could be, so we could think about exposing only some Converter, maybe through a new @ScriptSafe annotation that we would add on the safe Converter implementations.


I think this line is unused


Looks good.
Try to avoid using elements selectors in CSS. Use class selectors instead. Check out article https://developer.mozilla.org/en-US/docs/Web/Guide/CSS/Writing_efficient_CSS


且大气泡始终没有样式变化


Test fails for me with:

> Fatal error: Call to undefined function idx() in %s/hphp/test/slow/ext_hh_client/typecheck.php on line 3


There's a typo here: pla**y**ing. Also, the third line is missing a word. It ends with "You **sure** must be tired!".


to_binary_container är tänkt att användas som en privat del i message_base
försök gärna fixa det till
return make(T_META_LIST, K_META_LIST, meta_list);


Testing. @dczlonka could you take a look at tests in circle? It look like gradle is missing something.


Du måste ha en load_permissions här. load_permissions läser in rättigheter som är sparade i databasen.


I tested it a lot, and this implementation works (IF works) very poorly:
- ifeq LOCAL_LTO, okay, if it's exported it works
-ifneq LOCAL_CLANG still okay, not use for clang
-ifeq LOCAL_IS_HOST_MODULE, wut? what's the benefit of using as to host?

I managed to build ART libs with LTO edited by me, THEN i saw a real benefit.


Görs automatiskt, ta bort.


attr_initialized should take TRUE/FALSE. So should not be WORD.


This was causing an infinite loop. :+1:


http://stackoverflow.com/questions/20024690/is-there-byte-data-type-in-c
avoid using cryptopps datastypes:)


this and functions like it should be static


My brilliance hard at work.


Looking forward trying that out! :)


`FileHandle`


Please name this class Utilities. I can't even look at it :P


`WriteToMemo` -> `PrintRecord`


No, you have to remove this line. You should test that:
1. Your controller behaves correctly when user is not authorized (i.e. gives some error flash)
2. Your controller behaves correctly when user is authorized

You can do this in two contexts. Take a look at `spec/controllers/election/candidates_controller_spec.rb`, I fixed that one.


I would seggest that the Enumeration is refactored out and case objects are used instead. So you would have Update#NoUpdate, etc


Do not use `sign_in`, same reason as above


ditto


Add this back please


write this in message_base? so the binary copy is only done in few places.
why not:
inline Message_base::string to_binary(const Metahead& metahead); ?
because of the vector<Metahead>? there are more elegant ways


This too


Didn't understand what you did to jump to definition


* `Integer`
* `zp` -> `salary`
* space after comma


@jvasileff Yes, it seems you're right.


11학번 서성원


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1198
success: http://build.servo.org/builders/linux2/builds/1196
success: http://build.servo.org/builders/android/builds/1232
success: http://build.servo.org/builders/mac1/builds/1411
success: http://build.servo.org/builders/mac2/builds/1414
success: http://build.servo.org/builders/gonk/builds/378


Can I suggest something more like "A framework for creating games using the Mokap engine"?


Note that this file can cause problems with autowiring frameworks, as it is considered as ambiguious class.


#define (const size_t) METAHEAD_SIZE = 1024 ?
put this in glob.h as a configuration parameter?
use
const int TAGS_SIZE =  METAHEAD_SIZE - ... /*all other stuff*/


again, try to avoid this


it's ok, we have the same


Metahead Message_base::get_metahead(const key_type_t& key); ?
again, the vector problem can be fixed


are `params.{c,h}` missing from this commit?


fast-forwarding master to auto = 1282850b


Does that still work with my proposed change regarding "sm" + "md"? Looks like now it can also be one.


most names of variable are meaningful and easy to understand. but when i input n-digit integer instead of y/n,  "需要玩游戏吗（Y/N）？" comes out n times.


liuxdxiaodong


Cool. I will take over. Means, I will create a branch in my fork of xbmc and pull your changes in. Let's hope I will find the time to polish things up...


saw approval from Ms2ger
at https://github.com/servo/servo/commit/88b31933c405c65d5b832bb78939a725770dbeca


merging servo/servo/fix-more-gonk-warnings = 88b31933 into auto


servo/servo/fix-more-gonk-warnings = 88b31933 merged ok, testing candidate = 0d3574b5


Not necessary, 'addImage' already checks it imageUri!=null


Wonder if this should somehow converge with reusableEntity(). Aren't they variants of the same idea?


I think it's not clear to you, have you checked my pvr.demo?

AddTimerType is only for special types, the ones above are in kodi allready..

Let's say that you want kodi to record any time any channel recordings wich tvheaded support through autorec

PVR_TIMERTYPE type;
type.iTypeId             = any id above 0;
type.iLocalizedStringId  = string id in own strings file, lets lay of string "any time any channel"
type.bDependsChannel     = false; // as the type is any channel
type.bDependsTime        = false; // as it as any time
type.bDependsNewEpisodes = false;
type.bReadOnly           = false;
PVR->AddTimerType(&type);


You have just updated headers in .framework folder?


Okay -- will check it out here. The issue here was:

 1. I had a file produced using the base reference docx. The file had footnotes. Let's call that out.docx.
 2. I retooled it so it would have the proper styles. (This retooling changed settings.xml too).
 3. I used the now retooled file as my new reference docx, (`mv out.docx ~/.pandoc/reference.docx`).
 4. I produced a file without footnotes. This was corrupted.

Note that this corruption only occurs if we rename it as reference.docx and put it in the pandoc dir. If we give it as a command line option, it's fine (because then it only uses styles, and leaves out settings.xml).

This is all from memory from a month or so back -- haven't tested it with the current version.


don't use this, creates unnecessary mistakes and code complexity
for example already here are errors:
+  inline hash512_t(byte value[]){memcpy(data, value, 8);} // sizeof(hash512_t) == 64, not 8; 'byte' is not a c-type
+  inline const uint64_t *get_data() const {return data;} // data is array of uint64_t

do access raw data simply use (char*)&hash512_t and sizeof(hash512_t). but avoid excessive use of c-code


uname -sr displays darwin 14.3.0 which technically isn't the kernel. Might want to display this information. But not under Kernel.


I'd suggest renaming addImage to image, just to be consistent with method image below. Besides, plain 'image' is a better name, more consistent with other methods in the API


https://developers.google.com/youtube/player_parameters#hd

The `border`, `color1`, `egm`, `hd`, and `showsearch` parameters, which are all only supported for the deprecated AS2 Player API, have been moved to a new section named `deprecated parameters only used in the AS2 Player API`.


Please turn off optimization,
it seems that it bypass asserts, without optimizations you get back nine expected failures.
:-1:


So just to clarify further: the retooling of the output file to produce the new reference.docx introduced footnotes into the settings.xml file, and if I then used it as a reference to produce a file without footnotes, it would get confused. If the way that information is pulled from the reference has changed, this might no longer be necessary.


oh I see. Sorry. but still IMO casting to char* is a bit more clear than to byte*


@dereuromark yes! now you can have all 3 set ;)

```php
$this->Form->create($entity, ['align' => [
    'sm' => [
         'left' => 5,
         'middle' => 5,
         'right' => 2,
   ],
    'md' => [
         'left' => 2,
         'middle' => 6,
         'right' => 4,
   ],
]]);


well done! quite easy to understand but better if you set i j meaningful(maybe by adding some notes)


there is some trailing whitespaces here (i don't know why github does not display them)
please remove them


点击按钮后得到结果前其他按钮没有真正灭活。S3老师要求的是“并行”的效果(见wiki的评论，可能没看到所以算了)。


:clap:


Please correct reviews, verify that the 9 nine tests which are failling are the good ones and place "test faillures" decorator accordingly
:-1:


You forget to add normalize and reset scss to git


Why are you using both normalize and reset?
Normalize and Reset are used for the same purpose, you should stick with only one of them.
If not, please reply with arguments.


good job! especially for the notes for every function, but better if you delete '.h' in the 'include' statement because it's c not c++ style


I tested the BBvideo BBCode on 3.1 and it appears as a link. This one produces the same link with the same attributes. I assume there's something in JavaScript that turns those links into videos but it doesn't work on my local board on 3.1 or 3.2.


@ksooo actually my first PR was much simpler and did not used the addtimertype, but I was asked to make it more flexible for pvraddons

It where simple bitflags
https://github.com/Glenn-1990/xbmc/commit/d8fea222ea0d8fc48c1b13383dfadeb632d87013#diff-d21af80cbb78a67bffb9f5ff3ee708adR120


Correct. That is why you should capture shared_ptr (look into std::enable_shared_from_this::shared_from_this() ) to prolong objects' lifespan. Connection_base uses this technique as an example.
This however, might create another problem, the "circular reference" problem. Check std::weak_ptr.


My thought was that the Player class should not have a coupling to the GUI and therefore the expeditions have to be stored elsewhere. But maybe I'm wrong, the Player class could have a method List<Expedition> GetAllExpeditions() which returns all the player's expeditions if they are stored in the Player class. I don't know what the best solution is.


General comment - while developing LWL I realized there's so much room for a dedicated piece for animations (e.g. AnimationsBuilder). Not only for creating this kind of "semantic-agnostic" animations, but also pieces of animation that have specific meanings. For example, make an object oscillate or float in the air, idle/blinking animations (they usually have the same structure 1 long frame + a yoyo sequence of short frames), enter/exit animations (e.g. instantly scale to 0 and set alpha to 0, then increase scale to 1 and alpha to 1 in x sec).

What do you think?


wrong comment


Why do you need a `.toString()` here?


wrong lable


Why the renumber?


Can we change this to 'makeCircle'? After a while I just tried to use the 'make' prefix for all methods that return pieces of the model instead of DemoBUlder


You can remove the `;` and `return`s from this method (and also this declaration via multiple return)


Idem - makeChangeState
I just think it's good to have a naming convention that let us easily recognize what methods return model pieces that are unattached to the main structure, and which ones actually change the game model.

Also general comment for future discussion - should we just separate makeXX methods into a separate class(es)?


my qomment


makeIfElse


As Pete mentioned, you are using libusb-win32. For libusb-win32, WDK
v7600.16385.1 should work. You do not need VS2013 and I think it is not
supported either.

On Sat, Mar 28, 2015 at 4:32 PM, boltnut <notifications@github.com> wrote:

> I tried to compile the libusb-sys v1.2.6.0 project using Visual Studio
> community 2013 and WinDDK v7600.16385.1. and got some error messages as
> below:
>
> 1> Unable to configure WinDDK.
> 1>C:\Program Files
> (x86)\MSBuild\Microsoft.Cpp\v4.0\V120\Microsoft.MakeFile.Targets(38,5):
> error MSB3073: The command "CD
> "D:\LibUSB1260\libusb-win32-src-1.2.6.0\projects..\ddk_make"
> 1>C:\Program Files
> (x86)\MSBuild\Microsoft.Cpp\v4.0\V120\Microsoft.MakeFile.Targets(38,5):
> error MSB3073: CMD /C make.cmd "arch=x86" "app=driver" "noclean=true"
> "outdir=D:\LibUSB1260\libusb-win32-src-1.2.6.0\projects\Win32\Debug\libusb-sys\"
> "DIR_INTERMEDIATE=D:\LibUSB1260\libusb-win32-src-1.2.6.0\projects\Win32\Debug\libusb-sys\"
> "debugmode=true"" exited with code 1.
>
> By the way, I had modified the make.cfg and NMake Include Search Path
> setting based on my own environment.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/libusb/libusb/commit/ada59db186aa95215158129db8cd42e079daff07#commitcomment-10449234>
> .
>

--
Xiaofan


Make ifElse


Closed Visual Studio Online Task [vso#4345](https://azurecom.visualstudio.com/DefaultCollection/ACOM/_workitems#_a=edit&id=4345): 'This is a test usersnap task submission'

VSO State: Done


Since this field doesn't depend on the parameters, you could extract it to a
```groovy
    private static getCurrentUser() {
        SystemUtils.IS_OS_WINDOWS ? '0'
                                  : 'id -u'.execute().text.trim()
    }
```


Seems like this should be buf.len() rather than a constant. Otherwise, if the vector is large but filled to capacity, you are going to be reallocating it a lot unnecessarily at the beginning of the read.
I think the capacity increase behavior in this function should be identical to what would happen if we used buf.push() on every byte in r (ie it simply doubles the buf's capacity every time it is filled to capacity).  This is probably what the user will be expecting, and allows them to be more flexible by reading in data of varying sizes from multiple sources and having it be efficient in every case.


Good point. I don't think the player object needs to contain or get the expeditions. Other than maybe to keep track of the score. This needs thinking about.


Эта строка не даёт сработать следующей. Лучше вообще убрать её. В таком виде, кроме того, что она мешает операционной системе правильно выбрать интерпретатор, так ещё и совершенно неинформациоона.

Можно убрать добавление этой строчки по умолчанию в настройках PyCharm


Should we check if initGame is null? Then trigger exception, just to help understanding the preconditions.


Very nice additions in this commit, overall


Заполняющий символ тоже лучше вынести в константы. Лучше как поле класса `Field`


Yes, as per commit 152490ae1656352706cdee021b1adbc10a8bc0e2


My idea for having both TakeCard() in Player and TakeTopCard() in Deck is to reduce couplings. The Game class could then be the central hub and give a card to a player by calling activePlayer.TakeCard(deck.TakeTopCard()); which gives the effect of no coupling between the Player class and the Deck class. The idea is that everything goes through the Game Class. I don't know if this is a good solution, maybe it's overly complicated?


Create KisaCookie
Beyler bi sorum var ya. cookieGetir methodu varya ikinci method. İşte orda sırasıyla iç içe if-for-if döngüsü var.
Hani burda return cook demiş ya onun altında da return null demiş. bu methodda if döngüsünden sonra eğer doğruysa cook'a return
oluyo. peki sonra her koşulda yine null çalışmıyomu ? yoksa ifden sonra duruyo mu method? break de koymamış hoca.
YANİ BENCE HERTÜRLÜ BU METHODDAN NULL GELİR HACI


Good idea!


How about if (!this.gameDir.isDirectory() || !this.pluginsDir.isDirectory() && !this.pluginsDir.mkdirs()) ?


Обозначение яблока лучше занести внутрь класса `Field`, как и обозначение змеи


Как я уже говорил, не стоит называть таким образом поля, обозначающие координаты змеи


Thanks! I noticed I broke it, but didn't figure out the reason


Тогда уж `put_apples`


Плохое имя для блока.
Даже как для тестового задания.
Привыкай писать более "descriptive" имена (например, блок two-col-fixed) и т.п.

Такое название допустимо, если ты будешь делать свой Grid System и это будет означать первый в списке элемент\колонку\блок и т.п.


configure: loading site script /home/main/x86_64/include/site/x86_64
checking for a BSD-compatible install... /usr/bin/install -c
checking whether build environment is sane... yes
checking for x86_64-openwrt-linux-strip... x86_64-openwrt-linux-uclibc-strip
checking for a thread-safe mkdir -p... /bin/mkdir -p
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking whether make supports nested variables... yes
checking whether to enable maintainer-specific portions of Makefiles... no
checking build system type... x86_64-pc-linux-gnu
checking host system type... x86_64-openwrt-linux-gnu
configure: Configuring for alpine 2.20 (x86_64-openwrt-linux-gnu))
checking for x86_64-openwrt-linux-gcc... x86_64-openwrt-linux-uclibc-gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... yes
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether x86_64-openwrt-linux-uclibc-gcc accepts -g... yes
checking for x86_64-openwrt-linux-uclibc-gcc option to accept ISO C89... none needed
checking for style of include used by make... GNU
checking dependency style of x86_64-openwrt-linux-uclibc-gcc... gcc3
checking for x86_64-openwrt-linux-uclibc-gcc option to accept ISO C99... -std=gnu99
checking for x86_64-openwrt-linux-uclibc-gcc -std=gnu99 option to accept ISO Standard C... (cached) -std=gnu99
checking whether make sets $(MAKE)... (cached) yes
checking whether ln -s works... yes
checking for gawk... (cached) gawk
checking for x86_64-openwrt-linux-ranlib... x86_64-openwrt-linux-uclibc-gcc-ranlib
checking how to print strings... printf
checking for a sed that does not truncate output... /home/main/x86_64/staging_dir/host/bin/sed
checking for grep that handles long lines and -e... /home/main/x86_64/staging_dir/host/bin/grep
checking for egrep... /home/main/x86_64/staging_dir/host/bin/grep -E
checking for fgrep... /home/main/x86_64/staging_dir/host/bin/grep -F
checking for ld used by x86_64-openwrt-linux-uclibc-gcc -std=gnu99... x86_64-openwrt-linux-uclibc-ld
checking if the linker (x86_64-openwrt-linux-uclibc-ld) is GNU ld... yes
checking for BSD- or MS-compatible name lister (nm)... x86_64-openwrt-linux-uclibc-gcc-nm
checking the name lister (x86_64-openwrt-linux-uclibc-gcc-nm) interface... BSD nm
checking the maximum length of command line arguments... 3458764513820540925
checking whether the shell understands some XSI constructs... yes
checking whether the shell understands "+="... yes
checking how to convert x86_64-pc-linux-gnu file names to x86_64-openwrt-linux-gnu format... func_convert_file_noop
checking how to convert x86_64-pc-linux-gnu file names to toolchain format... func_convert_file_noop
checking for x86_64-openwrt-linux-uclibc-ld option to reload object files... -r
checking for x86_64-openwrt-linux-objdump... x86_64-openwrt-linux-uclibc-objdump
checking how to recognize dependent libraries... pass_all
checking for x86_64-openwrt-linux-dlltool... no
checking for dlltool... no
checking how to associate runtime and link libraries... printf %s\n
checking for x86_64-openwrt-linux-ar... x86_64-openwrt-linux-uclibc-gcc-ar
checking for archiver @FILE support... @
checking for x86_64-openwrt-linux-strip... (cached) x86_64-openwrt-linux-uclibc-strip
checking for x86_64-openwrt-linux-ranlib... (cached) x86_64-openwrt-linux-uclibc-gcc-ranlib
checking command to parse x86_64-openwrt-linux-uclibc-gcc-nm output from x86_64-openwrt-linux-uclibc-gcc -std=gnu99 object... ok
checking for sysroot... no
checking for x86_64-openwrt-linux-mt... no
checking for mt... mt
configure: WARNING: using cross tools not prefixed with host triplet
checking if mt is a manifest tool... no
checking how to run the C preprocessor... x86_64-openwrt-linux-uclibc-gcc -std=gnu99 -E
checking for ANSI C header files... yes
checking for sys/types.h... yes
checking for sys/stat.h... yes
checking for stdlib.h... yes
checking for string.h... yes
checking for memory.h... yes
checking for strings.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for unistd.h... yes
checking for dlfcn.h... yes
checking for objdir... .libs
checking if x86_64-openwrt-linux-uclibc-gcc -std=gnu99 supports -fno-rtti -fno-exceptions... no
checking for x86_64-openwrt-linux-uclibc-gcc -std=gnu99 option to produce PIC... -fPIC -DPIC
checking if x86_64-openwrt-linux-uclibc-gcc -std=gnu99 PIC flag -fPIC -DPIC works... yes
checking if x86_64-openwrt-linux-uclibc-gcc -std=gnu99 static flag -static works... yes
checking if x86_64-openwrt-linux-uclibc-gcc -std=gnu99 supports -c -o file.o... yes
checking if x86_64-openwrt-linux-uclibc-gcc -std=gnu99 supports -c -o file.o... (cached) yes
checking whether the x86_64-openwrt-linux-uclibc-gcc -std=gnu99 linker (x86_64-openwrt-linux-uclibc-ld -m elf_x86_64) supports shared libraries... yes
checking whether -lc should be explicitly linked in... no
checking dynamic linker characteristics... GNU/Linux ld.so
checking how to hardcode library paths into programs... immediate
checking whether stripping libraries is possible... yes
checking if libtool supports shared libraries... yes
checking whether to build shared libraries... yes
checking whether to build static libraries... yes
checking for ar... /usr/bin/ar
checking for rm... /bin/rm
checking for cp... /home/main/x86_64/staging_dir/host/bin/cp
checking for ln... /bin/ln
checking for ls... /bin/ls
checking for sed... (cached) /home/main/x86_64/staging_dir/host/bin/sed
checking for make... /usr/bin/make
checking whether NLS is requested... no
checking for msgfmt... /usr/bin/msgfmt
checking for gmsgfmt... /usr/bin/msgfmt
checking for xgettext... /usr/bin/xgettext
checking for msgmerge... /usr/bin/msgmerge
checking for ld used by GCC... x86_64-openwrt-linux-uclibc-ld -m elf_x86_64
checking if the linker (x86_64-openwrt-linux-uclibc-ld -m elf_x86_64) is GNU ld... yes
checking for shared library run path origin... done
checking for CFPreferencesCopyAppValue... no
checking for CFLocaleCopyCurrent... no
checking whether to use NLS... no
checking option: dmalloc enabled... no
checking option: debugging is enabled... no
checking option: optimization is enabled... yes
checking option: mouse support enabled... no
checking option: quotas enabled... no
checking option: From changing enabled... yes
checking option: background post enabled... yes
checking option: keyboard lock enabled... yes
checking option: from encoding enabled... no
checking for sendmail... /usr/sbin/sendmail
checking for inews... no
checking for passwd... /usr/bin/passwd
checking for hunspell... no
checking for aspell... aspell
checking for aspell... /usr/bin/aspell
checking for hunspell... no
checking for aspell... aspell
checking for aspell... /usr/bin/aspell
configure: Excluding LDAP Support
configure: Excluding TCL Support, and thus Web Alpine Support
checking for setupterm in -ltinfo... no
checking for setupterm in -lncurses... yes
checking for library containing dlopen... -ldl
checking Openssl library version >= 1.0.1c... configure: WARNING: cross compiling: not checking
checking for library containing TLSv1_2_client_method... no
checking for library containing SSLeay... no
configure: error: crypto library NOT found
make[3]: *** [/home/main/x86_64/build_dir/target-x86_64_uClibc-0.9.33.2/alpine-ssl/alpine-2.20/.configured_yyyyyyyyn] Error 1
make[3]: Leaving directory /home/main/x86_64/feeds/packages/mail/alpine' make[2]: *** [package/feeds/packages/alpine/compile] Error 2 make[2]: Leaving directory/home/main/x86_64'
make[1]: *** [/home/main/x86_64/staging_dir/target-x86_64_uClibc-0.9.33.2/stamp/.package_compile] Error 2
make[1]: Leaving directory `/home/main/x86_64'
make: *** [world] Error 2


Doesn't really matter, does it? :|


Lookin good! Should the Discard(Card card) and TakeTopCard(string colour) in Expedition be called from the player class? Is it easier and more modular to call it from the Game class?


Если ты планируешь, что змея будет расти, то стоит назвать эту функцию, как `put_base_snake` или что-то подобное, так как здесь устанавливается лишь одна точка змеи.


Вот это ты понял не правильно.
Для этого задания у враппера (.section) не должно быть фиксированной ширины и если менять ширину окна браузера, должно быть видно как Fluid блоки динамически меняют размер


r=jdm


not really xD


my comments are the same as liao, and i also realize my fault from the comments 👆.


Это нужно сделать не в движении змеи, а в создании змеи. Как-то логичнее получается


You were too lazy to check what happens, so you put it on the to-do list? Shitty coder.


Здесь стоит оперировать лишь с тем, что записано в списке `SNAKE`


We have to merge it into master. It's not there and thus this request type fails to be created.
Push this along the other changes for the next testbed RPM please @ticoann


Изменение длины змеи никак не отображается на списке `SNAKE`, хотя, по логике, должно бы


Those optimizations look much more safer, I have to try them :P


perhaps, you can add a Prints usage line 69


Лучше не использовать переменную `snake_length`, а вычислять этот параметр, учитывая, что змея хранится поточечно в списке


Not sure ExecutableDemoBuilder is the best choice. Also, I wonder if the separation between Engine and Editor Demo Builder is still a good idea. To me, it is just confusing, especially now that we have a separate project for this. I'd consider merging these two pieces (DemoBuilder + EditorDemoBuilder) into a single, Builder class.

What do you think?


Больше двух недель никаких исправлений и вопросов


There's sth wrong here. I guess es.eucm.ead.engine should be out


Ah okay. Honestly, the rest and web interface does in a way replace the form interface.


ok. I see why you did that. IMHO, we should try to make tests for TileContainer and that will probably tell us what we need to embed or not. We will see when it will be done what it needed or not and if we can gather them again.


I quickly wrote this code but haven't tested it yet, I'll try to have a look at it tomorrow.


```java
imageUtils = new DesktopImageUtils();
this.gameAssets = new GameAssets(Gdx.files, imageUtils);
```
??


@Grz3s are you sure about this:
```sql
('-D','Jane will love this one!','0','0','10','20244','0'),
('-E','Oooh, a shiny one!','0','0','10','20244','0'),
('-F','I think I can see the Sunwell from here!','0','0','10','20244','0'),
('-G','Can you really hear the ocean from one of these shells?','0','0','10','20244','0');
```

Are you really using Language = 10 (LANG_THALASSIAN) or is that the emote (EMOTE_STATE_DANCE), or is this a mistake?


Ah good catch.


I was about to make a pull request ahah. Good job :)


Thanks Alan I didn't notice it went to other branch


Thanks for both of these.


Just curious - what's the difference between forceExit = true and forceExit = false?


Function name is now "lms6002d_ctrl_impl" instead of "db_lms6002d", so we should update the log output.


Same as above.


It seems it's only needs linking if the function is actually called, so should I move it?


Uhm... Even though (for some bizarre reason) GCC seems to accept it, i think the correct function to call would be `std::copy()`, since `::copy()` might be a user-defined function.


I've done some tests and I'm unable to make the text wrap correctly. Leaving it to your expertise... :+1:


Hi, This is my first Git repo.


Очень плохое имя.
Если ты видишь\хочешь написать <tag class=tag/> (например, <section class=section> значит что-то не так.

В данном случае можно было бы добавить какое определение, что за секция: main-section, wrapper, page-layout и т.п.


all tests pass:
success: http://build.servo.org/builders/linux1/builds/1199
success: http://build.servo.org/builders/linux2/builds/1197
success: http://build.servo.org/builders/android/builds/1233
success: http://build.servo.org/builders/mac1/builds/1412
success: http://build.servo.org/builders/mac2/builds/1415
success: http://build.servo.org/builders/gonk/builds/379


MMm you sure this works? I presume this will return null unless snapshot is in same package as the class


Не семантично, если ты пишешь заголовок, то используй тег для заголовков.
Либо хотя бы параграф. span совсем не подходит.


Заголовок, не span


This is so great. I didn't realize that the problem was that the beam specifier wasn't actually being returned.

Awesome.

I've removed the to-do notes and just committed.


After this commit some test of Storage are broken because of the different filenames.
This will be fixed later.


Why package builder.converters? I'd expect this just to be in xx.ead.builder


gggg


Nice one


В принципе это дело привычки\вкуса.
Но лучше не ломать стереотипы и привычные для других имена.
Лучше как-то так:
.clearfix
.cf (так вроде пишут в bootstrap)


Sip


I need to check this - just in case it breaks RepoBuilder


fast-forwarding master to auto = 0d3574b5


Не делай такого никогда.
Это очень грязный хак для inline-block. С ним больше проблем, чем пользы.


Wondering if this may be a problem. Builder uses editor, so this adds an indirect dependency between engine and editor.


:+1:
the change to .container doesn't seem to affect much, so the change is totally ok.

The burger menu icon is still in the middle of the header for me on some smaller screen sizes; maybe it could be on the right hand side at all times?


saw approval from jdm
at https://github.com/Ms2ger/servo/commit/1ecfc9f5829febf44809ea991e87114314b63148


merging Ms2ger/servo/wpt_20150317 = 1ecfc9f5 into auto


Ms2ger/servo/wpt_20150317 = 1ecfc9f5 merged ok, testing candidate = b8ea10bf


Done!


These are the type of commits I like! Nice and easy to review :)


I'd move the message into a module level constant and also import that one in the tests.


Copy paste fail by me. I changed Open() to return the error code instead of true on success.
so if (Open()) means an error occured and not connection was successful opened.