Intent with tabs, align with spaces.


Can you add a `[type="number"]` to the `:focus` state as well?


Same.


Same as above.


develop (mjwitte) - x86_64-MacOS-10.9-clang: OK (1285 of 1285 tests passed)

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


Would be great if you could also add an example to the README to explain how to use this.


Yes, sure :) . But don't have a CAS server for real testing :)


Here I would probably be sensible to move the body of the long lamda function to a separate function that is called by the lamda function. I prefer to keep lamda functions as short as possible to keep the code easy to understand. But this is just cosmetics and and personal preference.


this is for style.


odds of this causing a real world problem are very close to 0 :) probably should get the length from site settings or something.


I've been doing some testing and since Google has pushed this commit https://github.com/imoseyon/leanKernel-shamu/commit/d6d53df36e665584912705e539fed0883933571a this commit doesn't seem to need to be reverted. Battery stats have been normal for myself and testers with combination of this commit added back in and the commit I linked to.


참고로 말하자면 혼자공부했음.


Rename to ```Apply_functor```? Maybe, it makes sense to grep for "Handle_".


Indentation/alignment issue. There are more occurrences below.


shouldn't be EOL?


Thanks for spotting! Please provide a clean-up commit.


Many thanks @reporobot, your collaboration is most welcome and appreciated.


What happens if the expiry is shorter than the content duration? Does it just stop streaming?

> On Sep 1, 2015, at 11:04 PM, Roonglit Chareonsupkul <notifications@github.com> wrote:
>
> Once the file is transcoded, it's up to us on managing access to the file. I thought that we need a streaming server, but it seems we don't need it. The way I'm doing now is just letting Zencoder push the transcoded file back to S3, and users will download it themselves. So, the expiring time would be the time for our user to get the video through their video player. We just need to make sure that it's enough for them to watch the content.
>
> —
> Reply to this email directly or view it on GitHub.
>


Why change the ordered list to inordered?


A [0,4] test has a bigger chance for passing than a [0,5] test.

The point is that if the current one fails with yellow (more wins than losses), we will never now if it would have passed with [0,4]. If the current test succeeds then all is well and you can queue the LTC test with [0,4].

I'm afraid that this test will fail with yellow, so we will have wasted some resources. I suggest adding the same patch to the queue again with [0,4] and leave the other one parked, till your new test is approved. You can than delete the old one.


I think that we need to copy the interface name because we printing it.


That explains why my `when`/`otherwise` blocks weren't working - forgot to wrap them in `choose`!


why do they need to be sorted? And does it make sense to sort them by pk?


You are very welcome! Thanks for the m3u8 library! When I have time I could fix and make those receiver functions (*MasterPlaylist.decode and *MediaPlaylist.decode) DRY. Also some playlist and tests around this would be great.


93651760-RowPerObjectReport (JasonGlazer) - Win64-Windows-7-VisualStudio-12: OK (1813 of 1813 tests passed)

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


Yep, that's it! I :heart: XML


let me try it. I'll get back to you in two minutes.


Then just change the line when the group is formed to log queued roles and matched roles. there you can debug if there was a problem. With this change anyone debugging any server with 40 members on queue will have a nightmare to check logs


Video.js will try to download the content, but if the access is expired, it will show that the video can't be downloaded, and user can't play the content. So, at least, we need to provide enough time for Video.js to download the entire content.


this breaks linux builds if libbluray is older, i.e in our PPAs


Fun fact: Belgian names starting with "van" are written with a capital letter, so it's "Van den Bergh". No-one except Belgians know I guess.


Would it make sense to add a third specialization for associative arrays:

```{D}
private JSONValue defaultToJSONImpl(T, SerializationOptions options)(in T object)
    if (isAssociativeArray!T &&  __traits(compiles, (in T t) { JSONValue(t);}))
{
  return JSONValue(object);
}
```

That might be more performant than converting every key/value pair ourselves? Or does that not do the right thing?


Проверил, действительно так. Все сделаю как ты говоришь.


K no worries I'm going to bed. Basically we just want to lock it down as
much as possible, I don't know how that is normally done with video

On Tue, Sep 1, 2015 at 11:34 PM, Roonglit Chareonsupkul <
notifications@github.com> wrote:

> let me try it. I'll get back to you in two minutes.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/lockstep/cdn_demo/commit/087a0b1f062decd605030beb08c972ef396aee9e#commitcomment-13015843>
> .
>


Thank you for the great explanation. It though I have to put them in the
main folder. Thank you for the great tool.

On Wed, Sep 2, 2015, 8:27 AM Björn Hurling <notifications@github.com> wrote:

> I see. You can just upgrade the tool without a problem.
>
> This change only affects where the bootstrap phase PUTS the initial play
> store data.
>
> The publish phase still respects flavours and buildTypes. Let's say you
> have your internal buildTypes plus a free and a paid flavour. Then play
> data is merged in the following order (left wins):
>
> freeInternal > internal > free > main
>
> In your case without flavours it's even easier:
>
> internal > main
>
> This is exactly the same order as in the normal resource merging process.
>
> You can see details in L71-L78
> <https://github.com/Triple-T/gradle-play-publisher/blob/d4c7b56185cc00593ed38141b23ae54741844bf1/src/main/groovy/de/triplet/gradle/play/PlayPublisherPlugin.groovy#L71>
> in PlayPublisherPlugin.groovy.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Triple-T/gradle-play-publisher/commit/d4c7b56185cc00593ed38141b23ae54741844bf1#commitcomment-13015204>
> .
>


Aha, you are using the registry and the key "packupdates" to store the modification date!
In that case the current update solution will probably work.

You should include this commit in the planned release. Then the 1.9 update later this year will be a good test cast for us, if updating from GitHub works like intended.

In my debug environment I can not watch Qt variables. Therefore it is quite hard to do real debugging.


Oh, you mean if BF_ASM is 1 we can't do X3? I didn't think of that.


what the hell is this.


No. phpBB 3.0 is EOM. End of Life is in or after November 2015.


I think its better to move the connectDB method to a separate file that handles db related stuff than having the function and calling it from the same location.


.DS_Store should be in your own global gitignore


perhaps: "let new joining nodes be promoted while convergence..."

(joining is already possible)


描述错误

这一版本的Category类是实现递归将一维数组根据pid组合新的顺序一维数组


similar here, sorry to be picky


Yes, The notifier block call the rxe_net_up when there is NETDEV_UP event and it will exit if there is not rxe device already added ,PSB

https://github.com/SoftRoCE/rxe-dev/blob/rxe_submission/drivers/infiniband/hw/rxe/rxe_net.c#L566


Why have a dedicated folder for birthdays? I would recommend users.birthday


abc


well, to be myself


we should perhaps also adjust the diagram? what is the source of that?


sneaky bug fix? ;)


Nice!


@Grinnz: why do I want to use ```Mojo::SQLite::Database::last_insert_id()``` instead of the code above?


perhaps we should add one more diagram, that shows how it is when the feature enabled?


Thx. I may have just gone overboard with the styling fix but this is fixed in the next commit. ;)


when I Change the input video name "nm" to others eg. nm = 'soccer_juggle',
![image](https://cloud.githubusercontent.com/assets/5250272/9624708/c4187954-5181-11e5-8c09-ac476f727df0.png)
the *.ppm images in ../save/misv/vox folder do not exsit
would you please tell me how to get this problem. thank you


I'm not sure it needs to be there explicitly, but it's a dependency of the oauth2-client class so you'd end up with it anyway.


Probably not without adding a whole lot more instances... but I don't think UndecidableInstances is all that terrible anyhow.


I suppose this is for fixing the labels in the web frontend? Please indicate the reason for this change.
Are there no users of prepare.release.boundaries() that can break by this change?


Ours make it to the boot animation then randomly reboot in the middle of it. Did you have a similar problem?


How far along is this? I'd like to get a functioning pretty printer merged in, at least eventually.


get==

UserInfoModel *userModelObj;

        NSData* data = [[NSUserDefaults standardUserDefaults] objectForKey:@"UserInfoKey"];
        userModelObj= [NSKeyedUnarchiver unarchiveObjectWithData:data];

        if ([[[NSUserDefaults standardUserDefaults] dictionaryRepresentation].allKeys containsObject:@"UserInfoKey"]) {
            userModelObj= [NSKeyedUnarchiver unarchiveObjectWithData:data];
        }
        NSLog(@"picPathvvv -->%@",userModelObj.picPath);

Set+++++++===>

-(BOOL)saveUserNameInDefaults
{
    UserInfoModel *modelObj = [UserDataManager getUserDataModel];
    if(!modelObj)
    {
        NSLog(@"No userInfoModel object in saveUserNameInDefaults method");
        return NO;
    }
    NSData *storedDataa = [[NSUserDefaults standardUserDefaults] objectForKey:@"WebServiceLoginJson"];
    NSMutableDictionary *jdic = [NSKeyedUnarchiver unarchiveObjectWithData:storedDataa];

     NSString *activity_level = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"activity_level"]];
     NSString *age = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"age"]];
     NSString *dateofbirth = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"dateofbirth"]];
     NSString *datetime = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"datetime"]];
     email = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"email"]];
     NSString *fitnessgoal = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"fitnessgoal"]];
     NSString *gender = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"gender"]];
     NSString *height = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"height"]];
     NSString *heightunit = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"heightunit"]];
     NSString *ID = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"id"]];
     NSString *mewardid = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"mewardid"]];
    // NSString *message = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"msg"]];
     NSString *UserName = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"name"]];
     NSString *pic_path = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"pic_path"]];
     NSString *serverId = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"serverId"]];
     NSString *targettime = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"targettime"]];
     NSString *targetweight = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"targetweight"]];
     NSString *weight = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"weight"]];
     NSString *weightunit = [NSString stringWithFormat:@"%@",[jdic objectForKey:@"weightunit"]];

    if ([gender isEqualToString:@"0"])
    {
        Gender=@"male";
    }
    else{
        Gender=@"female";
    }

    if ([activity_level isEqualToString:@"0"])
    {
        ActivityLevel=@"Sedentary";
    }
    else if ([activity_level isEqualToString:@"1"])
    {
        ActivityLevel=@"Low Active";
    }
    else if ([activity_level isEqualToString:@"2"])
    {
        ActivityLevel=@"Active";
    }
    else if ([activity_level isEqualToString:@"3"])
    {
        ActivityLevel=@"Intense Active";
    }
    [modelObj setEmail:email];
    [modelObj setUserName:UserName];
    [modelObj setGender:Gender];
    [modelObj setHeight:height];
    [modelObj setHeightUnit:heightunit];
    [modelObj setAge:age];
    [modelObj setWeight:weight];
    [modelObj setWeightUnit:weightunit];
    [modelObj setPicPath:pic_path];
    [modelObj setTargetWeight:targetweight];
    [modelObj setTargetTimeInWeeks:targettime];
    [modelObj setMewardID:mewardid];
    [modelObj setServerID:serverId];
    [modelObj setDateOfBirth:dateofbirth];
    [modelObj setDateTimeInMilliSeconds:datetime];
    [modelObj setDataBaseId:ID];
    [modelObj setFitnessGoal:fitnessgoal];
    [modelObj setActivityLevel:ActivityLevel];
    [modelObj setIsUserLoggedIn:YES];

    NSLog(@" saveUserNameInDefaults method");
    return [UserDataManager saveUserInfoDataModel:modelObj];
}


I had also assumed this would just follow the same path, and that the TCPLoadBalancer interface/functions would just become the LoadBalancer interface/functions. I still think that approach is a better fit, as we can make sure that errors about not supporting UDP get properly propagated back to the user without too much extra effort.

Also, can you move the computeHostTags bit out into a separate commit/PR? It's pretty orthogonal to the UDP change and that'll let us get it in faster.


Wohoo ! thank you @phellmes , I should remember to 'star' my own repo to receive notification. I'll merge and render the files this morning.
( note: and about the temporary mailing-list : https://groups.google.com/forum/#!forum/peppercarrot-translation-team )


Acked-by: Wolfgang Mauerer <wolfgang.mauerer@oth-regensburg.de>


Fixed in 4919a42


101884090--NewVenting-Availability-Schedule (lgu1234) - Win64-Windows-7-VisualStudio-12: OK (1814 of 1814 tests passed)

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


Proofread and approved!


ERROR: Nothing PROVIDES 'enigma2-plugin-settings-vhannibal.13e.19e.30w' (but /home/buildserver/openmips42-DEV/build-enviroment/meta-oe-alliance/meta-oe/recipes-oe-alliance/enigma2-channelsettings/channelsettings-enigma2-meta.bb DEPENDS on or otherwise requires it). Close matches:
  enigma2-plugin-settings-vhannibal13e.19e.30w
  enigma2-plugin-settings-vhannibal.quad.13e.19e.23e.28e
  enigma2-plugin-settings-vhannibal.quad.7e.13e.19e.42e


Done


This works only by change because the last assignment (to variable return) is used as return value.
I suppose you mean return(package...)?


Good catch, but I think this is a non-issue, really.

All this does is short-circuit lookups for the admin -- yes, that's probably not enough in the long run if we're looking to get bigger optimizations via short-circuiting -- but considering, say, `shoop/simple_cms/admin` having `foo.jinja`, as long as a `mystery` theme does not have the file `mystery/shoop/simple_cms/admin/foo.jinja`, all is dandy.


Thank you!


buildReilJar


Add space after ":"


That'd make it harder to debug plugin disappearance issues (which I foresee happening) in production environments...


related #990


Looks like this commit fixes this open issue: https://github.com/spring-cloud/spring-cloud-netflix/issues/488


I'd suggest to "shout" a bit less :) How about "Warning: Build target not yet implemented."


Basert på kodene som er brukt i MARC-postene, ja. Stort sett brukes ISO 639-3-kodene, men de finnes noen unntak, særlig der Iso-kodene er endret siden 639-2. Baskisk språk er et annet unntak.


Please name the function filter.installed.packages() or something according
to our style guide: https://github.com/siemens/codeface/wiki/Style-Guide


nazdar viem ti poslat aj ten subor ale az budem doma a spomeniem si na to.
ale ma to hacik me to bez tak nefungovalo. Ked dam nejaky link na fast share nastavim si tam
hesla k mojmu uctu tak to vypise : error ...
tak ze si bez tak myslim ze to nefungtuje ale tipek co to vytvoril mi hovoril ze jemu to chodi ?

> ** Sat, 29 Aug 2015 00:31:17 -0700 - "feixm1/fastshare.host" <reply@reply.github.com>, "feixm1/fastshare.host" <fastshare.host@noreply.github.com> **
>
> Dakujem za odpoved, stiahnute som to mal ale nevedel som kde a ako to spustit... a ked nemam linux, nevedel by si mi rovno poslat ten subor? Ci je potrebne to priamo u mna spustit?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/feixm1/fastshare.host/commit/a2f38f103bf5d0e2c84253c471c3b32bd86a5c2a#commitcomment-12960361


When using this, you don't have to manually update the composer.json file, we should update the README to reflect this change


Signed-off-by: Wolfgang Mauerer <wolfgang.mauerer@oth-regensburg.de>
(with the proposed changes)


Hi @purple-sun . You should use ghb_stream_win.exe to produce these pictures (.ppm) first to save the folder.


General comment: I very much appreciate the effort, but I'm not sure we should require anything besides a JVM and Maven to run the build. OTOH, can Gradle be downloaded by Maven as well?
My worry is that the build process will be hard to understand with many moving parts (as in "So this Maven thing downloads some stuff and then invokes another build system to do the actual work?"). The Gradle scripts themselves look easy enough to understand, though.


good, it's as same as a file in src dir


I did not know the "way of doing things". I'll change it later


94006714-MultipleTowers (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1818 of 1818 tests passed)

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


Please check in the git logs if the main author is Stefan Weber. Might also be Johannes Ebke.

Acked-by: Wolfgang Mauerer <wolfgang.mauerer@oth-regensburg.de>


Can we have IntelliJ automatically output its classes to target/classes/ instead (via some magic file in the project) or does that need to configured each time you import the project?


これってxcode開発するときのテンプレ的な感じですか？


Please try to keep to one logical change per commit. It's easier to review that way.


Add have the visibility.


If you change the argument list here, you also need to change it for

    def hg_branch_iter_local(repo):

at line 46 here.

I ran into this calling the hg_branch_iter_local function without the second argument and fixed it with a new function argument:

    def hg_branch_iter_local(repo, _):

The python interpreter is somehow not needed there.

Stacktrace:

    File "/var/lib/jenkins/.pyenv/versions/2.7.6/lib/python2.7/site-packages/jenkins_autojobs/hg.py", line 59, in list_branches
    return branch_iter(config['repo'], python)
    TypeError: hg_branch_iter_local() takes exactly 1 argument (2 given)


Slightly curious wording. Please use "Working directory" or something along these lines instead of `pwd`.
Is the condition "should" or "must"?


@vcunat: Thanks.


Shall we maybe just leave it out for 5.0 for now? The amount people on 5.0 fornexus is very low (0 to 4 downloads per day)
At a later moment we could add webview to the 5.0 package, but let it be disabled by default, and its keyword/installation only be triggered when it is detected that a 'fornexus' installation is applied.


require needs an additional composer update??


should be extension API


I think we will also need to keep the abort mechanism in place, if really a NON-opengapps package was found, and neither a 'fornexus' situation applies.


Actually: Neither. Some of these scripts can be run in a different working directory, others must be run inside.


any idea why you reverted the apiman version change?


@ccx-testerbot runtests


It looks like this is redundant.
@Resource is JSR-250 but is not Guice compatible (we use JSR-330 for injection).
Therefore you should replace @Resource by @Inject @Named(name = "sieverepository") to be fully JSR-330 without breaking anything.
Spring is supposed to support this.
References :
http://atinject.googlecode.com/svn/trunk/javadoc/javax/inject/Named.html
http://docs.spring.io/spring/docs/3.1.x/spring-framework-reference/html/beans.html#beans-standard-annotations-limitations


src/TestBundle/Resources/config/doctrine/Proyecto.orm.php is 'php' format, a quite mess :-1:  Doctrine always is easier with 'annotation' format.


This is very useful work, thanks!
Acked by: Wolfgang Mauerer <wolfgang.mauerer@oth-regensburg.de>


Not it does not: in the README, we state that one need to add `"avanzu/admin-theme-bundle": "~1.3"` to his composer.json, and then run `php composer.phar require avanzu/admin-theme-bundle`.
Actually, using `php composer.phar require avanzu/admin-theme-bundle` adds the line in the composer.json file if it doesn't exist, and then downloads and installs it. So we don't need to add the line manually


Windows is a corner case, but this commit introduces a change for all platforms, even when symlinks are safe. Please test if we are working in a Windows environment, and only then use --no-bin-links


You forgot to specify a branch, so I can't help you yet... Please try again by using "runtests {branch name}".


i see!


Autodesk_Performance (DeadParrot) - i386-Windows-7-VisualStudio-12: OK (1814 of 1814 tests passed)

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


Related to network_type, Correct fixed.

Related to link-local ipv6 addresses, i don't think we needed, because the addresses filed here used to build IPv4/IPv6 header which isn't require filling the IPv6 scope.


Please briefly document the rationale for this change in the commit description.
Missing newline at EOF.
Otherwise: Reviewed-by: Wolfgang Mauerer <wolfgang.mauerer@oth-regensburg.de>


Ok


Acked-by: Wolfgang Mauerer <wolfgang.mauerer@oth-regensburg.de>


I'm planning to fix some issues in documentation so I can handle this


It's a bad practice to mix different languages. I recommend you to apply class names always in english.


Please document the rationale for this change in the commit
Acked-by: Wolfgang Mauerer <wolfgang.mauerer@oth-regensburg.de>


Do the same with attribute names.


91977808_LBNL_NewVRFModel (rongpengzhang) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1294 of 1294 tests passed)

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


Es necesaria para los plugins de cake 2, o al menos cuando el proyecto en github no tiene el mismo nombre del plugin. En Cake 3 no es necesario, de hecho, como has visto, genera problemas.


I would not define a new action `all`, but instead inside the `get_lesson` only set `@lesson` if the parameter is present. It will not be present for the global route. And in the index just test if `@lesson` is set, and set `@learning_materials` accordingly. Also in your view I am guessing very little changes between the global index and the specific index, and you could do this conditionally (again: whether or not `@lesson` is present).

The reasons:

* the meaning of `index` is to show a list [of learning materials in this case], independent if there is a limiting scope or not
* less duplication of code hopefully


And removing kube3d?


@ccx-testerbot runtests master


This whole code needs to go to `Xccdf::Rule` class.

See there is `xccdf_rule_get_severity` is not called `xccdf_item_get_severity`. If you call Group.severity it could have segfault.


On 02/09/2015 09:12, Andreas Ringlstetter wrote:
> Actually: Neither. Some of these scripts can be run in a different
> working directory, others must be run inside.
then it's preferrable to enforce this via "must" (except there are
any benefits running some from a non-root location) --WM
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Ext3h/codeface/commit/18b18f72c0e01ffcafdb6516f7c01571d204f944#commitcomment-13016345>.
>


What do you mean by that? :D


for test


@majianpeng `osd_read_eio_on_bad_digest` was removed in #5173 . maybe we can just clear the outdata and set result to `-EIO`?


test comment


test comment2


Blame the bot. I still got no idea how it managed to get this commit.


Thank you for the request. It has been added to the queue (id: 86). To see the progress you can go to the [status](http://ccextractor.canihavesome.coffee/test/86) page. Please note that depending on the current queue, it could take a while before any results will be visible. In each case I will report back here once the tests are done.


That actually can't work:
#if defined function can never give useful answer; 'function' is not known at the time of the preprocessor call (unless something has a #define set_escdelay... then it's no longer a function name).

with ncurses 6.0, we MUST use set_escdelay, as ESCDELAY = 10 results in an error (ESCDELAY is expanded by macros to _nc_ESCDELAY, which is a function name... and assigning 10 to a function obviously also does not work)


you should only have the other constructor.
Default provider is of Module responsibility


you must install the module like other ones


this sutie.json is not working now, it is a temporary file. Cause pack script will be re-construct, at that time this project will be integrated. Now you can build this project following the README.


CassandraJamesServer should take the Module that defines the whole wiring


Oh, surprised you found this.

`Enum`/`Enumeration` is not supported yet. Also, it would be nice if the output was a bit more pretty, ie. formatted -- I would be wanting to check in the generated `.proto` into version control for human reference. I could probably spend a bit more time on it over the next week or two.


Autodesk_Performance (DeadParrot) - Win64-Windows-7-VisualStudio-12: OK (1814 of 1814 tests passed)

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


@doches Oopsie, I forgot that part, was so focusing on fixing that segfault, and I didn't even think about the possibility of updating label. Will fix it soon.


you have to construct the server with a DefaultModule overridden with some `Modules.override(new TestingModule())`


why a named param here?


you should use constant for names


@Inject private


no new, just requires it


I cannot compile the book since this change.
my pillar command in PATH is not used anymore


> Because they made a mistake naming the structure member.

As WinPcap does not support libpcap 1.7.3 let's rename those fields?

> Use a better editor.

Yeah ))) I have better idea. Maybe it will be helpful here. There is AStyle application. It can replace Tabs with Spaces or convert Spaces to Tabs. Also it can convert \r\n to \n and vice versa Also it can perform code formatting. If developer runs this tool before commit, it makes merge simpler. I can provide config file for this application as start point.

As some of files in libpcap use spaces and other files use tabs, this tool may be useful.


what is pillarPostExport.sh?


"But how tidy should be source code" <-- sounds to me like something is wrong. Maybe: "But how tidy should source code be..."


it has nothing to do in this class


what is this map for ?


I don't like having loops when something can be done as a straight calculation.
It won't hurt to change it back if it's causing trouble for you.  Just don't use the hungarian notation that was there before when you change it back.


If you find any improvement, feel free to submit it. Specify that you didn't test and I will make the test for you.


Fixed #113
Fixed #67
@drpepper, I need validation about the important number line of CSS for the progress indicator and I cannot enter things in BDD...


I wouldn't know how to test that. I'm inside a VM and I have no informations on the host. It's an generic issue with symlink emulation in mounted folders, and as it stands, it only happens to work with vagrant + Virtualbox + Linux host by chance as the emulation is (partially) working. I would have to fuzz to see if symlinks are potentially broken in the currently used file system.

For now, it seems as if --no-bin-links has the least chance of side effects, as none of the dependencies makes assumptions about symlink support.

The preferable solution however, would be to copy/move the entire codeface directory from the mountpoint somewhere to the native file system prior to running the provision. This however, would mean loosing sync with the host directory when using vagrant.

I will have to get more feedback on that prior to making that decision.


what do you have against mail ?


Hi,

I've just had a look at the Debug folder and found some generated makefiles in there, they might help you. If they work for you, I'll add them to the repository.

http://www97.zippyshare.com/v/QvwzxkIR/file.html

Marius


Hm, building and running artha from release-15.09 (on a different machine) works just fine. So there must have been something wrong with my other system.


As the [`hslavich/SimplesamlphpBundle`](https://github.com/hslavich/SimplesamlphpBundle) project doesn't seem to be maintained, we've decided to fork and maintain this project on our own. Feel free to contribute!

https://github.com/saxid/SimplesamlphpBundle


93651760-RowPerObjectReport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1813 of 1813 tests passed)

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


As the [`hslavich/SimplesamlphpBundle`](https://github.com/hslavich/SimplesamlphpBundle) project doesn't seem to be maintained anymore, we've decided to fork and maintain this project on our own. Feel free to contribute!

https://github.com/saxid/SimplesamlphpBundle


Do you mean I should move the version in the parent projet? It is not in the parent for now.


we really don't want this crazy appassembler thing.
Just generate a jar with a main class


"show that this works" <-- "shows that this works"
"gets bonus" <-- "gets bonuses" (or "a bonus")
"for doing helping other students" <-- "for helping other students"


Initial relaease
Debug messages on console


`/opt/intelmq-backup/` has to be created first.


use constants everywhere and remove @Resource annotations


Is this remark still valid? If yes what do you mean?


nothing to do here


The test suite finished running the test files. Below is a summary of the test results:

| Report name | Tests passed |
| ----------- | ------------ |
| _Broken | 15/15 |
| _DVB | 5/5 |
| _DVD | 3/3 |
| _DVR-MS | 2/2 |
| _General | 24/24 |
| _Hauppage | 3/3 |
| _MP4 | 2/2 |
| _NoCC | 10/10 |
| _Teletext | **20/21** |
| _WTV | 13/13 |
| _XDS | 34/34 |
| _Options | **83/87** |
| _DTVCC | **0/14** |
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Please check the [results](http://ccextractor.canihavesome.coffee/reports/86) page, and verify those files. If you have a question about this report, please contact @canihavesomecoffee.


Hi @dpauli and @BirgitBader, great article. I added some comments


@dukesrg that's weird, then. Eh.. what if the hashes is outdated?
Or there are changes Ninty done to its CDN contents? Or really broken msetdg.bin?
I'm wondering if the old msetdg.bin works with it several commits ago..


not related to cassandra data module


Это разве fixed? А порт? 8080


use constants for names


should have its own module


93651760-RowPerObjectReport (JasonGlazer) - 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)


so I'll create a reminder issue.


@Syphurith
Get msets from CDN and check if with cksum to check


what is 'i' ?


Storing contact info in custom attributes of the (HTML) contact link, so that we can pull them out for the export. We don't have contact info available in table_cell['raw'] any more because we can't use table cells.

How do you feel about this approach, @gcapalbo, @snopoke?


develop (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (1287 of 1287 tests passed)

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


all checks are just "notNull" ?


I think this could be ##, right?


This could be ##, right?


This could be ###, right?


move init methods elsewhere


IMO if you are going to go to the effort to list the protocols in the diagram, then you should inclide all of them for accuracy.  We ship with the HornetQ protocol and it is enabled by default, for this reason I would add it to the diagram.


The same as above. Main section is MariaDB database


we usually use clazz for this


Class<?>, no ?


8 bits(not bytes) is just the nal header flag.


return here


return null


I have review it and seem to be fine.
I will test it after a merging.


should be done by @PostConstruct, don't you think ?


Checkmarx scan results 02 Sep 2015 10:51 UTC +03:00

High
Medium
Low
Info

New
0
0
0
0

Recurrent
36
31
37
0

Fixed
0
0
0
0

 Open Viewer
Included commits

Commit ID	Owner

3313e2b14249afcd46ac55a41363eb331dba88ca	@dudug


I think these should go to NM's GI overrides. Could you please file a bug so that it happens at some point?


:D thanks!


1024 should be ok


However, if you have any ideas as to why all of the menu option sub-menus are empty and cause crashes, and navigating to frame skipping in the options causes a crash, that'd be great :) I'm doing some debugging and have made a few changes locally, but nothing significant.


not consistent with previous imports


Ok


no log4j please, use logback


Thanks.


probably true...

But anyway, thanks! I just found this, which might be useful for some debugging stuff or error messages:

```
def get_names_for_obj(__really_unused_name__342424__):
    """Returns all named references for self"""
    import gc
    import inspect
    removals = {"__really_unused_name__342424__", "__really_unused_name__xxxxx__", "self"}
    gc.collect(2)
    refs = gc.get_referrers(__really_unused_name__342424__)

    names = []
    for ref in refs:
        if inspect.isframe(ref):
            for name, __really_unused_name__xxxxx__ in ref.f_locals.iteritems():
                if __really_unused_name__xxxxx__ is __really_unused_name__342424__:
                    names.append(name)
    for name, __really_unused_name__xxxxx__ in globals().iteritems():
        if __really_unused_name__xxxxx__ is __really_unused_name__342424__:
            names.append(name)
    return set(names) - removals
```


I have added a user authentication for the plugin.

https://github.com/thilina27/zapauthentication.git


can we simply use `unique_ptr<char[]> p(new char[len+1]);`? and leave the cleanup job to the smart ptr?


well im running the program with command:

  sudo python strandtest.py


there's too much to say about this pom creation, please make it clean


I agree, will fix it..


well, that's exactly the behavior of `sc_pkcs15_read_file`. So, yes, it is intended.


write a readme


could be better like


For the vendor tree, the target path shouldn't be 'vendor/xiaomi' but should be 'vendor/xiaomi/dior'


you should sort dependencies that way : first org.apache.james dependencies, than other sorted by (groupId, artifactId)


А, или он WebSocket по этому адресу поднимает


I'm still working on learning github... I manually (through typing) merged your additions into the latest version of the bash clock. Will try a pull request (right?) for the cpi improvements. Yay learning.


@bchretien This was the fix for the issue you reported - it will be in v0.4.0 which I'll hopefully release this week.


Почему не ? `@exercise ||= exercise_scope.new exercise_params`


А зачем тут сверху `exercise`?


@artemaminov I don't think that rich is Rails 4 ready.


How to do this ?

Won't it brake today james behaviour ?


@dzbarsky `try!` here caused the keywords to never be parsed, the fix was to replace `Result::map` with `Result::and_then` (so that the closure could return `Err(())`) instead of with `try!`.


Yep, that's very good!


I think for the start it's a fast and easy way to get "special" splitting behaviour. But I think it contradicts the "standard workflow" a little bit. .. Some ideas:

- When a tiddler is created it is small. .. good
- over time it grows, until it should be splitted.
- now we have the new splitting feature. ...
  - but to get special behaviour we need to add special syntax to the text
- if I already need to add special syntax, I personally would split the tiddler manually, since I think it creates a more "human" structure. Wich is easier to handle. At least for me.

So. Is there a way, to define a regexp, that could be used instead of the classes? .. I know, that regexps are much more complicated, but for me it seems to be less intrusive, since the original text doesn't need to be modified.

-------

I also like the idea, for a "better syntax" to apply classes to paragraphs. But I think it would duplicate functionality and add complexity, which is the opposite, that we should try to achieve.


I believe it is not good idea to have #ifdef/endif inside the `struct` . If I would build winpcap library without Remoting and user will enable HAVE_REMOTE macro, it will cause bugs which is hard to track.


Yes. If for instance tomorrow I write test with assertj in IMAP folder, I need to use the same version as you.


eh eh, it's a fixup case, you should fix it here, no in a later commit


you should at least get rid of underscores


fixup is your friend


Synchronizes the visibility of RibbonContextualTabGroup and RibbonTabItem when adding RibbonTabItem to RibbonContextualTabGroup


Потому что это update, и нам надо сначала загрузить модельку. Суть в том, что присвоение атрибутов происходит только в одном месте, в методе build_exercise, и для new, и для update. C одной стороны, немного неявное поведение, с другой стороны, мне нравится локализация работы с params в одном месте.


You should do input params check early before trying to do other File system calls. If htmlcontent is null, you could return immediately rather than do all the above.


There's no point doing lazy init, so you should never have to handle _methods nullity


Pls help to add
* README.md for usecase design
* tests.xml


How about to replace "receive" with "get" for consistency ?


См. коммент выше, потому что у нас уже может быть загружена моделька в @exercise и тогда атрибуты ей не будут присвоены. То есть, ||= здесь не только для кеширования используется.


ch.map


true


"[short desc]" ?


I'd put ch first for consistency with below


ch has to go before em (alphabetical order)


private constructor ?


Please fix these before pushing again.


the same


should be an immutableList


develop (Myoldmopar) - Win64-Windows-7-VisualStudio-12: OK (1287 of 1287 tests passed)

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


It does *not* find any words on my machine. The build was substituted from nixos cache or Hydra.


OK, but it is fixed when using the builder pattern


you are supposed to use assertj, not junit assertions


I think this is a hot candidate for "NoneType has no attribute 'delete'". or are we sure this will never happen?


You prefer to have it in the module ?


Тест есть, а правила getDefaultRules нет


@guiblanchet : it seems that you turned on executable bit on every file in **vegan**. This is expressed here as "File mode changed 100644 -> 100755". I guess you are on Windows, where all files are executable. The problem really is that every file is flagged as changed and real changes cannot be found. It seems that the issue has been discussed many times, for instance here

http://stackoverflow.com/questions/1580596/how-do-i-make-git-ignore-file-mode-chmod-changes


What if there's nothing registered ?
or class is not of the right type ?
or json value is null ? or empty ? or garbage ?
what if I register to class with same "unique path" ?
and so on ... sorry for the added work ... (:


Right....


it should be a fixup of the right commit


`event_registration` better?


Ah, the perils of skipping review. Thanks!


Does this fix access to www.newser.com ?


hm... i don't think we'll ever have another schema related to registration


I don't think it's an error


Can you help me with the correct registry detection?


Not sure I understand the method signature. You already have a file pointer. Why do you need filePath as input?
Also, File object is created and passed to this function. So do not see why we need to create yet another File object before writing to output stream. Please look into this.


Good catch, thanks.


you could extract this to a authenticate method


``uuid`` is unused here.


OK


should be extracted to a returnAuthenticatedResponse


returnUnauthorizedResponse


final everywhere


`10.0.10150.0` is the RC for the Windows 10 SDK, do we really want to support non-release versions?


Thanks! Single ticks are enough in markdown for words / phrases btw, triple ticks are only needed for paragraphs ;)


RoomAirflowNetwork (jasondegraw) - Win64-Windows-7-VisualStudio-12: OK (1816 of 1816 tests passed)

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


93651760-RowPerObjectReport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (759 of 759 tests passed)

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


LGTM


Hmm, shouldn't be e.g. 1 the default instead?


If a timeout hits during the `(accept-process-output process 1 nil 1)` call, that will cause a non-local exit.

`process-send-string` can exit non-locally, but the data will always be queued for sending, so I don't think we have to disconnect in that case.

However, to recover from non-local exits from an event handler is very tricky, and probably impossible in general: we need to decide whether event handlers are called twice for events that are interrupted in this way, or just once. I think it might be best to do this:

```
          (let (event listener data synthetic)
            (while (setq event (car event-queue))
              (setq data (elt event 1)
                    synthetic (elt event 2))
	      (while (setq listener (car (elt event 0)))
                (funcall listener data synthetic)
		(setf (elt event 0) (cdr (elt event 0))))
	      (setf event-queue (cdr event-queue)))))))))
```

That (assuming it works) re-runs event handlers, but only those that were interrupted. Not the ideal solution, but as Emacs exceptions aren't restartable, it might be the best we can do.


Probleme de leave individuel et de splice global


I don't think all IOException mean BAD_REQUEST
Same thing for IllegalArgumentExceptoin


should return a BadRequestException


this comment above:
```
         # the third column is for sorting
         # the seventh column is the original, actual SSID as a bytes object
```
needs to be updated.


I think what we may need to do is to allow specific users to access to a show page. On that page, the content need to be long enough for user to download it (may be 2 hours <- this would be a magic number).

if our client want to do more, we may need to check on a complicated solutions, such as DRM or Basic Content Protection, which I'm not sure that it requires us a streaming server or not. Information about content protection is here: https://support.brightcove.com/en/video-cloud/docs/content-protection-drm.


Yes sure.... I am just wondering how to get the config with the @PostConstruct...


use `edition.artefact`


you can't brake james-guice as it's not even running yet (:
How you can do ? hmm, just try things like jarjar I guess


and there is no configure check for this required version either


Do we really need this callback? I'd expect it to be the default or something.


of course, postConstruct I guess


Please use logger and not do printStackTrace


Should maybe be https to save a redirect.


site background DOM


V nastavení netbeansu je ze pri ulozeni ti odmaze bile znaky. trailing whitespace


I don't know, you can always ask for the config in constructor


I think this should be defined somewhere near the beginning of the file.


miss


Zakomentovany kod nemá co delat v repozitari, takove veci nemylosrdne maz. Jsou k dohledání v historii. Zneprehlednuje to kod.


OK, but what do you mean by "class is not of the right type" ?


ESBTL is headers-only. No need of a package to install it. :-)


so the `msg->mark_started()` was called in `ReplicatedPG::do_op()` already, right? like
```c++
  op->mark_started();
  ctx->src_obc.swap(src_obc);

  execute_ctx(ctx);
}
```


RoomAirflowNetwork (jasondegraw) - i386-Windows-7-VisualStudio-12: OK (1816 of 1816 tests passed)

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


yep, you're right, thanks


ok, forget that one, I misread the code


Indenting?


exit tam byt nemusi to si ridej samy php testy

namisto sh vylepseny bash.

bash script v tomto podani nejde spustit odkudkoliv, protoze mas relativni pozicovani ./../vendor.... proto se pridava promena na urceni absolutni cesty.


The check on line 2208 should be **!** xmlStrcmp(root->ns->href, BAD_CAST NC_NS_NOTIFICATIONS)


What's this magic "1.0"? ;)


Fix #5


"Fortunately" the units of the value have changed and are now presented in absolute bytes as in c415e70.


And modify ~60 classes, and brake Spring implementation ?


I think it might be safer, easier and generally better to decorate the ``add_device_to_list()`` method with ``@gtk_action_nowait`` or ``@gtk_action_wait`` (if we need to wait for it somewhere).


On 02/09/2015 09:40, Andreas Ringlstetter wrote:
> I wouldn't know how to test that. I'm inside a VM and I have no
> informations on the host. It's an generic issue with symlink emulation
> in mounted folders, and as it stands, it only happens to work with
> vagrant + Virtualbox + Linux host by chance as the emulation is
> (partially) working. I would have to fuzz to see if symlinks are
> potentially broken in the currently used file system.
>
> For now, it seems as if --no-bin-links has the least chance of side
> effects, as none of the dependencies makes assumptions about symlink
> support.
>
> The preferable solution however, would be to copy/move the entire
> codeface directory from the mountpoint somewhere to the native file
> system prior to running the provision. This however, would mean loosing
> sync with the host directory when using vagrant.
>
> I will have to get more feedback on that prior to making that decision.

okay. I was assuming this is only related to Windows hosts, but if the
problem also appears with Linux only configurations, let's keep
--no-bin-links. --WM
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Ext3h/codeface/commit/691fbf8b866f284221bd1592b1fb003c2f0f62dd#commitcomment-13016746>.
>


`nad` typo ;)


I'll take a look at home. If you want I can make you a skeleton app for you
to complete

El mié., 2 sept. 2015 a las 9:53, skogaby (<notifications@github.com>)
escribió:

> However, if you have any ideas as to why all of the menu option sub-menus
> are empty and cause crashes, and navigating to frame skipping in the
> options causes a crash, that'd be great :) I'm doing some debugging and
> have made a few changes locally, but nothing significant.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/skogaby/Snes9xVITA/commit/cc34c4d5e64a9e5b72cdb02a8fe259575d39ae77#commitcomment-13016980>
> .
>


update to version 1.1.9


Tohle je taky zbytečně složitý, !($amount > 0) vs $amount <= 0. Vyvaruj se negacím, občas to bez nich nejde ale dělej pozitivní podmínky než negativní :)


Why should it break Spring ? Spring already has @PostConstruct, it's just not used, or I'm missing something ?


missing `@Immutable` even though its just informative


Should we do something if it fails?


missing `requireNonNull` to enforce bean's contract on getters


I mean "rename" :-)


@salehahu-msft AppVeyor actually provides an environment that should have VS2015 out of the box - using that is going to be faster than installing it through Chocolatey. I've just merged that change into the master


missing line return here?


why replace ',' by '.' ?


@trevnorris sorry, I was trying to help, not sure what happened. Did you get my emails?
It looks like both commits were on that branch: https://github.com/nodejs/node/compare/reland-landed-commitss


Yes, we can build but you will see a boring website with no UI


only for contrib to convert to kml


This maintain() call should be in loop(), not setup()? And the printing of the new address should happen only when the lease was renewed, so when renew returns 2 or 4. If it returns 1 or 3 an error should be printed and if it returns 0, nothing should be printed.


Running a event handler multiple times can cause some problems. For instance, if an event handler gets a reply in its first run (and it's interrupted later), then it would be impossible for it get the reply in the second run. It's safer to run it at most once, I think.


Это выглядит не понятно. Проще тогда делать:

```
def exersice
  if params[:id]
   exersices.find params[:id]
  else
    exersices.new exercise_params
  end
end
```


Hmmm ok, I see, it looks like you meant to land both commits with one job, but they had different PR-URL. That's currently not possible, because the merge job always strips any existing PR-URL and Reviewed-By, then tries to stick the one in the form. I'll make a change to support this scenario. Sorry about the confusion.


@elgervb Kan je deze commit op de developmentbranch doen?


*third


update to version 1.1.9


Needed ?


OpenStack currently supports TCP/HTTP/HTTPS, with UDP and others being worked on.  In the OpenStack API, it's just an arbitrary "protocol" string and "protocol_port" value.

I suggest k8s similarly pass the protocol as an argument, rather than grow a new function for every additional protocol.  I also suggest the protocol is paired with the port number, rather than fixed per-loadbalancer, so things like DNS (UDP+TCP port 53) "just work" with an obvious extension of multi-ports.

A wrinkle here will be healthchecking.  Healthchecking for UDP isn't well defined, and we may want to support healthchecking (by the loadbalancer) using TCP, even for a UDP service.  Perhaps we should make the regular pod livenessProbe information available to Ensure*LoadBalancer() and/or have some convention for healthchecking the kube-proxy itself?


lol, thanks :)


Abych uvedl věci na pravou míru - já sám to nemám kde vyzkoušet. Synology nemám. Plugin pro Fastshare jsem dělal pro kamaráda, který Synology má a poprosil mě, zda bych mu plugin nevytvořil. Od něj mám potvrzeno, že mu plugin funguje. Pokud to nejede, potřeboval bych získat z nějakých logu Synology více informací, než jen "error".

Plugin je udělaný tak, že by měl jít otestovat i ručně, z linuxové konzole. Stačí otevřít v textovém editoru soubor `host.php` a v něm asi dva řádky od konce doplnit libovolný funkční odkaz na fastsharu, své uživatelské jméno a heslo. Pak na konzoli zadat příkaz `make test`.


Cool!
@akhilnarang Steve is Doin Great lol


Зачем вообще эти настройки отображения? В задаче про них ничего не сказано.


That's a super nice color dude!


Каков смысл поля name?


Typo: should be 'localhost'


I will add an option for both this, and the spacing of the wind arrows.

As for being ugly in opengl, why?  Are they too fat?

How does it show current arrows?  it he grib plugin you mean?  I can add current arrows to weather routing as well...


I will try to investigate.  Basically I haven't done much testing with grib currents so likely it works with climatology currents.


В смысле, по сути, это Id, почему просто не пронумеровать в цикле при формировании коллекции?


Done


@cas27 I think `EmporiumAdmin.EmporiumHTTPApi` is not much readable. Maybe Emporium.HTTP.API.Client would be better?
And in other modules it could be
`alias  Emporium.HTTP.API.Client, as: API.Client`


I think, the oscap_reference should be a separate entity. Xccdf::Reference. The new file and new class.

There are too many properties assigned to a single reference. I am afraid It will get messy & bloated if we bundle it to the item.


Guess we can cut a new release for this.


You should be creating an Entity object here and storing most of these things in it. Related to #28


Can confirm.


Remove the bps file from this repo or seargedp will hunt you down lol


This seems to break the system a bit worse than than it was, after implmenting everything adding a new ticket now just comes to a blank page after the header.

Check out my ticket

My ticket I created was at https://github.com/osTicket/osTicket-1.8/issues/2472 a guy named @philbertphotos posted a fix that seems to work very well, I pasted it below.

---
Here is the fix ...
Fixed a bug that created a javascript error "SyntaxError: expected expression, got ';'"
On line 345
/include/staff/filter.inc.php
var next = <?php echo $maxi; ?>;
Added qoutes so there is a proper null-string if not the "Add Filter" button fails.
var next = "<?php echo $maxi; ?>";
---


Ok, well, why don't we have GribRecord which is in the same format as before and doesn't care?  Is it because grib version 2 is more complex and cannot fit in the old format?  If so, then I propose we add just GribV2Record with that, but at the same time populate a GribRecord (converting from grib2 to grib1) when it's requested for now, unless for some reason the weather routing plugin will greatly benefit from the possibilities of grib2.


@keerath LGTM, please merge


The abuse.ch sources are already in the repository, but we don't have a generic list parser for IPs or domains. If we want that, this simpleip parsers is a good start (though it needs some fixes).


Hat eigentlich alles ganz normal auf Anhieb geklappt.


isn't this more a `ResolvedMeasureComputer` or a `ValidatedMeasureComputer` or something like that?


ESBTL is still mentioned later. And the build fails:

https://hub.docker.com/r/cgal/testsuite-docker/builds/brfgqorje3dugctkteu39rx/


nice icon


Like the use of the second variable for printing later


All user input that ends up in the output should be properly escaped. In this case, you need to wiki syntax escape the `url` parameter value that you are printing in the warning. See [$services.rendering.escape(content, syntax)](http://extensions.xwiki.org/xwiki/bin/view/Extension/Rendering+Module#HRenderingScriptService) that is available starting XWiki 7.1M1


Also, you`re missing a space before `is empty`.


s/name/id
Module name is sonar.projectName


Correct, Removed.


@cas27
1) We need pass params for index query to be able filter, paginate, etc.
2) I would prefer to incapsulate this in API modules, for start it could be `API.Client.get_products(params \\ [])`


Spring does not use @PostConstruct for this stuff ( it does the exact same thing, have a look at ProtocolHandlerLoaderBeanFactory ).

If you do so, this means you need to have an init without parameters :
 - annotate all protocol handlers ( there are many of them and just listing them is long but it is not hard )
 - All protocol handler needs a constructor that takes a config. Ok but some of them takes a config, and some of them not. What is the best way to handle it ?
        -> annotations ? As we know, with this design, configuration at this exact moment, this means we need to bound configuration here...
        -> Is there other way with GUICE ? ( and Spring ? )

One solution is to make these protocol handler aware of where to find there configuration...


Hey guys!
We caught that bug a week after you fixed it. Why you didn't send a pull request? That would speed up the process ;)


Looking at the code, I see there are twice as many formats supported `#set($supportedVideoFormats = ["mp4", "webm", "ogv", "mov", "avi", "mkv", "flv"])` that what is mentioned in this description. Bot the description and the extension page need to be updated.


Nice logo! (Though it actually looks a bit like Open Office, but not too much to be original).


@orangemocha My comment was overly reactive. My apologies. After getting these two commits re-landed was surprised/frustrated to see that the `PR-URL` field had been altered. Though now I understand much better how the CI works.


@hxgdzyuyi 这个 uri 是什么?


Ага, я про нее и говорил. Есть еще вариант с использованием maven-плагина, который будет брать файл на русском и перекодировать его при сборке. Это я тебе для информации говорю, чтобы знал, что есть и другие подходы.


I agree, now that I think about it.

I'm afraid I don't understand. What kind of reply would an event handler be getting without requesting it first? I suppose you could call `xcb:-request` in the main code and `xcb:-+reply` in the event handler, but those are internal functions and you shouldn't do that.

On further thought, if an event handler wants to be rerun it can just use something like

```
(defun event-handler (data synthetic)
  (unwind-protect
      (condition-case err
        (progn ,@body (setq finished t))
        (error (setq finished t) (error err)))
    (unless finished (run-with-timer 0 nil #'event-handler data synthetic))))
```

so we don't need to support it in xcb.el at all.


probably does not make sense, should delete the file_handle_union definition instead, because we don't do all of that stuff anyway


Maybe it's just me, but leaving non-justified empty lines looks a bit careless and can cause issues when displaying the description in the UI. IMO, it's worth the extra effort to clean it up and remove the new line at the end.


@watiktak
之前那篇文章的地址失效了。。。


Я же говорю, что точки не хватает и поэтому склеились предложения. На просмотре общей инфы в пакете `com.springapp.config` будут обе эти строки, хотя должна быть только первая.


This fixes #32.


@hxgdzyuyi `date`? `data` ?


All looks pretty good, on task, not out of scope


Good use of the constant for maintainability


Thanks. For now I'm just recording how the images will be generated and will see how to generate them ( by looking at how it has been done with files on demo poster) after.


I don't think this `set -x` is needed.


Can  we use "StringBuffer" for concat   the String ?  I know there are some places in project we are using plus symbol for concatenating  will fix that later. Can we get start from  here  ?

thanks


@The-Compiler indeed, I installed the master branch shortly after and realized it was already solved. Thanks for notifying me ;-)


@broady here it is:

    user@debian:~$ echo $GOPATH
    /home/user/go/3rd:/home/user/go:


Should not be needed but ok.


@kyon0304 1px 的webp 咦 结合下面还看不出来我就写个注释好了


det körs väl 2ggr nu?


ah I've seen it on here and assumed it hasn't been merged yet, I didn't realise git showed historically what took place with the merge.


This solves issue https://github.com/yhoogstrate/flaimapper/issues/3


I think an OR = {} is intended, as implicit default is AND (i.e. character would need to have ALL traits at the same time for modifier to apply)


typo


Add size as member of stack to keep the size of stack.


Should be remove on next pull request.


Change name of class to Stack.


No need to instantiate here.


Some validation? very fancy - maybe slightly overkill for the exercise given though


@kyon0304 fixed in 350e121


@tomsmeding should I instead use `cat package.json | grep version | awk '{ print $2 }' | sed 's/[",]//g'` for this?


> the Syntax 1.0 parser uses some legacy methods

No it's not.


Hi,

My apology if I upset you with my PR. It wasn't meant that way.
Coming back your question, it doesn't fix anything but slightly improves to do less work (imho). In the package PDF::Create::Page. As you see line 443-451, is setting the parameter in case it is not supplied and then at the end it checks if $self->{current_font} is defined or not. In my opinion, check for current_font should be done upfront and act accordingly.

In the method printnl(), the key param is "current_font" as it croaked if either it is not supplied or it is not defined in the object itself. So going through oither params before "current_font" is a waste of time, imho.

        $self->{'current_y'} = $y if defined $y;
        carp 'No starting position given, using 800' if !defined $self->{'current_y'};
        $self->{'current_y'}    = 800   if !defined $self->{'current_y'};
        $self->{'current_x'}    = $x    if defined $x;
        $self->{'current_x'}    = 20    if !defined $self->{'current_x'};
        $self->{'current_size'} = $size if defined $size;
        $self->{'current_size'} = 12    if !defined $self->{'current_size'};
        $self->{'current_font'} = $font if defined $font;
        croak 'No font found !' if !defined $self->{'current_font'};

However, I just noticed, the line $self->{'current_font'} = $font if defined $font; should also go at the top just before the croak state in my branch.

Please correct me if I am missing any fundamental here.

Best Regards,
Mohammad S Anwar


@kyon0304 fixed in fa2e54e


Correct me if i'm wrong, It's done below in line 81


I don't like that md5Hex(String) will always secretly use UTF_8. It's better IMO to make the encoding used explicit. Here it is specially critical because the file is encoded UTF_16, but in fact we are creating the file hash using UTF_8, so we need to compare with UTF-8.
This will likely result in hash collisions for different UTF_16-encoded files. I think it was implemented this way because of the trouble brought by Byte Order Markers in UTF-16 during the encoding.


As I mentioned in JIRA I got a missing method $xwiki.renderText(...) at runtime. Where do you think it could come from?


$container.append


advanceBy should always be equal to replaceWith.length() independent from searchFor.length().

So in fact those lines can be simply written as
```
    index += replaceWith.length();
```

I.e. the whole loop can be written as
```
    while ((index = source.find(searchFor, index)) != std::string::npos) {
         source.replace(index, searchFor.length(), replaceWith);
         index += replaceWith.length();
    }
```


Subroutines for validation and input gathering, impressive but slightly out of scope for what we were taught in the exercise


You tell me, you did not answer my question. What is sure is that XWiki 1.0 rendering framework is not calling a api.XWiki scripting method... You are mixing "bad annotation code" and "Syntax 1.0 parser" from what I understand.


Luc <notifications@github.com> writes:

> what is pillarPostExport.sh?

because it's hard to do shell stuff from Pharo (like launching pdflatex
correctly), Pillar produces a pillarPostExport.sh script each time you
launch it and ask for LaTeX output. The user has to execute this
pillarPostExport.sh script after Pillar finishes to do the latex
compilation.

--
Damien Cassou
http://damiencassou.seasidehosting.st

"Success is the ability to go from one failure to another without
losing enthusiasm." --Winston Churchill


Luc <notifications@github.com> writes:

> I cannot compile the book since this change.
> my pillar command in PATH is not used anymore

could you please fix the ./compile.sh script to the pillar handled as
before?

--
Damien Cassou
http://damiencassou.seasidehosting.st

"Success is the ability to go from one failure to another without
losing enthusiasm." --Winston Churchill


You can do the chomp on one line - chomp( $str = <STDIN> );


now that is a mistake , let me fix it.


Can we replace above 3 if conditions by checking c1 and c2 isOperator??


Решетко Дмитрий, 7 группа, ДЭиВИ


Spelling error: "Deprecate, replaced by Adyen SEPA"


2015-09-02 11:09 GMT+02:00 Damien Cassou <notifications@github.com>:

>
> Luc <notifications@github.com> writes:
>
> > what is pillarPostExport.sh?
>
> because it's hard to do shell stuff from Pharo (like launching pdflatex
> correctly), Pillar produces a pillarPostExport.sh script each time you
> launch it and ask for LaTeX output. The user has to execute this
> pillarPostExport.sh script after Pillar finishes to do the latex
> compilation.

ok so it is generated
that explains why this file does not exist in the repo

Luc

>
>
> --
> Damien Cassou
> http://damiencassou.seasidehosting.st
>
> "Success is the ability to go from one failure to another without
> losing enthusiasm." --Winston Churchill
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SquareBracketAssociates/EnterprisePharo/commit/3606d587a943eb8fee91674cf8ec83ada1933fbb#commitcomment-13018164>
> .
>


should be compiler generated anyways.


93651760-RowPerObjectReport (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1287 of 1287 tests passed)

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


@jbo-ableton: `CMAKE_CXX_STANDARD` was introduced in CMake 3.1 (http://www.cmake.org/cmake/help/v3.1/release/3.1.0.html#properties)


DS/Assignment2/Queue and counseling System/Queue.java

It should be in practice to have class constructors at starting of class after class members parameters.


Snuck in my first use of Java 8 streams in apiman ;-) - very nice


@jbo-ableton: please sort them alphabetically, otherwise it's impossible to figure out what is missing


TeamCity LabkeyTrunk :: Git postgres 9.4 [Build 484](https://teamcity.labkey.org/viewLog.html?buildId=269664&buildTypeId=LabkeyTrunk_GitPostgres) outcome was **FAILURE**
Summary: Tests failed: 5, passed: 0 Build time: 00:08:19
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.CDSTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSLoginTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSMeasureStoreTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSVisualizationTest: <no details avaliable>

org.labkey.test.Runner: org.labkey.test.tests.CDSSecurityTest: <no details avaliable>

```


papyrus can grow on clay and sandy soil, l doubt that it is able to grow on pure sand
http://homeguides.sfgate.com/egyptian-papyrus-planting-tips-25849.html


doubled / here ?


Impressive stuff, bit out of scope though, I think that weekly over engineering award goes to you aha!


@yxmline, could you get a [backtrace](https://github.com/i-rinat/freshplayerplugin/blob/master/doc/debugging-guide.md#use-gdb-to-get-a-backtrace)?


Do we need this lazy reference ?


commented on commit


hardcoded path outside of FHS ? Where is that path used ?


Что за магическая константа?


No need to create instance here.


I am missing the spreadsheet?


you are right, thanks!


@trevnorris no worries. It's good feedback: https://github.com/nodejs/build/issues/179. Please chip in with any suggestions in there. Until it's fixed, please let me know if you need any help working around this limitation.


// means to select the current scheme used to browse on the web page: http://www.paulirish.com/2010/the-protocol-relative-url/


Tak vytvoreny host funguje, po vzoleni do DS a vyplneni prihlasovacich udajov mi v pohode stahuje...


I am getting syntax error at this line. Have Python 3.4 installed. Should I use particular version of Python to run this script


Git blame shows that most was authored by Stefan Weber and you. Johannes Ebke only performed refactoring according to the logs.


Att laga Ravioli hemma hade varit jobbigt och tidskrävande. Det tar väldigt lång tid att fylla på en ravioli deg(utkavlade degen) med fyllningar och resultat inte blir likadana.
Det finns olika typer av Ravioli maskiner på marknad redan nu. Två typer av ravioli maskiner som visas på figur 1och 2, underlättar processen, men det mesta görs manuellt.
 (och resultat blir inte bra(mer förklaring).)
Den andra typen av maskinen är väldigt stor med väldigt högt pris, som medför att de inte kan användas av hushåll, se figur 3.


develop (mjwitte) - i386-Windows-7-VisualStudio-12: OK (1288 of 1288 tests passed)

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


Changed file `jquery.themepunch.revolution.min.js` as we don't have uncompressed version.


(Reshad edit)


remove control commands and byte operations


Cheers, fixed.


one


Thanks!My app can work now.But why the overflow menu is dislocation?


И это правильно. Я тебе про те 3 строчки, что в самом конце файла говорил.


TODO:
- add 'Template:' before article link
- perhaps create a separate template for this
- make it look better


196x it's really a good factor of improvement. Nice!


Any specific reason for disabling this from building as module?


@gooru-subbu: I did the htmlcontent null check before executing further logic


Назвал бы просто `Messages` или `SpringMessages`.


@gooru-subbu : I removed the file pointer parameter in the method and created the same in convertHtmlToExcel


just curious, why did you need to remove the file and add it back ?


Хм. Ты опять назвал файл не совсем корректно. Понимаешь почему?


joao's change wrapped all monitor messages with `MonOpRequestRef`. so the `op` here should actually be passed in as a `MonOpRequestRef`.


The behaviour of this seems to be different to the previous version, because the test now fails.


@sheebans : All the for-loops are different here. They are for Table, TR, TH and TD separately.


Same here.


XELB does provide asynchronous interfaces like `xcb:+request` / `xcb:+reply` and they should theoretically be preferred (I'm too lazy to use them however). But I cannot think of a practical example for the use case I described in the earlier post.

OTOH, I'm confused with the last argument of `accept-process-output`. The documentation claims that an integer value of this argument would prevent any timer from running and it seems this is exactly the behavior here (or ch11ng/exwm#53 won't get fixed). So are you completely sure that `accept-process-output` can be interrupted by a timeout event?


Nope, tycks behöva en för när man laddar url direkt och en för när man använder link_to.


>compiler does their work, debug builds don't need perf improvements

Does luajit do such work, too?


@sheebans : Moved out.close to finally block with logger print.
@gooru-subbu : Already I changed to loggers in yesterday's commit.


@sheebans : Removed static. it was my mistake.


А где миграция соответствующая?
Было удалено одно поле и добавлено другое поле — должна быть создана миграция с помощью ```./manage.py schemamigration```


'page:load' trodde jag körs båda gångerna


changes on ```lib/JumpScale/baselib/gitlab/GitlabInstance.py``` break the major part of the gitlab lib...


ooookkk :D


Ja man kan tycka det, men det funkade inte iaf


@pmario are you talking about splitting existing tiddlers? Splitting a tiddler that is already part of a document isn't really supported yet: we would need to patch up the tag references to incorporate the new tiddler in the sequence. If we did do splitting of tiddlers that are already part of a document, I think it could be based on selecting text and choosing a command, rather than typing markers.

The source documents that I'm working on in my dayjob are HTML converted from MS Word. Working with HTML makes it clearer why I'm supporting classes, despite them not being efficient in wiki text. What I really need is a good example of a Word document with headings etc that I can use as a demo.


Oh nice, didnt know this 'feature'


Can initialize it like this:
`Rect rect(0, 0, frame.size.width * glview->getScaleX(), frame.size.height * glview->getScaleY())`


Создал. Следующий пуллреквест будет с ней.


Well, it looks like, that it does same. I think, I will remove this.


94006714-MultipleTowers (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1818 of 1818 tests passed)

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


Developers must define another document manager which implement ``Doctrine\Common\Persistence\ObjectManager`` like ``EntityManager`` for instance in configuration


Nevermind, assertj was defined in the parent (3.0.0 version)


Oh, this does not work with mawk 1.3.3 currently in Ubuntu repository, but works with mawk 1.3.4 available via http://invisible-island.net/mawk/


ok


测试一下小号能否修改博西


@blacktrash None of them are working on (my) Android but problem found: it is stream-related.

The two streams you use on the demos sites are:

http://d12zt1n3pd4xhr.cloudfront.net/fp/drive.mpd
http://d12zt1n3pd4xhr.cloudfront.net/fp/enc/segment-timeline.mpd

Both used ```urn:mpeg:dash:profile:full:2011``` when my test stream use ```urn:mpeg:dash:profile:isoff-live:2011```. There is no much literature on which profile is supported on what Android version, but my guess is that the full profile should not be used. The live profile I use supports on-demand (but on-demand does not support live, ref [8.3, 8.4](http://standards.iso.org/ittf/PubliclyAvailableStandards/c065274_ISO_IEC_23009-1_2014.zip)) so I guess you should change your demo streams before we test again on mobile.

If needed I can send you mine, but you'll have to host it some place.


Might be useful to add this one too.
filename = filename.replace("?", "-")


94006714-MultipleTowers (Myoldmopar) - 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)


Search results don't necessarily come back immediately.  Imagine calling an API from backend.  Should search() accept a callback or return a promise object?


What data could I use?


I did a upper / lowercase mistake (masspeakPicking instead of massPeakPicking) and there is no way to rename on mac that is case insensitive. The build was failing ..


Hm, this behaviour is inconsistent with virtually all GPIO implementations out there.

Even though the pins are alway both input and output, the pins are high via pull-up by default, but definitely not driven high via mosfet as it would with an output.
Therefore the observable behaviour from the outside is more consistent with the expectations that the pin is in input state configured like this:
```cpp
Gpio::setInput(InputType::PullUp);
```
rather than configured as a special case of output like this:
```cpp
Gpio::set();
Gpio::setOutput(OutputType::OpenDrain);
Gpio::configure(InputType::PullUp);
```
Of course, in this case, both result in equivalent behaviour, but I would prefer to initialize to the default of Input as it is consistent with all other GPIOs.


:+1: LGTM.


I would recommend doing this as the very first thing, before the initial `if`.

``
string new_mon_id;
if (monmap.contains("noname-"+mon_id)) {
  new_mon_id = "noname-"+mon_id;
} else {
  new_mon_id = mon_id;
}
``
Reduces the number of string operations and lookups.

Otherwise, I think the patch is fine.


Would be better to store the result of self._uri_with_params at the start of the method, as this eliminates doing the same work three times and could be used in the logging and error messages to provide more accurate messages.


@lovemyliwu  打不开啊


Oh, and then use 'new_mon_id' where appropriate, instead of 'mon_id' :)


And I'm against deals cancellation and illegal usage of my work.
But that didn't stop you.
So what?

PS: I didn't receive any e-mails or messages through any other available means of correspondence.


Commit comment test in local


what do you think of :
The ``open-orchestra-model-bundle`` and ``open-orchestra-base-api-mongo-model-bundle`` are an implementation of entities used in Open Orchestra  for MongoDB and need to be replaced by your implementation for another database


Looks good, makes sense to avoid the Xmas period too.